* [PATCH 1/2] arm64: dts: marvell: add CP110 uart peripherals @ 2018-01-30 8:01 Baruch Siach 2018-01-30 8:01 ` [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers Baruch Siach 0 siblings, 1 reply; 6+ messages in thread From: Baruch Siach @ 2018-01-30 8:01 UTC (permalink / raw) To: linux-arm-kernel The CP110 component has 4 uart peripherals. All of them use the same clock gate for slow peripherals that is shared with the i2c and spi peripherals. Signed-off-by: Baruch Siach <baruch@tkos.co.il> --- arch/arm64/boot/dts/marvell/armada-cp110.dtsi | 40 +++++++++++++++++++++++++++ 1 file changed, 40 insertions(+) diff --git a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi index a8af4136dbe7..a422cd981a0b 100644 --- a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi +++ b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi @@ -290,6 +290,46 @@ status = "disabled"; }; + CP110_LABEL(uart0): serial at 702000 { + compatible = "snps,dw-apb-uart"; + reg = <0x702000 0x100>; + reg-shift = <2>; + interrupts = <ICU_GRP_NSR 122 IRQ_TYPE_LEVEL_HIGH>; + reg-io-width = <1>; + clocks = <&CP110_LABEL(clk) 1 21>; + status = "disabled"; + }; + + CP110_LABEL(uart1): serial at 702100 { + compatible = "snps,dw-apb-uart"; + reg = <0x702100 0x100>; + reg-shift = <2>; + interrupts = <ICU_GRP_NSR 123 IRQ_TYPE_LEVEL_HIGH>; + reg-io-width = <1>; + clocks = <&CP110_LABEL(clk) 1 21>; + status = "disabled"; + }; + + CP110_LABEL(uart2): serial at 702200 { + compatible = "snps,dw-apb-uart"; + reg = <0x702200 0x100>; + reg-shift = <2>; + interrupts = <ICU_GRP_NSR 124 IRQ_TYPE_LEVEL_HIGH>; + reg-io-width = <1>; + clocks = <&CP110_LABEL(clk) 1 21>; + status = "disabled"; + }; + + CP110_LABEL(uart3): serial at 702300 { + compatible = "snps,dw-apb-uart"; + reg = <0x702300 0x100>; + reg-shift = <2>; + interrupts = <ICU_GRP_NSR 125 IRQ_TYPE_LEVEL_HIGH>; + reg-io-width = <1>; + clocks = <&CP110_LABEL(clk) 1 21>; + status = "disabled"; + }; + CP110_LABEL(nand): nand at 720000 { /* * Due to the limitation of the pins available -- 2.15.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers 2018-01-30 8:01 [PATCH 1/2] arm64: dts: marvell: add CP110 uart peripherals Baruch Siach @ 2018-01-30 8:01 ` Baruch Siach 2018-01-30 10:27 ` Thomas Petazzoni 2018-01-30 16:05 ` Russell King - ARM Linux 0 siblings, 2 replies; 6+ messages in thread From: Baruch Siach @ 2018-01-30 8:01 UTC (permalink / raw) To: linux-arm-kernel Add description of the J25 and J27 UART headers of the Macchiatobin. They use uart peripherals that the CP0 (J25) and CP1 (J27) provides. Signed-off-by: Baruch Siach <baruch@tkos.co.il> --- arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts index 626e9d0462c3..3ce15b740bb0 100644 --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts @@ -163,6 +163,12 @@ }; }; +&cp0_uart1 { + pinctrl-names = "default"; + pinctrl-0 = <&cp0_uart1_pins>; + status = "okay"; +}; + &cp0_mdio { pinctrl-names = "default"; pinctrl-0 = <&cp0_ge_mdio_pins>; @@ -195,6 +201,10 @@ marvell,pins = "mpp37", "mpp38"; marvell,function = "i2c0"; }; + cp0_uart1_pins: uart1-pins { + marvell,pins = "mpp40", "mpp41"; + marvell,function = "uart1"; + }; cp0_xhci_vbus_pins: xhci0-vbus-pins { marvell,pins = "mpp47"; marvell,function = "gpio"; @@ -290,6 +300,16 @@ marvell,pins = "mpp12", "mpp13", "mpp14", "mpp15", "mpp16"; marvell,function = "spi1"; }; + cp1_uart0_pins: uart0-pins { + marvell,pins = "mpp6", "mpp7"; + marvell,function = "uart0"; + }; +}; + +&cp1_uart0 { + pinctrl-names = "default"; + pinctrl-0 = <&cp1_uart0_pins>; + status = "okay"; }; &cp1_sata0 { -- 2.15.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers 2018-01-30 8:01 ` [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers Baruch Siach @ 2018-01-30 10:27 ` Thomas Petazzoni 2018-01-30 16:05 ` Russell King - ARM Linux 1 sibling, 0 replies; 6+ messages in thread From: Thomas Petazzoni @ 2018-01-30 10:27 UTC (permalink / raw) To: linux-arm-kernel Hello, On Tue, 30 Jan 2018 10:01:26 +0200, Baruch Siach wrote: > Add description of the J25 and J27 UART headers of the Macchiatobin. They use > uart peripherals that the CP0 (J25) and CP1 (J27) provides. > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> Thanks. One minor nit below. > --- > arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts > index 626e9d0462c3..3ce15b740bb0 100644 > --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts > +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts > @@ -163,6 +163,12 @@ > }; > }; > > +&cp0_uart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&cp0_uart1_pins>; > + status = "okay"; > +}; It'd be nice to indicate in the DT that this is accessible through J25. > +&cp1_uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&cp1_uart0_pins>; > + status = "okay"; > }; And this through J27. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers 2018-01-30 8:01 ` [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers Baruch Siach 2018-01-30 10:27 ` Thomas Petazzoni @ 2018-01-30 16:05 ` Russell King - ARM Linux 2018-01-30 16:47 ` Baruch Siach 1 sibling, 1 reply; 6+ messages in thread From: Russell King - ARM Linux @ 2018-01-30 16:05 UTC (permalink / raw) To: linux-arm-kernel On Tue, Jan 30, 2018 at 10:01:26AM +0200, Baruch Siach wrote: > Add description of the J25 and J27 UART headers of the Macchiatobin. They use > uart peripherals that the CP0 (J25) and CP1 (J27) provides. The headers are just fed with the GPIO-level signals, which means any industry standard usage would require additional RS232 level shifting hardware. So, they're certainly not "generic" serial ports. While these are labelled up as UARTs, there's no reason to dedicate them as such, they could equally be used to connect to a smart battery/ UPS system as a SMBus port, or even to control system LEDs / read the status of system buttons. Do you have an application for these headers? If not, it's probably best to leave them alone (until we have DT overlay support sorted to allow boot-loader configuration of hardware such as these - which is being worked on for TI and other boards.) -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 8.8Mbps down 630kbps up According to speedtest.net: 8.21Mbps down 510kbps up ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers 2018-01-30 16:05 ` Russell King - ARM Linux @ 2018-01-30 16:47 ` Baruch Siach 2018-01-30 16:58 ` Russell King - ARM Linux 0 siblings, 1 reply; 6+ messages in thread From: Baruch Siach @ 2018-01-30 16:47 UTC (permalink / raw) To: linux-arm-kernel Hi Russell, On Tue, Jan 30, 2018 at 04:05:17PM +0000, Russell King - ARM Linux wrote: > On Tue, Jan 30, 2018 at 10:01:26AM +0200, Baruch Siach wrote: > > Add description of the J25 and J27 UART headers of the Macchiatobin. They use > > uart peripherals that the CP0 (J25) and CP1 (J27) provides. > > The headers are just fed with the GPIO-level signals, which means any > industry standard usage would require additional RS232 level shifting > hardware. So, they're certainly not "generic" serial ports. > > While these are labelled up as UARTs, there's no reason to dedicate > them as such, they could equally be used to connect to a smart battery/ > UPS system as a SMBus port, or even to control system LEDs / read > the status of system buttons. > > Do you have an application for these headers? If not, it's probably > best to leave them alone (until we have DT overlay support sorted to > allow boot-loader configuration of hardware such as these - which is > being worked on for TI and other boards.) The console UART on the FTDI CON9 microUSB is not accessible when the board is mounted on a COTS ATX enclosure. So some users would like UART access through the UART headers. Would you be fine with this patch when 'status = "okay"' is omitted? baruch -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers 2018-01-30 16:47 ` Baruch Siach @ 2018-01-30 16:58 ` Russell King - ARM Linux 0 siblings, 0 replies; 6+ messages in thread From: Russell King - ARM Linux @ 2018-01-30 16:58 UTC (permalink / raw) To: linux-arm-kernel On Tue, Jan 30, 2018 at 06:47:12PM +0200, Baruch Siach wrote: > Hi Russell, > > On Tue, Jan 30, 2018 at 04:05:17PM +0000, Russell King - ARM Linux wrote: > > On Tue, Jan 30, 2018 at 10:01:26AM +0200, Baruch Siach wrote: > > > Add description of the J25 and J27 UART headers of the Macchiatobin. They use > > > uart peripherals that the CP0 (J25) and CP1 (J27) provides. > > > > The headers are just fed with the GPIO-level signals, which means any > > industry standard usage would require additional RS232 level shifting > > hardware. So, they're certainly not "generic" serial ports. > > > > While these are labelled up as UARTs, there's no reason to dedicate > > them as such, they could equally be used to connect to a smart battery/ > > UPS system as a SMBus port, or even to control system LEDs / read > > the status of system buttons. > > > > Do you have an application for these headers? If not, it's probably > > best to leave them alone (until we have DT overlay support sorted to > > allow boot-loader configuration of hardware such as these - which is > > being worked on for TI and other boards.) > > The console UART on the FTDI CON9 microUSB is not accessible when the board is > mounted on a COTS ATX enclosure. So some users would like UART access through > the UART headers. Please include this information in the commit log. I'll also point out that the latest board revision also offers a UART header for the standard console port too, so even if the microUSB is not available, the standard console is still usable. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 8.8Mbps down 630kbps up According to speedtest.net: 8.21Mbps down 510kbps up ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-01-30 16:58 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-01-30 8:01 [PATCH 1/2] arm64: dts: marvell: add CP110 uart peripherals Baruch Siach 2018-01-30 8:01 ` [PATCH 2/2] arm64: dts: marvell: mcbin: enable uart headers Baruch Siach 2018-01-30 10:27 ` Thomas Petazzoni 2018-01-30 16:05 ` Russell King - ARM Linux 2018-01-30 16:47 ` Baruch Siach 2018-01-30 16:58 ` Russell King - ARM Linux
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox