| Message ID | 20210301060518.19550-1-a-govindraju@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: ti: k3-am642-evm: Add support for SPI EEPROM | expand |
On 3/1/21 11:35 AM, Aswath Govindraju wrote: > Add pinmux details and device tree node for the EEPROM attached to SPI0 > module in main domain. > > Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> > --- Reviewed-by: Vignesh Raghavendra <vigneshr@ti.com> Regards Vignesh > > This patch depends on, > https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210301055109.17626-3-a-govindraju@ti.com/ > > arch/arm64/boot/dts/ti/k3-am642-evm.dts | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-am642-evm.dts b/arch/arm64/boot/dts/ti/k3-am642-evm.dts > index bfd849a29655..bc5bd7f896ab 100644 > --- a/arch/arm64/boot/dts/ti/k3-am642-evm.dts > +++ b/arch/arm64/boot/dts/ti/k3-am642-evm.dts > @@ -139,6 +139,15 @@ > AM64X_IOPAD(0x02a8, PIN_OUTPUT, 0) /* (E19) USB0_DRVVBUS */ > >; > }; > + > + main_spi0_pins_default: main-spi0-pins-default { > + pinctrl-single,pins = < > + AM64X_IOPAD(0x0210, PIN_INPUT, 0) /* (D13) SPI0_CLK */ > + AM64X_IOPAD(0x0208, PIN_OUTPUT, 0) /* (D12) SPI0_CS0 */ > + AM64X_IOPAD(0x0214, PIN_OUTPUT, 0) /* (A13) SPI0_D0 */ > + AM64X_IOPAD(0x0218, PIN_INPUT, 0) /* (A14) SPI0_D1 */ > + >; > + }; > }; > > &main_uart0 { > @@ -245,6 +254,19 @@ > pinctrl-0 = <&main_usb0_pins_default>; > }; > > +&main_spi0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&main_spi0_pins_default>; > + ti,pindir-d0-out-d1-in = <1>; > + eeprom@0 { > + compatible = "microchip,93lc46b"; > + reg = <0>; > + spi-max-frequency = <1000000>; > + spi-cs-high; > + data-size = <16>; > + }; > +}; > + > &sdhci0 { > /* emmc */ > bus-width = <8>; >
On 20:18-20210309, Vignesh Raghavendra wrote: > > > On 3/1/21 11:35 AM, Aswath Govindraju wrote: > > Add pinmux details and device tree node for the EEPROM attached to SPI0 > > module in main domain. > > > > Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> > > --- > > Reviewed-by: Vignesh Raghavendra <vigneshr@ti.com> > > Regards > Vignesh > > > > > This patch depends on, > > https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210301055109.17626-3-a-govindraju@ti.com/ Can you drop this dependency and rebase on top of my -next branch or linux-next ? I am not able to apply the patch directly and would like to avoid hand modifying the patch. > > > > arch/arm64/boot/dts/ti/k3-am642-evm.dts | 22 ++++++++++++++++++++++ > > 1 file changed, 22 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/ti/k3-am642-evm.dts b/arch/arm64/boot/dts/ti/k3-am642-evm.dts > > index bfd849a29655..bc5bd7f896ab 100644 > > --- a/arch/arm64/boot/dts/ti/k3-am642-evm.dts > > +++ b/arch/arm64/boot/dts/ti/k3-am642-evm.dts > > @@ -139,6 +139,15 @@ > > AM64X_IOPAD(0x02a8, PIN_OUTPUT, 0) /* (E19) USB0_DRVVBUS */ > > >; > > }; > > + > > + main_spi0_pins_default: main-spi0-pins-default { > > + pinctrl-single,pins = < > > + AM64X_IOPAD(0x0210, PIN_INPUT, 0) /* (D13) SPI0_CLK */ > > + AM64X_IOPAD(0x0208, PIN_OUTPUT, 0) /* (D12) SPI0_CS0 */ > > + AM64X_IOPAD(0x0214, PIN_OUTPUT, 0) /* (A13) SPI0_D0 */ > > + AM64X_IOPAD(0x0218, PIN_INPUT, 0) /* (A14) SPI0_D1 */ > > + >; > > + }; > > }; > > > > &main_uart0 { > > @@ -245,6 +254,19 @@ > > pinctrl-0 = <&main_usb0_pins_default>; > > }; > > > > +&main_spi0 { > > + pinctrl-names = "default"; > > + pinctrl-0 = <&main_spi0_pins_default>; > > + ti,pindir-d0-out-d1-in = <1>; > > + eeprom@0 { > > + compatible = "microchip,93lc46b"; > > + reg = <0>; > > + spi-max-frequency = <1000000>; > > + spi-cs-high; > > + data-size = <16>; > > + }; > > +}; > > + > > &sdhci0 { > > /* emmc */ > > bus-width = <8>; > >
Hi Nishanth, On 09/03/21 8:24 pm, Nishanth Menon wrote: > On 20:18-20210309, Vignesh Raghavendra wrote: >> >> >> On 3/1/21 11:35 AM, Aswath Govindraju wrote: >>> Add pinmux details and device tree node for the EEPROM attached to SPI0 >>> module in main domain. >>> >>> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> >>> --- >> >> Reviewed-by: Vignesh Raghavendra <vigneshr@ti.com> >> >> Regards >> Vignesh >> >>> >>> This patch depends on, >>> https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210301055109.17626-3-a-govindraju@ti.com/ > > > Can you drop this dependency and rebase on top of my -next branch or > linux-next ? I am not able to apply the patch directly and would like to > avoid hand modifying the patch. > ok, I'll post a respin after dropping this dependency and picking up the reviewed by. Thanks, Aswath >>> >>> arch/arm64/boot/dts/ti/k3-am642-evm.dts | 22 ++++++++++++++++++++++ >>> 1 file changed, 22 insertions(+) >>> >>> diff --git a/arch/arm64/boot/dts/ti/k3-am642-evm.dts b/arch/arm64/boot/dts/ti/k3-am642-evm.dts >>> index bfd849a29655..bc5bd7f896ab 100644 >>> --- a/arch/arm64/boot/dts/ti/k3-am642-evm.dts >>> +++ b/arch/arm64/boot/dts/ti/k3-am642-evm.dts >>> @@ -139,6 +139,15 @@ >>> AM64X_IOPAD(0x02a8, PIN_OUTPUT, 0) /* (E19) USB0_DRVVBUS */ >>> >; >>> }; >>> + >>> + main_spi0_pins_default: main-spi0-pins-default { >>> + pinctrl-single,pins = < >>> + AM64X_IOPAD(0x0210, PIN_INPUT, 0) /* (D13) SPI0_CLK */ >>> + AM64X_IOPAD(0x0208, PIN_OUTPUT, 0) /* (D12) SPI0_CS0 */ >>> + AM64X_IOPAD(0x0214, PIN_OUTPUT, 0) /* (A13) SPI0_D0 */ >>> + AM64X_IOPAD(0x0218, PIN_INPUT, 0) /* (A14) SPI0_D1 */ >>> + >; >>> + }; >>> }; >>> >>> &main_uart0 { >>> @@ -245,6 +254,19 @@ >>> pinctrl-0 = <&main_usb0_pins_default>; >>> }; >>> >>> +&main_spi0 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&main_spi0_pins_default>; >>> + ti,pindir-d0-out-d1-in = <1>; >>> + eeprom@0 { >>> + compatible = "microchip,93lc46b"; >>> + reg = <0>; >>> + spi-max-frequency = <1000000>; >>> + spi-cs-high; >>> + data-size = <16>; >>> + }; >>> +}; >>> + >>> &sdhci0 { >>> /* emmc */ >>> bus-width = <8>; >>> >
diff --git a/arch/arm64/boot/dts/ti/k3-am642-evm.dts b/arch/arm64/boot/dts/ti/k3-am642-evm.dts index bfd849a29655..bc5bd7f896ab 100644 --- a/arch/arm64/boot/dts/ti/k3-am642-evm.dts +++ b/arch/arm64/boot/dts/ti/k3-am642-evm.dts @@ -139,6 +139,15 @@ AM64X_IOPAD(0x02a8, PIN_OUTPUT, 0) /* (E19) USB0_DRVVBUS */ >; }; + + main_spi0_pins_default: main-spi0-pins-default { + pinctrl-single,pins = < + AM64X_IOPAD(0x0210, PIN_INPUT, 0) /* (D13) SPI0_CLK */ + AM64X_IOPAD(0x0208, PIN_OUTPUT, 0) /* (D12) SPI0_CS0 */ + AM64X_IOPAD(0x0214, PIN_OUTPUT, 0) /* (A13) SPI0_D0 */ + AM64X_IOPAD(0x0218, PIN_INPUT, 0) /* (A14) SPI0_D1 */ + >; + }; }; &main_uart0 { @@ -245,6 +254,19 @@ pinctrl-0 = <&main_usb0_pins_default>; }; +&main_spi0 { + pinctrl-names = "default"; + pinctrl-0 = <&main_spi0_pins_default>; + ti,pindir-d0-out-d1-in = <1>; + eeprom@0 { + compatible = "microchip,93lc46b"; + reg = <0>; + spi-max-frequency = <1000000>; + spi-cs-high; + data-size = <16>; + }; +}; + &sdhci0 { /* emmc */ bus-width = <8>;
Add pinmux details and device tree node for the EEPROM attached to SPI0 module in main domain. Signed-off-by: Aswath Govindraju <a-govindraju@ti.com> --- This patch depends on, https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210301055109.17626-3-a-govindraju@ti.com/ arch/arm64/boot/dts/ti/k3-am642-evm.dts | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+)