Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752597AbbLUWB5 (ORCPT ); Mon, 21 Dec 2015 17:01:57 -0500 Received: from mail-lb0-f179.google.com ([209.85.217.179]:35604 "EHLO mail-lb0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752525AbbLUWBj (ORCPT ); Mon, 21 Dec 2015 17:01:39 -0500 Subject: Re: [PATCH 03/04] pinctrl: sh-pfc: r8a7794: Add missing dot clock signals To: Magnus Damm , linux-sh@vger.kernel.org References: <20151117031832.24924.60685.sendpatchset@little-apple> <20151117031902.24924.4588.sendpatchset@little-apple> Cc: linus.walleij@linaro.org, linux-kernel@vger.kernel.org, koji.matsuoka.xm@renesas.com, linux-gpio@vger.kernel.org, horms@verge.net.au, geert@glider.be, laurent.pinchart@ideasonboard.com From: Sergei Shtylyov Organization: Cogent Embedded Message-ID: <567876BE.60100@cogentembedded.com> Date: Tue, 22 Dec 2015 01:01:34 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20151117031902.24924.4588.sendpatchset@little-apple> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2304 Lines: 70 Hello. On 11/17/2015 06:19 AM, Magnus Damm wrote: > From: Magnus Damm > > Add missing r8a7794 DU dot clock output signals and in > particular the DU1_DOTCLKOUT0 signal on GP4_25 which is > needed by DU1 on the r8a7794 ALT board. > > The groups "du0_clk_out" and "du1_clk_out" are replaced by > "du0_dotclkout0", "du0_dotclkout1", "du1_dotclkout0" and > "du1_dotclkout1". > > Signed-off-by: Magnus Damm > --- > > drivers/pinctrl/sh-pfc/pfc-r8a7794.c | 34 ++++++++++++++++++++++++++-------- > 1 file changed, 26 insertions(+), 8 deletions(-) > > --- 0014/drivers/pinctrl/sh-pfc/pfc-r8a7794.c > +++ work/drivers/pinctrl/sh-pfc/pfc-r8a7794.c 2015-11-16 22:08:28.230513000 +0900 [...] > @@ -2942,8 +2956,10 @@ static const struct sh_pfc_pin_group pin > SH_PFC_PIN_GROUP(du0_rgb888), > SH_PFC_PIN_GROUP(du1_rgb666), > SH_PFC_PIN_GROUP(du1_rgb888), > - SH_PFC_PIN_GROUP(du0_clk_out), > - SH_PFC_PIN_GROUP(du1_clk_out), > + SH_PFC_PIN_GROUP(du0_clk0_out), > + SH_PFC_PIN_GROUP(du0_clk1_out), > + SH_PFC_PIN_GROUP(du1_clk0_out), > + SH_PFC_PIN_GROUP(du1_clk1_out), Hm, this kinda contradicts your chngnelog, no? > SH_PFC_PIN_GROUP(du0_sync), > SH_PFC_PIN_GROUP(du1_sync), > SH_PFC_PIN_GROUP(du0_cde), > @@ -3134,8 +3150,10 @@ static const char * const du_groups[] = > "du0_rgb888", > "du1_rgb666", > "du1_rgb888", > - "du0_clk_out", > - "du1_clk_out", > + "du0_dotclkout0", > + "du0_dotclkout1", > + "du1_dotclkout0", > + "du1_dotclkout1", > "du0_sync", > "du1_sync", > "du0_cde", I'm getting the following on the SILK board with this patch: [drm] Initialized drm 1.1.0 20060810 sh-pfc e6060000.pin-controller: does not have pin group du1_dotclkout0 sh-pfc e6060000.pin-controller: invalid group du1_dotclkout0 in map table [drm] Supports vblank timestamp caching Rev 2 (21.10.2013). [drm] No driver support for vblank timestamp query. rcar-du feb00000.display: fb0: frame buffer device [drm] Initialized rcar-du 1.0.0 20130110 on minor 0 MBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/