drm/udl: Don't track number of identical and sent pixels per line
A call to udl_render_hline() returns the number of identical and sent pixels. None of these values is used. Remove the parameters. Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de> Reviewed-by: Emil Velikov <emil.velikov@collabora.com> Acked-by: Gerd Hoffmann <kraxel@redhat.com> Link: https://patchwork.freedesktop.org/patch/msgid/20191206085954.9697-3-tzimmermann@suse.de
This commit is contained in:
@@ -93,8 +93,7 @@ udl_fb_user_fb_create(struct drm_device *dev,
|
|||||||
|
|
||||||
int udl_render_hline(struct drm_device *dev, int log_bpp, struct urb **urb_ptr,
|
int udl_render_hline(struct drm_device *dev, int log_bpp, struct urb **urb_ptr,
|
||||||
const char *front, char **urb_buf_ptr,
|
const char *front, char **urb_buf_ptr,
|
||||||
u32 byte_offset, u32 device_byte_offset, u32 byte_width,
|
u32 byte_offset, u32 device_byte_offset, u32 byte_width);
|
||||||
int *ident_ptr, int *sent_ptr);
|
|
||||||
|
|
||||||
struct drm_gem_object *udl_driver_gem_create_object(struct drm_device *dev,
|
struct drm_gem_object *udl_driver_gem_create_object(struct drm_device *dev,
|
||||||
size_t size);
|
size_t size);
|
||||||
|
@@ -68,8 +68,6 @@ int udl_handle_damage(struct drm_framebuffer *fb, int x, int y,
|
|||||||
struct udl_device *udl = to_udl(dev);
|
struct udl_device *udl = to_udl(dev);
|
||||||
int i, ret;
|
int i, ret;
|
||||||
char *cmd;
|
char *cmd;
|
||||||
int bytes_sent = 0;
|
|
||||||
int bytes_identical = 0;
|
|
||||||
struct urb *urb;
|
struct urb *urb;
|
||||||
int aligned_x;
|
int aligned_x;
|
||||||
int log_bpp;
|
int log_bpp;
|
||||||
@@ -115,8 +113,7 @@ int udl_handle_damage(struct drm_framebuffer *fb, int x, int y,
|
|||||||
const int dev_byte_offset = (fb->width * i + x) << log_bpp;
|
const int dev_byte_offset = (fb->width * i + x) << log_bpp;
|
||||||
if (udl_render_hline(dev, log_bpp, &urb, (char *)vaddr,
|
if (udl_render_hline(dev, log_bpp, &urb, (char *)vaddr,
|
||||||
&cmd, byte_offset, dev_byte_offset,
|
&cmd, byte_offset, dev_byte_offset,
|
||||||
width << log_bpp,
|
width << log_bpp))
|
||||||
&bytes_identical, &bytes_sent))
|
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -127,7 +124,6 @@ int udl_handle_damage(struct drm_framebuffer *fb, int x, int y,
|
|||||||
*cmd++ = 0xAF;
|
*cmd++ = 0xAF;
|
||||||
len = cmd - (char *) urb->transfer_buffer;
|
len = cmd - (char *) urb->transfer_buffer;
|
||||||
ret = udl_submit_urb(dev, urb, len);
|
ret = udl_submit_urb(dev, urb, len);
|
||||||
bytes_sent += len;
|
|
||||||
} else
|
} else
|
||||||
udl_urb_completion(urb);
|
udl_urb_completion(urb);
|
||||||
|
|
||||||
|
@@ -212,8 +212,7 @@ static void udl_compress_hline16(
|
|||||||
int udl_render_hline(struct drm_device *dev, int log_bpp, struct urb **urb_ptr,
|
int udl_render_hline(struct drm_device *dev, int log_bpp, struct urb **urb_ptr,
|
||||||
const char *front, char **urb_buf_ptr,
|
const char *front, char **urb_buf_ptr,
|
||||||
u32 byte_offset, u32 device_byte_offset,
|
u32 byte_offset, u32 device_byte_offset,
|
||||||
u32 byte_width,
|
u32 byte_width)
|
||||||
int *ident_ptr, int *sent_ptr)
|
|
||||||
{
|
{
|
||||||
const u8 *line_start, *line_end, *next_pixel;
|
const u8 *line_start, *line_end, *next_pixel;
|
||||||
u32 base16 = 0 + (device_byte_offset >> log_bpp) * 2;
|
u32 base16 = 0 + (device_byte_offset >> log_bpp) * 2;
|
||||||
@@ -237,7 +236,6 @@ int udl_render_hline(struct drm_device *dev, int log_bpp, struct urb **urb_ptr,
|
|||||||
int len = cmd - (u8 *) urb->transfer_buffer;
|
int len = cmd - (u8 *) urb->transfer_buffer;
|
||||||
if (udl_submit_urb(dev, urb, len))
|
if (udl_submit_urb(dev, urb, len))
|
||||||
return 1; /* lost pixels is set */
|
return 1; /* lost pixels is set */
|
||||||
*sent_ptr += len;
|
|
||||||
urb = udl_get_urb(dev);
|
urb = udl_get_urb(dev);
|
||||||
if (!urb)
|
if (!urb)
|
||||||
return 1; /* lost_pixels is set */
|
return 1; /* lost_pixels is set */
|
||||||
|
Reference in New Issue
Block a user