From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Fri, 14 Jun 2013 16:51:38 +0000 Subject: Re: [PATCH v3 5/5] ARM: shmobile: lager: map HSCIF0/1 to unused pins Message-Id: <2744880.yL7dTA9vlz@avalon> List-Id: References: <1370015825-29383-6-git-send-email-ulrich.hecht@gmail.com> In-Reply-To: <1370015825-29383-6-git-send-email-ulrich.hecht@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-sh@vger.kernel.org Hi Ulrich ans Simon, On Wednesday 12 June 2013 23:08:47 Simon Horman wrote: > On Fri, May 31, 2013 at 05:57:05PM +0200, Ulrich Hecht wrote: > > These pins are not actually connected on the board, but at least they > > don't conflict with anything else. That's actually not true, the hscif0_data pins are used by the DU RGB output. If the HSCIF0 and HSCIF1 are not wired on the board I don't think they should be added to the pinctrl maps. Simon, should we revert this patch now or later when I'll submit DU patches for Lager ? > > Signed-off-by: Ulrich Hecht > > Thanks, I have queued this up in the pinmux branch. > > > --- > > > > arch/arm/mach-shmobile/board-lager.c | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/arch/arm/mach-shmobile/board-lager.c > > b/arch/arm/mach-shmobile/board-lager.c index 1e99b17..d8bd237 100644 > > --- a/arch/arm/mach-shmobile/board-lager.c > > +++ b/arch/arm/mach-shmobile/board-lager.c > > @@ -77,6 +77,10 @@ static const struct pinctrl_map lager_pinctrl_map[] = { > > > > /* SCIF1 (CN20: DEBUG SERIAL1) */ > > PIN_MAP_MUX_GROUP_DEFAULT("sh-sci.7", "pfc-r8a7790", > > > > "scif1_data", "scif1"), > > > > + PIN_MAP_MUX_GROUP_DEFAULT("sh-sci.8", "pfc-r8a7790", > > + "hscif0_data", "hscif0"), > > + PIN_MAP_MUX_GROUP_DEFAULT("sh-sci.9", "pfc-r8a7790", > > + "hscif1_data_b", "hscif1"), > > > > }; > > > > static void __init lager_add_standard_devices(void) -- Regards, Laurent Pinchart