* [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 @ 2014-06-10 2:34 Nobuhiro Iwamatsu 2014-06-10 4:14 ` Magnus Damm 0 siblings, 1 reply; 10+ messages in thread From: Nobuhiro Iwamatsu @ 2014-06-10 2:34 UTC (permalink / raw) To: linux-arm-kernel Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we want to use, we will need to disable the GPIO key and remodeling of the board. Therefore status of hscif1 does not set the "okay". Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> --- V2: Remove hscif1_clk from renesas,groups. This pin does not use. arch/arm/boot/dts/r8a7791-koelsch.dts | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts b/arch/arm/boot/dts/r8a7791-koelsch.dts index b2e6616..0992673 100644 --- a/arch/arm/boot/dts/r8a7791-koelsch.dts +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts @@ -22,6 +22,7 @@ aliases { serial6 = &scif0; serial7 = &scif1; + serial16 = &hscif1; }; chosen { @@ -258,6 +259,11 @@ renesas,function = "scif1"; }; + hscif1_pins: serial2 { + renesas,groups = "hscif1_data", "hscif1_ctrl"; + renesas,function = "hscif1"; + }; + ether_pins: ether { renesas,groups = "eth_link", "eth_mdio", "eth_rmii"; renesas,function = "eth"; @@ -334,6 +340,11 @@ status = "okay"; }; +&hscif1 { + pinctrl-0 = <&hscif1_pins>; + pinctrl-names = "default"; +}; + &sdhi0 { pinctrl-0 = <&sdhi0_pins>; pinctrl-names = "default"; -- 2.0.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-10 2:34 [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 Nobuhiro Iwamatsu @ 2014-06-10 4:14 ` Magnus Damm 2014-06-10 7:00 ` Simon Horman 0 siblings, 1 reply; 10+ messages in thread From: Magnus Damm @ 2014-06-10 4:14 UTC (permalink / raw) To: linux-arm-kernel Hi Iwamatsu-san, On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> wrote: > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we want to > use, we will need to disable the GPIO key and remodeling of the board. > Therefore status of hscif1 does not set the "okay". > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > --- > V2: Remove hscif1_clk from renesas,groups. This pin does not use. Thanks for updating the patch. This looks fine to me. Acked-by: Magnus Damm <damm+renesas@opensource.se> Cheers, / magnus ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-10 4:14 ` Magnus Damm @ 2014-06-10 7:00 ` Simon Horman 2014-06-13 7:01 ` Laurent Pinchart 0 siblings, 1 reply; 10+ messages in thread From: Simon Horman @ 2014-06-10 7:00 UTC (permalink / raw) To: linux-arm-kernel On Tue, Jun 10, 2014 at 01:14:27PM +0900, Magnus Damm wrote: > Hi Iwamatsu-san, > > On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu > <nobuhiro.iwamatsu.yj@renesas.com> wrote: > > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we want to > > use, we will need to disable the GPIO key and remodeling of the board. > > Therefore status of hscif1 does not set the "okay". > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > --- > > V2: Remove hscif1_clk from renesas,groups. This pin does not use. > > Thanks for updating the patch. This looks fine to me. > > Acked-by: Magnus Damm <damm+renesas@opensource.se> Thanks, I will queue this up. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-10 7:00 ` Simon Horman @ 2014-06-13 7:01 ` Laurent Pinchart 2014-06-13 7:43 ` Simon Horman 0 siblings, 1 reply; 10+ messages in thread From: Laurent Pinchart @ 2014-06-13 7:01 UTC (permalink / raw) To: linux-arm-kernel Hi Simon, On Tuesday 10 June 2014 16:00:48 Simon Horman wrote: > On Tue, Jun 10, 2014 at 01:14:27PM +0900, Magnus Damm wrote: > > On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu wrote: > > > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we want > > > to use, we will need to disable the GPIO key and remodeling of the > > > board. Therefore status of hscif1 does not set the "okay". > > > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > --- > > > > > > V2: Remove hscif1_clk from renesas,groups. This pin does not use. > > > > Thanks for updating the patch. This looks fine to me. > > > > Acked-by: Magnus Damm <damm+renesas@opensource.se> > > Thanks, I will queue this up. Sorry for the late reply, but I have some doubts about this. The HRX0 and HTX0 signals are connected to push buttons and used as GPIOs. Beside, they're not routed to serial port transceiver. If we want to use HSCIF0 on those pins on Koelsch and extension board must be connected to the EXIO C connector, and two resistors and two capacitors should be unsoldered. I thus don't think the Koeslch DT should contain pinmux data for the HSCIF0 device. If an extension board is used and HSCIF0 needed, then the DT file for the extension board should define pinmuxing for the serial port. -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-13 7:01 ` Laurent Pinchart @ 2014-06-13 7:43 ` Simon Horman 2014-06-13 10:49 ` Laurent Pinchart 0 siblings, 1 reply; 10+ messages in thread From: Simon Horman @ 2014-06-13 7:43 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jun 13, 2014 at 09:01:27AM +0200, Laurent Pinchart wrote: > Hi Simon, > > On Tuesday 10 June 2014 16:00:48 Simon Horman wrote: > > On Tue, Jun 10, 2014 at 01:14:27PM +0900, Magnus Damm wrote: > > > On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu wrote: > > > > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we want > > > > to use, we will need to disable the GPIO key and remodeling of the > > > > board. Therefore status of hscif1 does not set the "okay". > > > > > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > > --- > > > > > > > > V2: Remove hscif1_clk from renesas,groups. This pin does not use. > > > > > > Thanks for updating the patch. This looks fine to me. > > > > > > Acked-by: Magnus Damm <damm+renesas@opensource.se> > > > > Thanks, I will queue this up. > > Sorry for the late reply, but I have some doubts about this. The HRX0 and > HTX0 signals are connected to push buttons and used as GPIOs. Beside, > they're not routed to serial port transceiver. If we want to use HSCIF0 > on those pins on Koelsch and extension board must be connected to the > EXIO C connector, and two resistors and two capacitors should be > unsoldered. I thus don't think the Koeslch DT should contain pinmux data > for the HSCIF0 device. If an extension board is used and HSCIF0 needed, > then the DT file for the extension board should define pinmuxing for the > serial port. If you would like me to revert this patch I am happy to consider doing so. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-13 7:43 ` Simon Horman @ 2014-06-13 10:49 ` Laurent Pinchart 2014-06-13 11:16 ` Geert Uytterhoeven 0 siblings, 1 reply; 10+ messages in thread From: Laurent Pinchart @ 2014-06-13 10:49 UTC (permalink / raw) To: linux-arm-kernel Hi Simon, On Friday 13 June 2014 16:43:07 Simon Horman wrote: > On Fri, Jun 13, 2014 at 09:01:27AM +0200, Laurent Pinchart wrote: > > On Tuesday 10 June 2014 16:00:48 Simon Horman wrote: > > > On Tue, Jun 10, 2014 at 01:14:27PM +0900, Magnus Damm wrote: > > > > On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu wrote: > > > > > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we > > > > > want > > > > > to use, we will need to disable the GPIO key and remodeling of the > > > > > board. Therefore status of hscif1 does not set the "okay". > > > > > > > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > > > > > --- > > > > > > > > > > V2: Remove hscif1_clk from renesas,groups. This pin does not use. > > > > > > > > Thanks for updating the patch. This looks fine to me. > > > > > > > > Acked-by: Magnus Damm <damm+renesas@opensource.se> > > > > > > Thanks, I will queue this up. > > > > Sorry for the late reply, but I have some doubts about this. The HRX0 and > > HTX0 signals are connected to push buttons and used as GPIOs. Beside, > > they're not routed to serial port transceiver. If we want to use HSCIF0 > > on those pins on Koelsch and extension board must be connected to the > > EXIO C connector, and two resistors and two capacitors should be > > unsoldered. I thus don't think the Koeslch DT should contain pinmux data > > for the HSCIF0 device. If an extension board is used and HSCIF0 needed, > > then the DT file for the extension board should define pinmuxing for the > > serial port. > > If you would like me to revert this patch I am happy to consider doing so. That's one option, but I'd like to get feedback on my point of view before reverting it. Who knows, I might be wrong ;-) -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-13 10:49 ` Laurent Pinchart @ 2014-06-13 11:16 ` Geert Uytterhoeven 2014-06-13 13:24 ` Laurent Pinchart 2014-06-16 0:09 ` Simon Horman 0 siblings, 2 replies; 10+ messages in thread From: Geert Uytterhoeven @ 2014-06-13 11:16 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jun 13, 2014 at 12:49 PM, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > On Friday 13 June 2014 16:43:07 Simon Horman wrote: >> On Fri, Jun 13, 2014 at 09:01:27AM +0200, Laurent Pinchart wrote: >> > On Tuesday 10 June 2014 16:00:48 Simon Horman wrote: >> > > On Tue, Jun 10, 2014 at 01:14:27PM +0900, Magnus Damm wrote: >> > > > On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu wrote: >> > > > > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we >> > > > > want >> > > > > to use, we will need to disable the GPIO key and remodeling of the >> > > > > board. Therefore status of hscif1 does not set the "okay". >> > > > > >> > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> >> > > > > --- >> > > > > >> > > > > V2: Remove hscif1_clk from renesas,groups. This pin does not use. >> > > > >> > > > Thanks for updating the patch. This looks fine to me. >> > > > >> > > > Acked-by: Magnus Damm <damm+renesas@opensource.se> >> > > >> > > Thanks, I will queue this up. >> > >> > Sorry for the late reply, but I have some doubts about this. The HRX0 and >> > HTX0 signals are connected to push buttons and used as GPIOs. Beside, >> > they're not routed to serial port transceiver. If we want to use HSCIF0 >> > on those pins on Koelsch and extension board must be connected to the >> > EXIO C connector, and two resistors and two capacitors should be >> > unsoldered. I thus don't think the Koeslch DT should contain pinmux data >> > for the HSCIF0 device. If an extension board is used and HSCIF0 needed, >> > then the DT file for the extension board should define pinmuxing for the >> > serial port. You mean HRX1, HTX1, and HSCIF1? HTX1 and HRX1 can be connected to the DEBUG1 transceiver by stuffing zero-Ohm resisters to R834 and R835. About your comment about the resistors and capacitors, which applies to both HSCIF0 and HSCIF1: probably the serial port will still work if these are present. >> If you would like me to revert this patch I am happy to consider doing so. > > That's one option, but I'd like to get feedback on my point of view before > reverting it. Who knows, I might be wrong ;-) Nevertheless, I agree with Laurent. HSCIF1 (and HSCIF0) cannot be used on the Koelsch board as-is, hence they should not be present in the plain Koelsch DTS. They should be put in an overlay DTS instead. 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-13 11:16 ` Geert Uytterhoeven @ 2014-06-13 13:24 ` Laurent Pinchart 2014-06-13 14:21 ` Geert Uytterhoeven 2014-06-16 0:09 ` Simon Horman 1 sibling, 1 reply; 10+ messages in thread From: Laurent Pinchart @ 2014-06-13 13:24 UTC (permalink / raw) To: linux-arm-kernel Hi Geert, On Friday 13 June 2014 13:16:34 Geert Uytterhoeven wrote: > On Fri, Jun 13, 2014 at 12:49 PM, Laurent Pinchart wrote: > > On Friday 13 June 2014 16:43:07 Simon Horman wrote: > >> On Fri, Jun 13, 2014 at 09:01:27AM +0200, Laurent Pinchart wrote: > >> > On Tuesday 10 June 2014 16:00:48 Simon Horman wrote: > >> > > On Tue, Jun 10, 2014 at 01:14:27PM +0900, Magnus Damm wrote: > >> > > > On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu wrote: > >> > > > > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if > >> > > > > we want to use, we will need to disable the GPIO key and > >> > > > > remodeling of the board. Therefore status of hscif1 does not set > >> > > > > the "okay". > >> > > > > > >> > > > > Signed-off-by: Nobuhiro Iwamatsu > >> > > > > <nobuhiro.iwamatsu.yj@renesas.com> > >> > > > > --- > >> > > > > > >> > > > > V2: Remove hscif1_clk from renesas,groups. This pin does not > >> > > > > use. > >> > > > > >> > > > Thanks for updating the patch. This looks fine to me. > >> > > > > >> > > > Acked-by: Magnus Damm <damm+renesas@opensource.se> > >> > > > >> > > Thanks, I will queue this up. > >> > > >> > Sorry for the late reply, but I have some doubts about this. The HRX0 > >> > and HTX0 signals are connected to push buttons and used as GPIOs. > >> > Beside, they're not routed to serial port transceiver. If we want to > >> > use HSCIF0 on those pins on Koelsch and extension board must be > >> > connected to the EXIO C connector, and two resistors and two capacitors > >> > should be unsoldered. I thus don't think the Koeslch DT should contain > >> > pinmux data for the HSCIF0 device. If an extension board is used and > >> > HSCIF0 needed, then the DT file for the extension board should define > >> > pinmuxing for the serial port. > > You mean HRX1, HTX1, and HSCIF1? *sigh*... :-) > HTX1 and HRX1 can be connected to the DEBUG1 transceiver by stuffing > zero-Ohm resisters to R834 and R835. > > About your comment about the resistors and capacitors, which applies to > both HSCIF0 and HSCIF1: probably the serial port will still work if > these are present. With a 4.7µF capacitive load on the transmission line, assuming a drive strength of 40mA (which is pretty high), switching between 0V and 3.3V takes 388µs, corresponding to 2579 Hz. You'll have to reduce the baud rate :-) > >> If you would like me to revert this patch I am happy to consider doing > >> so. > > > > That's one option, but I'd like to get feedback on my point of view before > > reverting it. Who knows, I might be wrong ;-) > > Nevertheless, I agree with Laurent. > > HSCIF1 (and HSCIF0) cannot be used on the Koelsch board as-is, hence they > should not be present in the plain Koelsch DTS. They should be put in > an overlay DTS instead. -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-13 13:24 ` Laurent Pinchart @ 2014-06-13 14:21 ` Geert Uytterhoeven 0 siblings, 0 replies; 10+ messages in thread From: Geert Uytterhoeven @ 2014-06-13 14:21 UTC (permalink / raw) To: linux-arm-kernel Hi Laurent, On Fri, Jun 13, 2014 at 3:24 PM, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: >> About your comment about the resistors and capacitors, which applies to >> both HSCIF0 and HSCIF1: probably the serial port will still work if >> these are present. > > With a 4.7µF capacitive load on the transmission line, assuming a drive > strength of 40mA (which is pretty high), switching between 0V and 3.3V takes > 388µs, corresponding to 2579 Hz. You'll have to reduce the baud rate :-) Right, 4.7µF is a bit high. So at least the capacitor has to go for today's typical serial speeds. 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 2014-06-13 11:16 ` Geert Uytterhoeven 2014-06-13 13:24 ` Laurent Pinchart @ 2014-06-16 0:09 ` Simon Horman 1 sibling, 0 replies; 10+ messages in thread From: Simon Horman @ 2014-06-16 0:09 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jun 13, 2014 at 01:16:34PM +0200, Geert Uytterhoeven wrote: > On Fri, Jun 13, 2014 at 12:49 PM, Laurent Pinchart > <laurent.pinchart@ideasonboard.com> wrote: > > On Friday 13 June 2014 16:43:07 Simon Horman wrote: > >> On Fri, Jun 13, 2014 at 09:01:27AM +0200, Laurent Pinchart wrote: > >> > On Tuesday 10 June 2014 16:00:48 Simon Horman wrote: > >> > > On Tue, Jun 10, 2014 at 01:14:27PM +0900, Magnus Damm wrote: > >> > > > On Tue, Jun 10, 2014 at 11:34 AM, Nobuhiro Iwamatsu wrote: > >> > > > > Koelsch can use HSCIF1 insetead of SCIF1 as serial port. But if we > >> > > > > want > >> > > > > to use, we will need to disable the GPIO key and remodeling of the > >> > > > > board. Therefore status of hscif1 does not set the "okay". > >> > > > > > >> > > > > Signed-off-by: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com> > >> > > > > --- > >> > > > > > >> > > > > V2: Remove hscif1_clk from renesas,groups. This pin does not use. > >> > > > > >> > > > Thanks for updating the patch. This looks fine to me. > >> > > > > >> > > > Acked-by: Magnus Damm <damm+renesas@opensource.se> > >> > > > >> > > Thanks, I will queue this up. > >> > > >> > Sorry for the late reply, but I have some doubts about this. The HRX0 and > >> > HTX0 signals are connected to push buttons and used as GPIOs. Beside, > >> > they're not routed to serial port transceiver. If we want to use HSCIF0 > >> > on those pins on Koelsch and extension board must be connected to the > >> > EXIO C connector, and two resistors and two capacitors should be > >> > unsoldered. I thus don't think the Koeslch DT should contain pinmux data > >> > for the HSCIF0 device. If an extension board is used and HSCIF0 needed, > >> > then the DT file for the extension board should define pinmuxing for the > >> > serial port. > > You mean HRX1, HTX1, and HSCIF1? > > HTX1 and HRX1 can be connected to the DEBUG1 transceiver by stuffing > zero-Ohm resisters to R834 and R835. > > About your comment about the resistors and capacitors, which applies to > both HSCIF0 and HSCIF1: probably the serial port will still work if > these are present. > > >> If you would like me to revert this patch I am happy to consider doing so. > > > > That's one option, but I'd like to get feedback on my point of view before > > reverting it. Who knows, I might be wrong ;-) > > Nevertheless, I agree with Laurent. > > HSCIF1 (and HSCIF0) cannot be used on the Koelsch board as-is, hence they > should not be present in the plain Koelsch DTS. They should be put in > an overlay > DTS instead. Thanks, I have reverted this patch and will probably subsequently drop it and the revert when I rebase my branches on top of v3.16-rc1 (due to ARM-SoC process requirements). Lets revisit this if necessary. ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-06-16 0:09 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-06-10 2:34 [PATCH v2] ARM: shmobile: koelsch: Add support HSCIF1 Nobuhiro Iwamatsu 2014-06-10 4:14 ` Magnus Damm 2014-06-10 7:00 ` Simon Horman 2014-06-13 7:01 ` Laurent Pinchart 2014-06-13 7:43 ` Simon Horman 2014-06-13 10:49 ` Laurent Pinchart 2014-06-13 11:16 ` Geert Uytterhoeven 2014-06-13 13:24 ` Laurent Pinchart 2014-06-13 14:21 ` Geert Uytterhoeven 2014-06-16 0:09 ` Simon Horman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).