mirror of
https://github.com/torvalds/linux.git
synced 2026-09-23 22:14:03 +02:00
drm/solomon: remove unneeded variables in blit functions
Remove unneeded 'ret' variables in ssd130x_fb_blit_rect(),
ssd132x_fb_blit_rect(), and ssd133x_fb_blit_rect() functions.
These functions initialize ret to 0 and return it unchanged,
so return 0 directly instead.
Fixes: 2258f03989 ("drm/solomon: Move calls to drm_gem_fb_end_cpu*()")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202606301409.I0ctsf41-lkp@intel.com/
Signed-off-by: Iker Pedrosa <ikerpedrosam@gmail.com>
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
Link: https://patch.msgid.link/20260709-fix-ssd130x-v1-1-1272cb3dc85e@gmail.com
Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
This commit is contained in:
parent
ec6444a00c
commit
e5320be8a5
|
|
@ -897,7 +897,6 @@ static int ssd130x_fb_blit_rect(struct drm_framebuffer *fb,
|
|||
struct ssd130x_device *ssd130x = drm_to_ssd130x(fb->dev);
|
||||
struct iosys_map dst;
|
||||
unsigned int dst_pitch;
|
||||
int ret = 0;
|
||||
|
||||
/* Align y to display page boundaries */
|
||||
rect->y1 = round_down(rect->y1, SSD130X_PAGE_HEIGHT);
|
||||
|
|
@ -910,7 +909,7 @@ static int ssd130x_fb_blit_rect(struct drm_framebuffer *fb,
|
|||
|
||||
ssd130x_update_rect(ssd130x, rect, buf, data_array);
|
||||
|
||||
return ret;
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int ssd132x_fb_blit_rect(struct drm_framebuffer *fb,
|
||||
|
|
@ -922,7 +921,6 @@ static int ssd132x_fb_blit_rect(struct drm_framebuffer *fb,
|
|||
struct ssd130x_device *ssd130x = drm_to_ssd130x(fb->dev);
|
||||
unsigned int dst_pitch;
|
||||
struct iosys_map dst;
|
||||
int ret = 0;
|
||||
|
||||
/* Align x to display segment boundaries */
|
||||
rect->x1 = round_down(rect->x1, SSD132X_SEGMENT_WIDTH);
|
||||
|
|
@ -936,7 +934,7 @@ static int ssd132x_fb_blit_rect(struct drm_framebuffer *fb,
|
|||
|
||||
ssd132x_update_rect(ssd130x, rect, buf, data_array);
|
||||
|
||||
return ret;
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int ssd133x_fb_blit_rect(struct drm_framebuffer *fb,
|
||||
|
|
@ -948,7 +946,6 @@ static int ssd133x_fb_blit_rect(struct drm_framebuffer *fb,
|
|||
const struct drm_format_info *fi = drm_format_info(DRM_FORMAT_RGB332);
|
||||
unsigned int dst_pitch;
|
||||
struct iosys_map dst;
|
||||
int ret = 0;
|
||||
|
||||
if (!fi)
|
||||
return -EINVAL;
|
||||
|
|
@ -960,7 +957,7 @@ static int ssd133x_fb_blit_rect(struct drm_framebuffer *fb,
|
|||
|
||||
ssd133x_update_rect(ssd130x, rect, data_array, dst_pitch);
|
||||
|
||||
return ret;
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int ssd130x_primary_plane_atomic_check(struct drm_plane *plane,
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user