From mboxrd@z Thu Jan 1 00:00:00 1970 From: Geert Uytterhoeven Subject: Re: [PATCH 1/2] pinctrl: sh-pfc: r8a7791: add missing HSCIF1 pinmux data Date: Thu, 30 Mar 2017 10:58:50 +0200 Message-ID: References: <20170329183653.982359372@cogentembedded.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <20170329183653.982359372@cogentembedded.com> Sender: linux-renesas-soc-owner@vger.kernel.org To: Sergei Shtylyov Cc: Laurent Pinchart , Geert Uytterhoeven , Linus Walleij , Linux-Renesas , "linux-gpio@vger.kernel.org" , stable List-Id: linux-gpio@vger.kernel.org On Wed, Mar 29, 2017 at 8:36 PM, Sergei Shtylyov wrote: > The R8A7791 PFC driver was apparently based on the preliminary revisions > of the user's manual, which omitted the HSCIF1 group E signals in the > IPSR4 register description. This would cause HSCIF1's probe to fail with > the messages like below: > > sh-pfc e6060000.pfc: cannot locate data/mark enum_id for mark 1989 > sh-sci e62c8000.serial: Error applying setting, reverse things back > sh-sci: probe of e62c8000.serial failed with error -22 > > Add the neceassary PINMUX_IPSR_MSEL() invocations for the HSCK1_E, > HCTS1#_E, and HRTS1#_E signals... > > Fixes: 508845196238 ("pinctrl: sh-pfc: r8a7791 PFC support") > Signed-off-by: Sergei Shtylyov > CC: stable@vger.kernel.org Reviewed-by: Geert Uytterhoeven Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds