Re: [PATCH/RFT] arm64: dts: renesas: r8a77995-draak: Add MSIOF ch2 pins support
Hi Simon, On Thu, Sep 6, 2018 at 11:18 AM Simon Horman wrote: > On Wed, Sep 05, 2018 at 05:22:07AM +0900, Yoshihiro Kaneko wrote: > > From: Hiromitsu Yamasaki > > > > This patch adds support for MSIOF ch2 pinctrl to use the LVDS Control > > Connector (CN41) for the Draak board on the R8A77995 SoC. > > > > Signed-off-by: Hiromitsu Yamasaki > > Signed-off-by: Takeshi Kihara > > [ykaneko0...@gmail.com: deleted the unused reference to 'msiof_ref_clk'] > > Signed-off-by: Yoshihiro Kaneko > > Hi Kaneko-san, > > sorry about this but I think we should drop this patch as > it is my understanding that msiof2 is not used on the Draak board. The connector is described in the Hardware Manual as "LVDS CONT connector" in the LVDS chaper, but it is not clear to me what exactly is supposed to be connected to it, and how standard it is. Laurent: do you know more? Note that Ebisu has the same connector, but the signals are shared with SW4, making them mutually exclusive. The Ebisu Hardware Manual does mention the connector, but not in the Display Output (LVDS) chapter. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@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
Re: [PATCH/RFT] arm64: dts: renesas: r8a77995-draak: Add MSIOF ch2 pins support
On Wed, Sep 05, 2018 at 05:22:07AM +0900, Yoshihiro Kaneko wrote: > From: Hiromitsu Yamasaki > > This patch adds support for MSIOF ch2 pinctrl to use the LVDS Control > Connector (CN41) for the Draak board on the R8A77995 SoC. > > Signed-off-by: Hiromitsu Yamasaki > Signed-off-by: Takeshi Kihara > [ykaneko0...@gmail.com: deleted the unused reference to 'msiof_ref_clk'] > Signed-off-by: Yoshihiro Kaneko Hi Kaneko-san, sorry about this but I think we should drop this patch as it is my understanding that msiof2 is not used on the Draak board. > --- > > This patch is based on the devel branch of Simon Horman's renesas tree. > > arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 13 + > 1 file changed, 13 insertions(+) > > diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > index a8e8f26..e60b1e4 100644 > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > @@ -135,6 +135,12 @@ > function = "i2c1"; > }; > > + msiof2_pins: spi2 { > + groups = "msiof2_clk", "msiof2_sync_b", > + "msiof2_rxd", "msiof2_txd"; > + function = "msiof2"; > + }; > + > pwm0_pins: pwm0 { > groups = "pwm0_c"; > function = "pwm0"; > @@ -371,3 +377,10 @@ > }; > }; > }; > + > + { > + pinctrl-0 = <_pins>; > + pinctrl-names = "default"; > + /* In case of using this node, please enable this property */ > + /* status = "okay"; */ > +}; > -- > 1.9.1 >
Re: [PATCH/RFT] arm64: dts: renesas: r8a77995-draak: Add MSIOF ch2 pins support
Hi Kaneko-san, On Tue, Sep 4, 2018 at 10:22 PM Yoshihiro Kaneko wrote: > From: Hiromitsu Yamasaki > > This patch adds support for MSIOF ch2 pinctrl to use the LVDS Control > Connector (CN41) for the Draak board on the R8A77995 SoC. > > Signed-off-by: Hiromitsu Yamasaki > Signed-off-by: Takeshi Kihara > [ykaneko0...@gmail.com: deleted the unused reference to 'msiof_ref_clk'] > Signed-off-by: Yoshihiro Kaneko Reviewed-by: Geert Uytterhoeven Minor nit below. > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > @@ -371,3 +377,10 @@ > }; > }; > }; > + > + { Please insert following alphabetical order. > + pinctrl-0 = <_pins>; > + pinctrl-names = "default"; > + /* In case of using this node, please enable this property */ I think this deserves some explanation about CN41. > + /* status = "okay"; */ > +}; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@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
[PATCH/RFT] arm64: dts: renesas: r8a77995-draak: Add MSIOF ch2 pins support
From: Hiromitsu Yamasaki This patch adds support for MSIOF ch2 pinctrl to use the LVDS Control Connector (CN41) for the Draak board on the R8A77995 SoC. Signed-off-by: Hiromitsu Yamasaki Signed-off-by: Takeshi Kihara [ykaneko0...@gmail.com: deleted the unused reference to 'msiof_ref_clk'] Signed-off-by: Yoshihiro Kaneko --- This patch is based on the devel branch of Simon Horman's renesas tree. arch/arm64/boot/dts/renesas/r8a77995-draak.dts | 13 + 1 file changed, 13 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts index a8e8f26..e60b1e4 100644 --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts @@ -135,6 +135,12 @@ function = "i2c1"; }; + msiof2_pins: spi2 { + groups = "msiof2_clk", "msiof2_sync_b", +"msiof2_rxd", "msiof2_txd"; + function = "msiof2"; + }; + pwm0_pins: pwm0 { groups = "pwm0_c"; function = "pwm0"; @@ -371,3 +377,10 @@ }; }; }; + + { + pinctrl-0 = <_pins>; + pinctrl-names = "default"; + /* In case of using this node, please enable this property */ + /* status = "okay"; */ +}; -- 1.9.1