mbox series

[0/2] drm/vkms: A couple of trivial cleanups

Message ID 20230406110235.3092055-1-javierm@redhat.com (mailing list archive)
Headers show
Series drm/vkms: A couple of trivial cleanups | expand

Message

Javier Martinez Canillas April 6, 2023, 11:02 a.m. UTC
Hello,

This series contains two trivial cleanups for the vkms driver.

Patch #1 just gets rid of a wrapper helper that wasn't really adding that
much value and patch #2 drops the <drm/drm_simple_kms_helper.h> header
that was only used to call the drm_simple_encoder_init() function helper.

Best regards,
Javier


Javier Martinez Canillas (2):
  drm/vkms: Drop vkms_connector_destroy() wrapper
  drm/vkms: Remove <drm/drm_simple_kms_helper.h> include

 drivers/gpu/drm/vkms/vkms_output.c | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)


base-commit: 77d08a2de6a43521f5a02848f11185b6f46af21c

Comments

Thomas Zimmermann April 6, 2023, 11:15 a.m. UTC | #1
Hi

Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>

for the series.

Best regards
Thomas

Am 06.04.23 um 13:02 schrieb Javier Martinez Canillas:
> Hello,
> 
> This series contains two trivial cleanups for the vkms driver.
> 
> Patch #1 just gets rid of a wrapper helper that wasn't really adding that
> much value and patch #2 drops the <drm/drm_simple_kms_helper.h> header
> that was only used to call the drm_simple_encoder_init() function helper.
> 
> Best regards,
> Javier
> 
> 
> Javier Martinez Canillas (2):
>    drm/vkms: Drop vkms_connector_destroy() wrapper
>    drm/vkms: Remove <drm/drm_simple_kms_helper.h> include
> 
>   drivers/gpu/drm/vkms/vkms_output.c | 15 +++++++--------
>   1 file changed, 7 insertions(+), 8 deletions(-)
> 
> 
> base-commit: 77d08a2de6a43521f5a02848f11185b6f46af21c
Maíra Canal April 6, 2023, 11:31 a.m. UTC | #2
Hi,

Reviewed-by: Maíra Canal <mcanal@igalia.com>

for the series.

Thanks for helping to cleanup vkms!

Best Regards,
- Maíra Canal

On 4/6/23 08:02, Javier Martinez Canillas wrote:
> Hello,
> 
> This series contains two trivial cleanups for the vkms driver.
> 
> Patch #1 just gets rid of a wrapper helper that wasn't really adding that
> much value and patch #2 drops the <drm/drm_simple_kms_helper.h> header
> that was only used to call the drm_simple_encoder_init() function helper.
> 
> Best regards,
> Javier
> 
> 
> Javier Martinez Canillas (2):
>    drm/vkms: Drop vkms_connector_destroy() wrapper
>    drm/vkms: Remove <drm/drm_simple_kms_helper.h> include
> 
>   drivers/gpu/drm/vkms/vkms_output.c | 15 +++++++--------
>   1 file changed, 7 insertions(+), 8 deletions(-)
> 
> 
> base-commit: 77d08a2de6a43521f5a02848f11185b6f46af21c
Javier Martinez Canillas April 7, 2023, 11:22 a.m. UTC | #3
Javier Martinez Canillas <javierm@redhat.com> writes:

> Hello,
>
> This series contains two trivial cleanups for the vkms driver.
>
> Patch #1 just gets rid of a wrapper helper that wasn't really adding that
> much value and patch #2 drops the <drm/drm_simple_kms_helper.h> header
> that was only used to call the drm_simple_encoder_init() function helper.
>
> Best regards,
> Javier
>
>
> Javier Martinez Canillas (2):
>   drm/vkms: Drop vkms_connector_destroy() wrapper
>   drm/vkms: Remove <drm/drm_simple_kms_helper.h> include
>

Pushed to drm-misc (drm-misc-next). Thanks!