2017-04-05 07:33:11

by Neil Armstrong

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

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 <[email protected]>
---
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

2017-04-05 08:14:34

by Laurent Pinchart

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

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 <[email protected]>

Acked-by: Laurent Pinchart <[email protected]>

> ---
> 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

2017-04-06 08:59:34

by Neil Armstrong

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

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 <[email protected]>
>
> Acked-by: Laurent Pinchart <[email protected]>
>
>> ---
>> 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