Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1616656 > unrolled thread

[PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420

Started byNeil Armstrong <narmstrong@baylibre.com>
First post2017-04-05 09:40 +0200
Last post2017-04-06 11:00 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420 Neil Armstrong <narmstrong@baylibre.com> - 2017-04-05 09:40 +0200
    Re: [PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420 Laurent Pinchart <laurent.pinchart@ideasonboard.com> - 2017-04-05 10:20 +0200
      Re: [PATCH] drm: bridge: dw-hdmi: remove unused  hdmi_bus_fmt_is_yuv420 Neil Armstrong <narmstrong@baylibre.com> - 2017-04-06 11:00 +0200

#1616656 — [PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420

FromNeil Armstrong <narmstrong@baylibre.com>
Date2017-04-05 09:40 +0200
Subject[PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420
Message-ID<tsNHQ-85H-15@gated-at.bofh.it>
Remove usused yet hdmi_bus_fmt_is_yuv420 function.

Fixes: def23aa7e982 ("drm: bridge: dw-hdmi: Switch to V4L bus format and encodings")
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
 drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
index 84cc949..7dd6699 100644
--- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
+++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
@@ -605,20 +605,6 @@ static bool hdmi_bus_fmt_is_yuv422(unsigned int bus_format)
 	}
 }
 
-static bool hdmi_bus_fmt_is_yuv420(unsigned int bus_format)
-{
-	switch (bus_format) {
-	case MEDIA_BUS_FMT_UYYVYY8_0_5X24:
-	case MEDIA_BUS_FMT_UYYVYY10_0_5X30:
-	case MEDIA_BUS_FMT_UYYVYY12_0_5X36:
-	case MEDIA_BUS_FMT_UYYVYY16_0_5X48:
-		return true;
-
-	default:
-		return false;
-	}
-}
-
 static int hdmi_bus_fmt_color_depth(unsigned int bus_format)
 {
 	switch (bus_format) {
-- 
1.9.1

[toc] | [next] | [standalone]


#1616677

FromLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Date2017-04-05 10:20 +0200
Message-ID<tsOkx-8y-15@gated-at.bofh.it>
In reply to#1616656
Hi Neil,

Thank you for the patch.

On Wednesday 05 Apr 2017 09:32:59 Neil Armstrong wrote:
> Remove usused yet hdmi_bus_fmt_is_yuv420 function.
> 
> Fixes: def23aa7e982 ("drm: bridge: dw-hdmi: Switch to V4L bus format and
> encodings") Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
>  drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 14 --------------
>  1 file changed, 14 deletions(-)
> 
> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
> b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c index 84cc949..7dd6699 100644
> --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
> +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
> @@ -605,20 +605,6 @@ static bool hdmi_bus_fmt_is_yuv422(unsigned int
> bus_format) }
>  }
> 
> -static bool hdmi_bus_fmt_is_yuv420(unsigned int bus_format)
> -{
> -	switch (bus_format) {
> -	case MEDIA_BUS_FMT_UYYVYY8_0_5X24:
> -	case MEDIA_BUS_FMT_UYYVYY10_0_5X30:
> -	case MEDIA_BUS_FMT_UYYVYY12_0_5X36:
> -	case MEDIA_BUS_FMT_UYYVYY16_0_5X48:
> -		return true;
> -
> -	default:
> -		return false;
> -	}
> -}
> -
>  static int hdmi_bus_fmt_color_depth(unsigned int bus_format)
>  {
>  	switch (bus_format) {

-- 
Regards,

Laurent Pinchart

[toc] | [prev] | [next] | [standalone]


#1617658 — Re: [PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420

FromNeil Armstrong <narmstrong@baylibre.com>
Date2017-04-06 11:00 +0200
SubjectRe: [PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420
Message-ID<ttbqN-6ei-13@gated-at.bofh.it>
In reply to#1616677
On 04/05/2017 10:14 AM, Laurent Pinchart wrote:
> Hi Neil,
> 
> Thank you for the patch.
> 
> On Wednesday 05 Apr 2017 09:32:59 Neil Armstrong wrote:
>> Remove usused yet hdmi_bus_fmt_is_yuv420 function.
>>
>> Fixes: def23aa7e982 ("drm: bridge: dw-hdmi: Switch to V4L bus format and
>> encodings") Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> 
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> 
>> ---
>>  drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 14 --------------
>>  1 file changed, 14 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
>> b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c index 84cc949..7dd6699 100644
>> --- a/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
>> +++ b/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c
>> @@ -605,20 +605,6 @@ static bool hdmi_bus_fmt_is_yuv422(unsigned int
>> bus_format) }
>>  }
>>
>> -static bool hdmi_bus_fmt_is_yuv420(unsigned int bus_format)
>> -{
>> -	switch (bus_format) {
>> -	case MEDIA_BUS_FMT_UYYVYY8_0_5X24:
>> -	case MEDIA_BUS_FMT_UYYVYY10_0_5X30:
>> -	case MEDIA_BUS_FMT_UYYVYY12_0_5X36:
>> -	case MEDIA_BUS_FMT_UYYVYY16_0_5X48:
>> -		return true;
>> -
>> -	default:
>> -		return false;
>> -	}
>> -}
>> -
>>  static int hdmi_bus_fmt_color_depth(unsigned int bus_format)
>>  {
>>  	switch (bus_format) {
> 

Applied to drm-misc-next

Thanks,
Neil

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web