Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933427AbdDFI7e (ORCPT ); Thu, 6 Apr 2017 04:59:34 -0400 Received: from mail-wr0-f178.google.com ([209.85.128.178]:35955 "EHLO mail-wr0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932517AbdDFI71 (ORCPT ); Thu, 6 Apr 2017 04:59:27 -0400 Subject: Re: [PATCH] drm: bridge: dw-hdmi: remove unused hdmi_bus_fmt_is_yuv420 To: Laurent Pinchart References: <1491377579-9353-1-git-send-email-narmstrong@baylibre.com> <1867549.4Y9HFCsCxn@avalon> Cc: dri-devel@lists.freedesktop.org, laurent.pinchart+renesas@ideasonboard.com, architt@codeaurora.org, Jose.Abreu@synopsys.com, linux-kernel@vger.kernel.org From: Neil Armstrong Organization: Baylibre Message-ID: Date: Thu, 6 Apr 2017 10:59:21 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <1867549.4Y9HFCsCxn@avalon> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1349 Lines: 48 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 > > Acked-by: Laurent Pinchart > >> --- >> 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