Message ID | cover.1605823502.git.cristian.ciocaltea@gmail.com |
---|---|
Headers | show |
Series | Add CMU/RMU/DMA/MMC/I2C support for Actions Semi S500 SoCs | expand |
On Fri, 20 Nov 2020 at 00:56, Cristian Ciocaltea <cristian.ciocaltea@gmail.com> wrote: > > Add MMC controller nodes for Actions Semi S500 SoC, in order to > facilitate access to SD/EMMC/SDIO cards. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org> Kind regards Uffe > --- > arch/arm/boot/dts/owl-s500.dtsi | 33 +++++++++++++++++++++++++++++++++ > 1 file changed, 33 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index b16172615db0..7af7c9e1119d 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -241,5 +241,38 @@ dma: dma-controller@b0260000 { > clocks = <&cmu CLK_DMAC>; > power-domains = <&sps S500_PD_DMA>; > }; > + > + mmc0: mmc@b0230000 { > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > + reg = <0xb0230000 0x38>; > + interrupts = <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD0>; > + resets = <&cmu RESET_SD0>; > + dmas = <&dma 2>; > + dma-names = "mmc"; > + status = "disabled"; > + }; > + > + mmc1: mmc@b0234000 { > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > + reg = <0xb0234000 0x38>; > + interrupts = <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD1>; > + resets = <&cmu RESET_SD1>; > + dmas = <&dma 3>; > + dma-names = "mmc"; > + status = "disabled"; > + }; > + > + mmc2: mmc@b0238000 { > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > + reg = <0xb0238000 0x38>; > + interrupts = <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD2>; > + resets = <&cmu RESET_SD2>; > + dmas = <&dma 4>; > + dma-names = "mmc"; > + status = "disabled"; > + }; > }; > }; > -- > 2.29.2 >
On Tue, Nov 24, 2020 at 03:25:15PM +0100, Ulf Hansson wrote: > On Fri, 20 Nov 2020 at 00:56, Cristian Ciocaltea > <cristian.ciocaltea@gmail.com> wrote: > > > > Add MMC controller nodes for Actions Semi S500 SoC, in order to > > facilitate access to SD/EMMC/SDIO cards. > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org> > > Kind regards > Uffe > Thanks for the review! Regards, Cristi > > > --- > > arch/arm/boot/dts/owl-s500.dtsi | 33 +++++++++++++++++++++++++++++++++ > > 1 file changed, 33 insertions(+) > > > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > > index b16172615db0..7af7c9e1119d 100644 > > --- a/arch/arm/boot/dts/owl-s500.dtsi > > +++ b/arch/arm/boot/dts/owl-s500.dtsi > > @@ -241,5 +241,38 @@ dma: dma-controller@b0260000 { > > clocks = <&cmu CLK_DMAC>; > > power-domains = <&sps S500_PD_DMA>; > > }; > > + > > + mmc0: mmc@b0230000 { > > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > > + reg = <0xb0230000 0x38>; > > + interrupts = <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_SD0>; > > + resets = <&cmu RESET_SD0>; > > + dmas = <&dma 2>; > > + dma-names = "mmc"; > > + status = "disabled"; > > + }; > > + > > + mmc1: mmc@b0234000 { > > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > > + reg = <0xb0234000 0x38>; > > + interrupts = <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_SD1>; > > + resets = <&cmu RESET_SD1>; > > + dmas = <&dma 3>; > > + dma-names = "mmc"; > > + status = "disabled"; > > + }; > > + > > + mmc2: mmc@b0238000 { > > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > > + reg = <0xb0238000 0x38>; > > + interrupts = <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_SD2>; > > + resets = <&cmu RESET_SD2>; > > + dmas = <&dma 4>; > > + dma-names = "mmc"; > > + status = "disabled"; > > + }; > > }; > > }; > > -- > > 2.29.2 > >
On Fri, Nov 20, 2020 at 01:55:55AM +0200, Cristian Ciocaltea wrote: > Add Clock Management Unit for Actions Semi S500 SoC. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index 1dbe4e8b38ac..5d5ad9db549b 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -5,6 +5,7 @@ > * Copyright (c) 2016-2017 Andreas Färber > */ > > +#include <dt-bindings/clock/actions,s500-cmu.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/power/owl-s500-powergate.h> > > @@ -70,6 +71,12 @@ hosc: hosc { > #clock-cells = <0>; > }; > > + losc: losc { > + compatible = "fixed-clock"; > + clock-frequency = <32768>; > + #clock-cells = <0>; > + }; > + > soc { > compatible = "simple-bus"; > #address-cells = <1>; > @@ -169,6 +176,13 @@ uart6: serial@b012c000 { > status = "disabled"; > }; > > + cmu: clock-controller@b0160000 { > + compatible = "actions,s500-cmu"; > + reg = <0xb0160000 0x8000>; > + clocks = <&hosc>, <&losc>; > + #clock-cells = <1>; > + }; > + > timer: timer@b0168000 { > compatible = "actions,s500-timer"; > reg = <0xb0168000 0x8000>; > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:55:56AM +0200, Cristian Ciocaltea wrote: > Set Clock Management Unit clocks for the UART nodes of Actions Semi > S500 SoCs. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Also in this patch itself, you need to remove the dummy "uart3_clk" from S500 dts. With that, Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index 5d5ad9db549b..ac3d04c75dd5 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -131,6 +131,7 @@ uart0: serial@b0120000 { > compatible = "actions,s500-uart", "actions,owl-uart"; > reg = <0xb0120000 0x2000>; > interrupts = <GIC_SPI 29 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_UART0>; > status = "disabled"; > }; > > @@ -138,6 +139,7 @@ uart1: serial@b0122000 { > compatible = "actions,s500-uart", "actions,owl-uart"; > reg = <0xb0122000 0x2000>; > interrupts = <GIC_SPI 30 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_UART1>; > status = "disabled"; > }; > > @@ -145,6 +147,7 @@ uart2: serial@b0124000 { > compatible = "actions,s500-uart", "actions,owl-uart"; > reg = <0xb0124000 0x2000>; > interrupts = <GIC_SPI 31 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_UART2>; > status = "disabled"; > }; > > @@ -152,6 +155,7 @@ uart3: serial@b0126000 { > compatible = "actions,s500-uart", "actions,owl-uart"; > reg = <0xb0126000 0x2000>; > interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_UART3>; > status = "disabled"; > }; > > @@ -159,6 +163,7 @@ uart4: serial@b0128000 { > compatible = "actions,s500-uart", "actions,owl-uart"; > reg = <0xb0128000 0x2000>; > interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_UART4>; > status = "disabled"; > }; > > @@ -166,6 +171,7 @@ uart5: serial@b012a000 { > compatible = "actions,s500-uart", "actions,owl-uart"; > reg = <0xb012a000 0x2000>; > interrupts = <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_UART5>; > status = "disabled"; > }; > > @@ -173,6 +179,7 @@ uart6: serial@b012c000 { > compatible = "actions,s500-uart", "actions,owl-uart"; > reg = <0xb012c000 0x2000>; > interrupts = <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_UART6>; > status = "disabled"; > }; > > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:55:57AM +0200, Cristian Ciocaltea wrote: > Add reset controller property and bindings header for the > Actions Semi S500 SoC DTS. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index ac3d04c75dd5..a57ce7d6d745 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -8,6 +8,7 @@ > #include <dt-bindings/clock/actions,s500-cmu.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/power/owl-s500-powergate.h> > +#include <dt-bindings/reset/actions,s500-reset.h> > > / { > compatible = "actions,s500"; > @@ -188,6 +189,7 @@ cmu: clock-controller@b0160000 { > reg = <0xb0160000 0x8000>; > clocks = <&hosc>, <&losc>; > #clock-cells = <1>; > + #reset-cells = <1>; > }; > > timer: timer@b0168000 { > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:00AM +0200, Cristian Ciocaltea wrote: > Add DMA controller node for Actions Semi S500 SoC. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index a57ce7d6d745..449e9807c4ec 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -207,5 +207,19 @@ sps: power-controller@b01b0100 { > reg = <0xb01b0100 0x100>; > #power-domain-cells = <1>; > }; > + > + dma: dma-controller@b0260000 { > + compatible = "actions,s500-dma"; > + reg = <0xb0260000 0xd00>; > + interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 59 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 60 IRQ_TYPE_LEVEL_HIGH>; > + #dma-cells = <1>; > + dma-channels = <12>; > + dma-requests = <46>; > + clocks = <&cmu CLK_DMAC>; > + power-domains = <&sps S500_PD_DMA>; > + }; > }; > }; > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:01AM +0200, Cristian Ciocaltea wrote: > Add pinctrl node for Actions Semi S500 SoC. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index 449e9807c4ec..b16172615db0 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -6,6 +6,7 @@ > */ > > #include <dt-bindings/clock/actions,s500-cmu.h> > +#include <dt-bindings/gpio/gpio.h> > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/power/owl-s500-powergate.h> > #include <dt-bindings/reset/actions,s500-reset.h> > @@ -208,6 +209,25 @@ sps: power-controller@b01b0100 { > #power-domain-cells = <1>; > }; > > + pinctrl: pinctrl@b01b0000 { > + compatible = "actions,s500-pinctrl"; > + reg = <0xb01b0000 0x40>, /* GPIO */ > + <0xb01b0040 0x10>, /* Multiplexing Control */ > + <0xb01b0060 0x18>, /* PAD Control */ > + <0xb01b0080 0xc>; /* PAD Drive Capacity */ > + clocks = <&cmu CLK_GPIO>; > + gpio-controller; > + gpio-ranges = <&pinctrl 0 0 132>; > + #gpio-cells = <2>; > + interrupt-controller; > + #interrupt-cells = <2>; > + interrupts = <GIC_SPI 36 IRQ_TYPE_LEVEL_HIGH>, /* GPIOA */ > + <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>, /* GPIOB */ > + <GIC_SPI 38 IRQ_TYPE_LEVEL_HIGH>, /* GPIOC */ > + <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>, /* GPIOD */ > + <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; /* GPIOE */ > + }; > + > dma: dma-controller@b0260000 { > compatible = "actions,s500-dma"; > reg = <0xb0260000 0xd00>; > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:03AM +0200, Cristian Ciocaltea wrote: > Add MMC controller nodes for Actions Semi S500 SoC, in order to > facilitate access to SD/EMMC/SDIO cards. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 33 +++++++++++++++++++++++++++++++++ > 1 file changed, 33 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index b16172615db0..7af7c9e1119d 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -241,5 +241,38 @@ dma: dma-controller@b0260000 { > clocks = <&cmu CLK_DMAC>; > power-domains = <&sps S500_PD_DMA>; > }; > + > + mmc0: mmc@b0230000 { > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > + reg = <0xb0230000 0x38>; > + interrupts = <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD0>; > + resets = <&cmu RESET_SD0>; > + dmas = <&dma 2>; > + dma-names = "mmc"; > + status = "disabled"; > + }; > + > + mmc1: mmc@b0234000 { > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > + reg = <0xb0234000 0x38>; > + interrupts = <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD1>; > + resets = <&cmu RESET_SD1>; > + dmas = <&dma 3>; > + dma-names = "mmc"; > + status = "disabled"; > + }; > + > + mmc2: mmc@b0238000 { > + compatible = "actions,s500-mmc", "actions,owl-mmc"; > + reg = <0xb0238000 0x38>; > + interrupts = <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD2>; > + resets = <&cmu RESET_SD2>; > + dmas = <&dma 4>; > + dma-names = "mmc"; > + status = "disabled"; > + }; > }; > }; > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:07AM +0200, Cristian Ciocaltea wrote: > Add I2C controller nodes for Actions Semi S500 SoC. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 40 +++++++++++++++++++++++++++++++++ > 1 file changed, 40 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index 7af7c9e1119d..55f8b8c2e149 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -193,6 +193,46 @@ cmu: clock-controller@b0160000 { > #reset-cells = <1>; > }; > > + i2c0: i2c@b0170000 { > + compatible = "actions,s500-i2c"; > + reg = <0xb0170000 0x4000>; > + clocks = <&cmu CLK_I2C0>; > + interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + }; > + > + i2c1: i2c@b0174000 { > + compatible = "actions,s500-i2c"; > + reg = <0xb0174000 0x4000>; > + clocks = <&cmu CLK_I2C1>; > + interrupts = <GIC_SPI 26 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + }; > + > + i2c2: i2c@b0178000 { > + compatible = "actions,s500-i2c"; > + reg = <0xb0178000 0x4000>; > + clocks = <&cmu CLK_I2C2>; > + interrupts = <GIC_SPI 27 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + }; > + > + i2c3: i2c@b017c000 { > + compatible = "actions,s500-i2c"; > + reg = <0xb017c000 0x4000>; > + clocks = <&cmu CLK_I2C3>; > + interrupts = <GIC_SPI 28 IRQ_TYPE_LEVEL_HIGH>; > + #address-cells = <1>; > + #size-cells = <0>; > + status = "disabled"; > + }; > + > timer: timer@b0168000 { > compatible = "actions,s500-timer"; > reg = <0xb0168000 0x8000>; > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:08AM +0200, Cristian Ciocaltea wrote: > Add SIRQ controller node for Actions Semi S500 SoC. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500.dtsi | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > index 55f8b8c2e149..cd635f222d26 100644 > --- a/arch/arm/boot/dts/owl-s500.dtsi > +++ b/arch/arm/boot/dts/owl-s500.dtsi > @@ -233,6 +233,16 @@ i2c3: i2c@b017c000 { > status = "disabled"; > }; > > + sirq: interrupt-controller@b01b0200 { > + compatible = "actions,s500-sirq"; > + reg = <0xb01b0200 0x4>; > + interrupt-controller; > + #interrupt-cells = <2>; > + interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>, /* SIRQ0 */ > + <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>, /* SIRQ1 */ > + <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>; /* SIRQ2 */ > + }; > + > timer: timer@b0168000 { > compatible = "actions,s500-timer"; > reg = <0xb0168000 0x8000>; > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:09AM +0200, Cristian Ciocaltea wrote: > Remove the UART fixed clock for RoseapplePi SBC and switch to using > the clock provided by CMU. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Please club this patch with the one adding CMU clock. Thanks, Mani > --- > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/arch/arm/boot/dts/owl-s500-roseapplepi.dts b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > index a2087e617cb2..800edf5d2d12 100644 > --- a/arch/arm/boot/dts/owl-s500-roseapplepi.dts > +++ b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > @@ -25,12 +25,6 @@ memory@0 { > device_type = "memory"; > reg = <0x0 0x80000000>; /* 2GB */ > }; > - > - uart2_clk: uart2-clk { > - compatible = "fixed-clock"; > - clock-frequency = <921600>; > - #clock-cells = <0>; > - }; > }; > > &twd_timer { > @@ -43,5 +37,4 @@ &timer { > > &uart2 { > status = "okay"; > - clocks = <&uart2_clk>; > }; > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:10AM +0200, Cristian Ciocaltea wrote: > Add uSD support for RoseapplePi SBC using a fixed regulator as a > temporary solution until PMIC support becomes available. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > --- > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 50 ++++++++++++++++++++++ > 1 file changed, 50 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500-roseapplepi.dts b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > index 800edf5d2d12..fe9ae3619422 100644 > --- a/arch/arm/boot/dts/owl-s500-roseapplepi.dts > +++ b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > @@ -14,6 +14,7 @@ / { > model = "Roseapple Pi"; > > aliases { > + mmc0 = &mmc0; > serial2 = &uart2; > }; > > @@ -25,6 +26,55 @@ memory@0 { > device_type = "memory"; > reg = <0x0 0x80000000>; /* 2GB */ > }; > + > + /* Fixed regulator used in the absence of PMIC */ > + sd_vcc: sd-vcc { Is this the exact name in the schematics? Thanks, Mani > + compatible = "regulator-fixed"; > + regulator-name = "fixed-3.1V"; > + regulator-min-microvolt = <3100000>; > + regulator-max-microvolt = <3100000>; > + regulator-always-on; > + }; > +}; > + > +&pinctrl { > + mmc0_pins: mmc0-pins { > + pinmux { > + groups = "sd0_d0_mfp", "sd0_d1_mfp", "sd0_d2_d3_mfp", > + "sd0_cmd_mfp", "sd0_clk_mfp"; > + function = "sd0"; > + }; > + > + drv-pinconf { > + groups = "sd0_d0_d3_drv", "sd0_cmd_drv", "sd0_clk_drv"; > + drive-strength = <8>; > + }; > + > + bias0-pinconf { > + pins = "sd0_d0", "sd0_d1", "sd0_d2", > + "sd0_d3", "sd0_cmd"; > + bias-pull-up; > + }; > + > + bias1-pinconf { > + pins = "sd0_clk"; > + bias-pull-down; > + }; > + }; > +}; > + > +/* uSD */ > +&mmc0 { > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc0_pins>; > + no-sdio; > + no-mmc; > + no-1-8-v; > + cd-gpios = <&pinctrl 117 GPIO_ACTIVE_LOW>; > + bus-width = <4>; > + vmmc-supply = <&sd_vcc>; > + vqmmc-supply = <&sd_vcc>; > }; > > &twd_timer { > -- > 2.29.2 >
On Fri, Nov 20, 2020 at 01:56:11AM +0200, Cristian Ciocaltea wrote: > Add pinctrl definitions for the I2C controllers used in RoseapplePi SBC. > For the moment enable only I2C0, which is used by the ATC2603C PMIC. > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> Earlier we used to add "_default" suffix for the pin groups to differentiate between active and sleep states. But I guess we can just keep the suffix away until we hit usecase. Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks, Mani > --- > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 44 ++++++++++++++++++++++ > 1 file changed, 44 insertions(+) > > diff --git a/arch/arm/boot/dts/owl-s500-roseapplepi.dts b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > index fe9ae3619422..ff91561ca99c 100644 > --- a/arch/arm/boot/dts/owl-s500-roseapplepi.dts > +++ b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > @@ -37,7 +37,51 @@ sd_vcc: sd-vcc { > }; > }; > > +&i2c0 { > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&i2c0_pins>; > +}; > + > +&i2c1 { > + status = "disabled"; > + pinctrl-names = "default"; > + pinctrl-0 = <&i2c1_pins>; > +}; > + > +&i2c2 { > + status = "disabled"; > + pinctrl-names = "default"; > + pinctrl-0 = <&i2c2_pins>; > +}; > + > &pinctrl { > + i2c0_pins: i2c0-pins { > + pinmux { > + groups = "i2c0_mfp"; > + function = "i2c0"; > + }; > + > + pinconf { > + pins = "i2c0_sclk", "i2c0_sdata"; > + bias-pull-up; > + }; > + }; > + > + i2c1_pins: i2c1-pins { > + pinconf { > + pins = "i2c1_sclk", "i2c1_sdata"; > + bias-pull-up; > + }; > + }; > + > + i2c2_pins: i2c2-pins { > + pinconf { > + pins = "i2c2_sclk", "i2c2_sdata"; > + bias-pull-up; > + }; > + }; > + > mmc0_pins: mmc0-pins { > pinmux { > groups = "sd0_d0_mfp", "sd0_d1_mfp", "sd0_d2_d3_mfp", > -- > 2.29.2 >
On Sat, Nov 28, 2020 at 12:52:12PM +0530, Manivannan Sadhasivam wrote: > On Fri, Nov 20, 2020 at 01:55:55AM +0200, Cristian Ciocaltea wrote: > > Add Clock Management Unit for Actions Semi S500 SoC. > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Thanks for taking the time to review this patch series! Kind regards, Cristi > Thanks, > Mani > > > --- > > arch/arm/boot/dts/owl-s500.dtsi | 14 ++++++++++++++ > > 1 file changed, 14 insertions(+) > > > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > > index 1dbe4e8b38ac..5d5ad9db549b 100644 > > --- a/arch/arm/boot/dts/owl-s500.dtsi > > +++ b/arch/arm/boot/dts/owl-s500.dtsi > > @@ -5,6 +5,7 @@ > > * Copyright (c) 2016-2017 Andreas Färber > > */ > > > > +#include <dt-bindings/clock/actions,s500-cmu.h> > > #include <dt-bindings/interrupt-controller/arm-gic.h> > > #include <dt-bindings/power/owl-s500-powergate.h> > > > > @@ -70,6 +71,12 @@ hosc: hosc { > > #clock-cells = <0>; > > }; > > > > + losc: losc { > > + compatible = "fixed-clock"; > > + clock-frequency = <32768>; > > + #clock-cells = <0>; > > + }; > > + > > soc { > > compatible = "simple-bus"; > > #address-cells = <1>; > > @@ -169,6 +176,13 @@ uart6: serial@b012c000 { > > status = "disabled"; > > }; > > > > + cmu: clock-controller@b0160000 { > > + compatible = "actions,s500-cmu"; > > + reg = <0xb0160000 0x8000>; > > + clocks = <&hosc>, <&losc>; > > + #clock-cells = <1>; > > + }; > > + > > timer: timer@b0168000 { > > compatible = "actions,s500-timer"; > > reg = <0xb0168000 0x8000>; > > -- > > 2.29.2 > >
On Sat, Nov 28, 2020 at 12:55:13PM +0530, Manivannan Sadhasivam wrote: > On Fri, Nov 20, 2020 at 01:55:56AM +0200, Cristian Ciocaltea wrote: > > Set Clock Management Unit clocks for the UART nodes of Actions Semi > > S500 SoCs. > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > Also in this patch itself, you need to remove the dummy "uart3_clk" from > S500 dts. With that, I have prepared the updated patch with the requested changes, including also the merge of the RoseapplePi related patch (no. 15 in this series). Thanks, Cristi > Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > > Thanks, > Mani > > > --- > > arch/arm/boot/dts/owl-s500.dtsi | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/arch/arm/boot/dts/owl-s500.dtsi b/arch/arm/boot/dts/owl-s500.dtsi > > index 5d5ad9db549b..ac3d04c75dd5 100644 > > --- a/arch/arm/boot/dts/owl-s500.dtsi > > +++ b/arch/arm/boot/dts/owl-s500.dtsi > > @@ -131,6 +131,7 @@ uart0: serial@b0120000 { > > compatible = "actions,s500-uart", "actions,owl-uart"; > > reg = <0xb0120000 0x2000>; > > interrupts = <GIC_SPI 29 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_UART0>; > > status = "disabled"; > > }; > > > > @@ -138,6 +139,7 @@ uart1: serial@b0122000 { > > compatible = "actions,s500-uart", "actions,owl-uart"; > > reg = <0xb0122000 0x2000>; > > interrupts = <GIC_SPI 30 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_UART1>; > > status = "disabled"; > > }; > > > > @@ -145,6 +147,7 @@ uart2: serial@b0124000 { > > compatible = "actions,s500-uart", "actions,owl-uart"; > > reg = <0xb0124000 0x2000>; > > interrupts = <GIC_SPI 31 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_UART2>; > > status = "disabled"; > > }; > > > > @@ -152,6 +155,7 @@ uart3: serial@b0126000 { > > compatible = "actions,s500-uart", "actions,owl-uart"; > > reg = <0xb0126000 0x2000>; > > interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_UART3>; > > status = "disabled"; > > }; > > > > @@ -159,6 +163,7 @@ uart4: serial@b0128000 { > > compatible = "actions,s500-uart", "actions,owl-uart"; > > reg = <0xb0128000 0x2000>; > > interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_UART4>; > > status = "disabled"; > > }; > > > > @@ -166,6 +171,7 @@ uart5: serial@b012a000 { > > compatible = "actions,s500-uart", "actions,owl-uart"; > > reg = <0xb012a000 0x2000>; > > interrupts = <GIC_SPI 34 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_UART5>; > > status = "disabled"; > > }; > > > > @@ -173,6 +179,7 @@ uart6: serial@b012c000 { > > compatible = "actions,s500-uart", "actions,owl-uart"; > > reg = <0xb012c000 0x2000>; > > interrupts = <GIC_SPI 35 IRQ_TYPE_LEVEL_HIGH>; > > + clocks = <&cmu CLK_UART6>; > > status = "disabled"; > > }; > > > > -- > > 2.29.2 > >
On Sat, Nov 28, 2020 at 01:06:50PM +0530, Manivannan Sadhasivam wrote: > On Fri, Nov 20, 2020 at 01:56:09AM +0200, Cristian Ciocaltea wrote: > > Remove the UART fixed clock for RoseapplePi SBC and switch to using > > the clock provided by CMU. > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > Please club this patch with the one adding CMU clock. Done. Thanks, Cristi > Thanks, > Mani > > > --- > > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 7 ------- > > 1 file changed, 7 deletions(-) > > > > diff --git a/arch/arm/boot/dts/owl-s500-roseapplepi.dts b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > index a2087e617cb2..800edf5d2d12 100644 > > --- a/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > +++ b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > @@ -25,12 +25,6 @@ memory@0 { > > device_type = "memory"; > > reg = <0x0 0x80000000>; /* 2GB */ > > }; > > - > > - uart2_clk: uart2-clk { > > - compatible = "fixed-clock"; > > - clock-frequency = <921600>; > > - #clock-cells = <0>; > > - }; > > }; > > > > &twd_timer { > > @@ -43,5 +37,4 @@ &timer { > > > > &uart2 { > > status = "okay"; > > - clocks = <&uart2_clk>; > > }; > > -- > > 2.29.2 > >
On Sat, Nov 28, 2020 at 01:08:51PM +0530, Manivannan Sadhasivam wrote: > On Fri, Nov 20, 2020 at 01:56:10AM +0200, Cristian Ciocaltea wrote: > > Add uSD support for RoseapplePi SBC using a fixed regulator as a > > temporary solution until PMIC support becomes available. > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > --- > > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 50 ++++++++++++++++++++++ > > 1 file changed, 50 insertions(+) > > > > diff --git a/arch/arm/boot/dts/owl-s500-roseapplepi.dts b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > index 800edf5d2d12..fe9ae3619422 100644 > > --- a/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > +++ b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > @@ -14,6 +14,7 @@ / { > > model = "Roseapple Pi"; > > > > aliases { > > + mmc0 = &mmc0; > > serial2 = &uart2; > > }; > > > > @@ -25,6 +26,55 @@ memory@0 { > > device_type = "memory"; > > reg = <0x0 0x80000000>; /* 2GB */ > > }; > > + > > + /* Fixed regulator used in the absence of PMIC */ > > + sd_vcc: sd-vcc { > > Is this the exact name in the schematics? Yes, it is referred as "SD_VCC". Once the support for the ATC260x PMIC will be available, the following item will be present in the regulators list: sd_vcc: switchldo1 { regulator-name = "SD_VCC"; regulator-min-microvolt = <3000000>; regulator-max-microvolt = <3300000>; regulator-always-on; regulator-boot-on; }; Thanks, Cristi > Thanks, > Mani > > > + compatible = "regulator-fixed"; > > + regulator-name = "fixed-3.1V"; > > + regulator-min-microvolt = <3100000>; > > + regulator-max-microvolt = <3100000>; > > + regulator-always-on; > > + }; > > +}; > > + > > +&pinctrl { > > + mmc0_pins: mmc0-pins { > > + pinmux { > > + groups = "sd0_d0_mfp", "sd0_d1_mfp", "sd0_d2_d3_mfp", > > + "sd0_cmd_mfp", "sd0_clk_mfp"; > > + function = "sd0"; > > + }; > > + > > + drv-pinconf { > > + groups = "sd0_d0_d3_drv", "sd0_cmd_drv", "sd0_clk_drv"; > > + drive-strength = <8>; > > + }; > > + > > + bias0-pinconf { > > + pins = "sd0_d0", "sd0_d1", "sd0_d2", > > + "sd0_d3", "sd0_cmd"; > > + bias-pull-up; > > + }; > > + > > + bias1-pinconf { > > + pins = "sd0_clk"; > > + bias-pull-down; > > + }; > > + }; > > +}; > > + > > +/* uSD */ > > +&mmc0 { > > + status = "okay"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&mmc0_pins>; > > + no-sdio; > > + no-mmc; > > + no-1-8-v; > > + cd-gpios = <&pinctrl 117 GPIO_ACTIVE_LOW>; > > + bus-width = <4>; > > + vmmc-supply = <&sd_vcc>; > > + vqmmc-supply = <&sd_vcc>; > > }; > > > > &twd_timer { > > -- > > 2.29.2 > >
On Sat, Nov 28, 2020 at 01:11:08PM +0530, Manivannan Sadhasivam wrote: > On Fri, Nov 20, 2020 at 01:56:11AM +0200, Cristian Ciocaltea wrote: > > Add pinctrl definitions for the I2C controllers used in RoseapplePi SBC. > > For the moment enable only I2C0, which is used by the ATC2603C PMIC. > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > Earlier we used to add "_default" suffix for the pin groups to > differentiate between active and sleep states. But I guess we can just > keep the suffix away until we hit usecase. I had actually used this suffix when I submitted the first revision of the S500 pinctrl driver, but I dropped it later in the review process since it was considered an uncommon approach: https://lore.kernel.org/lkml/CAL_Jsq+8bX5duv=116e=hve1L-h8a=5quqCHVtSAs4PjK6xc1w@mail.gmail.com/ Thanks, Cristi > Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > > Thanks, > Mani > > > --- > > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 44 ++++++++++++++++++++++ > > 1 file changed, 44 insertions(+) > > > > diff --git a/arch/arm/boot/dts/owl-s500-roseapplepi.dts b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > index fe9ae3619422..ff91561ca99c 100644 > > --- a/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > +++ b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > @@ -37,7 +37,51 @@ sd_vcc: sd-vcc { > > }; > > }; > > > > +&i2c0 { > > + status = "okay"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&i2c0_pins>; > > +}; > > + > > +&i2c1 { > > + status = "disabled"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&i2c1_pins>; > > +}; > > + > > +&i2c2 { > > + status = "disabled"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&i2c2_pins>; > > +}; > > + > > &pinctrl { > > + i2c0_pins: i2c0-pins { > > + pinmux { > > + groups = "i2c0_mfp"; > > + function = "i2c0"; > > + }; > > + > > + pinconf { > > + pins = "i2c0_sclk", "i2c0_sdata"; > > + bias-pull-up; > > + }; > > + }; > > + > > + i2c1_pins: i2c1-pins { > > + pinconf { > > + pins = "i2c1_sclk", "i2c1_sdata"; > > + bias-pull-up; > > + }; > > + }; > > + > > + i2c2_pins: i2c2-pins { > > + pinconf { > > + pins = "i2c2_sclk", "i2c2_sdata"; > > + bias-pull-up; > > + }; > > + }; > > + > > mmc0_pins: mmc0-pins { > > pinmux { > > groups = "sd0_d0_mfp", "sd0_d1_mfp", "sd0_d2_d3_mfp", > > -- > > 2.29.2 > >
On Sun, Nov 29, 2020 at 08:35:32PM +0200, Cristian Ciocaltea wrote: > On Sat, Nov 28, 2020 at 01:08:51PM +0530, Manivannan Sadhasivam wrote: > > On Fri, Nov 20, 2020 at 01:56:10AM +0200, Cristian Ciocaltea wrote: > > > Add uSD support for RoseapplePi SBC using a fixed regulator as a > > > temporary solution until PMIC support becomes available. > > > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > > --- > > > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 50 ++++++++++++++++++++++ > > > 1 file changed, 50 insertions(+) > > > > > > diff --git a/arch/arm/boot/dts/owl-s500-roseapplepi.dts b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > > index 800edf5d2d12..fe9ae3619422 100644 > > > --- a/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > > +++ b/arch/arm/boot/dts/owl-s500-roseapplepi.dts > > > @@ -14,6 +14,7 @@ / { > > > model = "Roseapple Pi"; > > > > > > aliases { > > > + mmc0 = &mmc0; > > > serial2 = &uart2; > > > }; > > > > > > @@ -25,6 +26,55 @@ memory@0 { > > > device_type = "memory"; > > > reg = <0x0 0x80000000>; /* 2GB */ > > > }; > > > + > > > + /* Fixed regulator used in the absence of PMIC */ > > > + sd_vcc: sd-vcc { > > > > Is this the exact name in the schematics? > > Yes, it is referred as "SD_VCC". Once the support for the ATC260x PMIC > will be available, the following item will be present in the regulators > list: > > sd_vcc: switchldo1 { > regulator-name = "SD_VCC"; > regulator-min-microvolt = <3000000>; > regulator-max-microvolt = <3300000>; > regulator-always-on; > regulator-boot-on; > }; okay, looks fine to me. Thanks, Mani > > Thanks, > Cristi > > > Thanks, > > Mani > > > > > + compatible = "regulator-fixed"; > > > + regulator-name = "fixed-3.1V"; > > > + regulator-min-microvolt = <3100000>; > > > + regulator-max-microvolt = <3100000>; > > > + regulator-always-on; > > > + }; > > > +}; > > > + > > > +&pinctrl { > > > + mmc0_pins: mmc0-pins { > > > + pinmux { > > > + groups = "sd0_d0_mfp", "sd0_d1_mfp", "sd0_d2_d3_mfp", > > > + "sd0_cmd_mfp", "sd0_clk_mfp"; > > > + function = "sd0"; > > > + }; > > > + > > > + drv-pinconf { > > > + groups = "sd0_d0_d3_drv", "sd0_cmd_drv", "sd0_clk_drv"; > > > + drive-strength = <8>; > > > + }; > > > + > > > + bias0-pinconf { > > > + pins = "sd0_d0", "sd0_d1", "sd0_d2", > > > + "sd0_d3", "sd0_cmd"; > > > + bias-pull-up; > > > + }; > > > + > > > + bias1-pinconf { > > > + pins = "sd0_clk"; > > > + bias-pull-down; > > > + }; > > > + }; > > > +}; > > > + > > > +/* uSD */ > > > +&mmc0 { > > > + status = "okay"; > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&mmc0_pins>; > > > + no-sdio; > > > + no-mmc; > > > + no-1-8-v; > > > + cd-gpios = <&pinctrl 117 GPIO_ACTIVE_LOW>; > > > + bus-width = <4>; > > > + vmmc-supply = <&sd_vcc>; > > > + vqmmc-supply = <&sd_vcc>; > > > }; > > > > > > &twd_timer { > > > -- > > > 2.29.2 > > >
Hi Cristi, On Fri, Nov 20, 2020 at 01:55:54AM +0200, Cristian Ciocaltea wrote: > Hi, > > This patchset brings a series of improvements for the Actions Semi S500 > SoCs family, by adding support for Clock & Reset Management Units, DMA, > MMC, I2C & SIRQ controllers. > > Please note the patches consist mostly of DTS and bindings/compatibles > changes, since all the work they depend on has been already merged, > i.e. clock fixes/additions, pinctrl driver, sirq driver. > > For the moment, I have only enabled the features I could test on > RoseapplePi SBC. > I was hoping to apply this series for v5.11 but we ran out of time. So expect this series to be in v5.12. Thanks, Mani > Thanks, > Cristi > > Changes in v2: > - Added new bindings/compatibles for S500 DMA, MMC & I2C controllers > - Added support for the SIRQ controller > - Added new entries in MAINTAINERS > - Updated naming of some patches in v1 > > Cristian Ciocaltea (18): > arm: dts: owl-s500: Add Clock Management Unit > arm: dts: owl-s500: Set CMU clocks for UARTs > arm: dts: owl-s500: Add Reset controller > dt-bindings: dma: owl: Add compatible string for Actions Semi S500 SoC > dmaengine: owl: Add compatible for the Actions Semi S500 DMA > controller > arm: dts: owl-s500: Add DMA controller > arm: dts: owl-s500: Add pinctrl & GPIO support > dt-bindings: mmc: owl: Add compatible string for Actions Semi S500 SoC > arm: dts: owl-s500: Add MMC support > dt-bindings: i2c: owl: Convert Actions Semi Owl binding to a schema > MAINTAINERS: Update entry for Actions Semi Owl I2C binding > i2c: owl: Add compatible for the Actions Semi S500 I2C controller > arm: dts: owl-s500: Add I2C support > arm: dts: owl-s500: Add SIRQ controller > arm: dts: owl-s500-roseapplepi: Use UART clock from CMU > arm: dts: owl-s500-roseapplepi: Add uSD support > arm: dts: owl-s500-roseapplepi: Add I2C pinctrl configuration > MAINTAINERS: Add linux-actions ML for Actions Semi Arch > > .../devicetree/bindings/dma/owl-dma.yaml | 5 +- > .../devicetree/bindings/i2c/i2c-owl.txt | 29 ---- > .../devicetree/bindings/i2c/i2c-owl.yaml | 62 ++++++++ > .../devicetree/bindings/mmc/owl-mmc.yaml | 4 +- > MAINTAINERS | 3 +- > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 97 +++++++++++- > arch/arm/boot/dts/owl-s500.dtsi | 140 ++++++++++++++++++ > drivers/dma/owl-dma.c | 1 + > drivers/i2c/busses/i2c-owl.c | 1 + > 9 files changed, 304 insertions(+), 38 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/i2c/i2c-owl.txt > create mode 100644 Documentation/devicetree/bindings/i2c/i2c-owl.yaml > > -- > 2.29.2 >
Hi Mani, On Sat, Dec 05, 2020 at 11:41:15AM +0530, Manivannan Sadhasivam wrote: > Hi Cristi, > > On Fri, Nov 20, 2020 at 01:55:54AM +0200, Cristian Ciocaltea wrote: > > Hi, > > > > This patchset brings a series of improvements for the Actions Semi S500 > > SoCs family, by adding support for Clock & Reset Management Units, DMA, > > MMC, I2C & SIRQ controllers. > > > > Please note the patches consist mostly of DTS and bindings/compatibles > > changes, since all the work they depend on has been already merged, > > i.e. clock fixes/additions, pinctrl driver, sirq driver. > > > > For the moment, I have only enabled the features I could test on > > RoseapplePi SBC. > > > > I was hoping to apply this series for v5.11 but we ran out of time. So expect > this series to be in v5.12. No problem, thanks a lot for taking care of this! Kind regards, Cristi > Thanks, > Mani > > > Thanks, > > Cristi > > > > Changes in v2: > > - Added new bindings/compatibles for S500 DMA, MMC & I2C controllers > > - Added support for the SIRQ controller > > - Added new entries in MAINTAINERS > > - Updated naming of some patches in v1 > > > > Cristian Ciocaltea (18): > > arm: dts: owl-s500: Add Clock Management Unit > > arm: dts: owl-s500: Set CMU clocks for UARTs > > arm: dts: owl-s500: Add Reset controller > > dt-bindings: dma: owl: Add compatible string for Actions Semi S500 SoC > > dmaengine: owl: Add compatible for the Actions Semi S500 DMA > > controller > > arm: dts: owl-s500: Add DMA controller > > arm: dts: owl-s500: Add pinctrl & GPIO support > > dt-bindings: mmc: owl: Add compatible string for Actions Semi S500 SoC > > arm: dts: owl-s500: Add MMC support > > dt-bindings: i2c: owl: Convert Actions Semi Owl binding to a schema > > MAINTAINERS: Update entry for Actions Semi Owl I2C binding > > i2c: owl: Add compatible for the Actions Semi S500 I2C controller > > arm: dts: owl-s500: Add I2C support > > arm: dts: owl-s500: Add SIRQ controller > > arm: dts: owl-s500-roseapplepi: Use UART clock from CMU > > arm: dts: owl-s500-roseapplepi: Add uSD support > > arm: dts: owl-s500-roseapplepi: Add I2C pinctrl configuration > > MAINTAINERS: Add linux-actions ML for Actions Semi Arch > > > > .../devicetree/bindings/dma/owl-dma.yaml | 5 +- > > .../devicetree/bindings/i2c/i2c-owl.txt | 29 ---- > > .../devicetree/bindings/i2c/i2c-owl.yaml | 62 ++++++++ > > .../devicetree/bindings/mmc/owl-mmc.yaml | 4 +- > > MAINTAINERS | 3 +- > > arch/arm/boot/dts/owl-s500-roseapplepi.dts | 97 +++++++++++- > > arch/arm/boot/dts/owl-s500.dtsi | 140 ++++++++++++++++++ > > drivers/dma/owl-dma.c | 1 + > > drivers/i2c/busses/i2c-owl.c | 1 + > > 9 files changed, 304 insertions(+), 38 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/i2c/i2c-owl.txt > > create mode 100644 Documentation/devicetree/bindings/i2c/i2c-owl.yaml > > > > -- > > 2.29.2 > >
On Sat, Nov 28, 2020 at 01:05:16PM +0530, Manivannan Sadhasivam wrote: > On Fri, Nov 20, 2020 at 01:56:07AM +0200, Cristian Ciocaltea wrote: > > Add I2C controller nodes for Actions Semi S500 SoC. > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> DTS changes usually go in via arm-soc, so I am not picking this unless there is a reason to do so.
On Wed, Dec 09, 2020 at 09:25:06PM +0100, Wolfram Sang wrote: > On Sat, Nov 28, 2020 at 01:05:16PM +0530, Manivannan Sadhasivam wrote: > > On Fri, Nov 20, 2020 at 01:56:07AM +0200, Cristian Ciocaltea wrote: > > > Add I2C controller nodes for Actions Semi S500 SoC. > > > > > > Signed-off-by: Cristian Ciocaltea <cristian.ciocaltea@gmail.com> > > > > Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > > DTS changes usually go in via arm-soc, so I am not picking this unless > there is a reason to do so. > No you should not. This patch will go through actions sub tree which me or Andreas will pick it. Thanks, Mani