On 8/13/20 9:36 AM, Patrice CHOTARD wrote:
> Hi Christophe
>
> On 7/31/20 9:53 AM, Christophe Kerello wrote:
>> This patch adds FMC2 External Bus Interface support on stm32mp157c.
>>
>> Signed-off-by: Christophe Kerello <[email protected]>
>> ---
>>
>> arch/arm/dts/stm32mp151.dtsi | 43
>> +++++++++++++++++++++++++++-------------
>> arch/arm/dts/stm32mp157c-ev1.dts | 16 ++++++++-------
>> 2 files changed, 38 insertions(+), 21 deletions(-)
Applied on u-boot-stm/master
Thanks
>>
>> diff --git a/arch/arm/dts/stm32mp151.dtsi b/arch/arm/dts/stm32mp151.dtsi
>> index 0d97f56..39d9e54 100644
>> --- a/arch/arm/dts/stm32mp151.dtsi
>> +++ b/arch/arm/dts/stm32mp151.dtsi
>> @@ -1328,23 +1328,38 @@
>> dma-requests = <48>;
>> };
>>
>> - fmc: nand-controller@58002000 {
>> - compatible = "st,stm32mp15-fmc2";
>> - reg = <0x58002000 0x1000>,
>> - <0x80000000 0x1000>,
>> - <0x88010000 0x1000>,
>> - <0x88020000 0x1000>,
>> - <0x81000000 0x1000>,
>> - <0x89010000 0x1000>,
>> - <0x89020000 0x1000>;
>> - interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>;
>> - dmas = <&mdma1 20 0x10 0x12000a02 0x0 0x0>,
>> - <&mdma1 20 0x10 0x12000a08 0x0 0x0>,
>> - <&mdma1 21 0x10 0x12000a0a 0x0 0x0>;
>> - dma-names = "tx", "rx", "ecc";
>> + fmc: memory-controller@58002000 {
>> + #address-cells = <2>;
>> + #size-cells = <1>;
>> + compatible = "st,stm32mp1-fmc2-ebi";
>> + reg = <0x58002000 0x1000>;
>> clocks = <&rcc FMC_K>;
>> resets = <&rcc FMC_R>;
>> status = "disabled";
>> +
>> + ranges = <0 0 0x60000000 0x04000000>, /* EBI CS 1 */
>> + <1 0 0x64000000 0x04000000>, /* EBI CS 2 */
>> + <2 0 0x68000000 0x04000000>, /* EBI CS 3 */
>> + <3 0 0x6c000000 0x04000000>, /* EBI CS 4 */
>> + <4 0 0x80000000 0x10000000>; /* NAND */
>> +
>> + nand-controller@4,0 {
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + compatible = "st,stm32mp1-fmc2-nfc";
>> + reg = <4 0x00000000 0x1000>,
>> + <4 0x08010000 0x1000>,
>> + <4 0x08020000 0x1000>,
>> + <4 0x01000000 0x1000>,
>> + <4 0x09010000 0x1000>,
>> + <4 0x09020000 0x1000>;
>> + interrupts = <GIC_SPI 48 IRQ_TYPE_LEVEL_HIGH>;
>> + dmas = <&mdma1 20 0x2 0x12000a02 0x0 0x0>,
>> + <&mdma1 20 0x2 0x12000a08 0x0 0x0>,
>> + <&mdma1 21 0x2 0x12000a0a 0x0 0x0>;
>> + dma-names = "tx", "rx", "ecc";
>> + status = "disabled";
>> + };
>> };
>>
>> qspi: spi@58003000 {
>> diff --git a/arch/arm/dts/stm32mp157c-ev1.dts
>> b/arch/arm/dts/stm32mp157c-ev1.dts
>> index b190565..46f81ce 100644
>> --- a/arch/arm/dts/stm32mp157c-ev1.dts
>> +++ b/arch/arm/dts/stm32mp157c-ev1.dts
>> @@ -157,14 +157,16 @@
>> pinctrl-0 = <&fmc_pins_a>;
>> pinctrl-1 = <&fmc_sleep_pins_a>;
>> status = "okay";
>> - #address-cells = <1>;
>> - #size-cells = <0>;
>>
>> - nand@0 {
>> - reg = <0>;
>> - nand-on-flash-bbt;
>> - #address-cells = <1>;
>> - #size-cells = <1>;
>> + nand-controller@4,0 {
>> + status = "okay";
>> +
>> + nand@0 {
>> + reg = <0>;
>> + nand-on-flash-bbt;
>> + #address-cells = <1>;
>> + #size-cells = <1>;
>> + };
>> };
>> };
>>
> Reviewed-by: Patrice Chotard <[email protected]>
>
> Thanks
>
> Patrice
>
