Re: [U-Boot] [PATCH 5/6] bmips: bcm63268: add support for brcmnand
Hi Daniel, “brcm,nand-bcm6368“ is the probe and interrupt part of the driver. The core part of the driver is the one that only supports HW rev 4.0 and newer. Best regards, Álvaro. De: Daniel Schwierzeck Enviado: miércoles, agosto 14, 2019 3:23 p. m. Para: Álvaro Fernández Rojas; u-boot@lists.denx.de; philippe.rey...@softathome.com; tr...@konsulko.com Asunto: Re: [PATCH 5/6] bmips: bcm63268: add support for brcmnand Am 13.08.19 um 21:19 schrieb Álvaro Fernández Rojas: > BCM63268 uses 4.0 HW nand controller, which is currently supported by > brcmnand driver. > > Signed-off-by: Álvaro Fernández Rojas > --- > arch/mips/dts/brcm,bcm63268.dtsi | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git a/arch/mips/dts/brcm,bcm63268.dtsi > b/arch/mips/dts/brcm,bcm63268.dtsi > index f8a72ef535..5294242529 100644 > --- a/arch/mips/dts/brcm,bcm63268.dtsi > +++ b/arch/mips/dts/brcm,bcm63268.dtsi > @@ -141,6 +141,24 @@ > status = "disabled"; > }; > > + nand: nand-controller@1200 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "brcm,nand-bcm6368", if BCM63268 is already supported by brcmnand, why should it match with your new driver? > + "brcm,brcmnand-v4.0", > + "brcm,brcmnand"; > + reg-names = "nand", > + "nand-cache", > + "nand-int-base"; > + reg = <0x1200 0x180>, > + <0x1600 0x200>, > + <0x10b0 0x10>; > + clocks = <&periph_clk BCM63268_CLK_NAND>; > + clock-names = "nand"; > + > + status = "disabled"; > + }; > + > periph_pwr: power-controller@1000184c { > compatible = "brcm,bcm6328-power-domain"; > reg = <0x1000184c 0x4>; > -- - Daniel ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 5/6] bmips: bcm63268: add support for brcmnand
Am 13.08.19 um 21:19 schrieb Álvaro Fernández Rojas: > BCM63268 uses 4.0 HW nand controller, which is currently supported by > brcmnand driver. > > Signed-off-by: Álvaro Fernández Rojas > --- > arch/mips/dts/brcm,bcm63268.dtsi | 18 ++ > 1 file changed, 18 insertions(+) > > diff --git a/arch/mips/dts/brcm,bcm63268.dtsi > b/arch/mips/dts/brcm,bcm63268.dtsi > index f8a72ef535..5294242529 100644 > --- a/arch/mips/dts/brcm,bcm63268.dtsi > +++ b/arch/mips/dts/brcm,bcm63268.dtsi > @@ -141,6 +141,24 @@ > status = "disabled"; > }; > > + nand: nand-controller@1200 { > + #address-cells = <1>; > + #size-cells = <0>; > + compatible = "brcm,nand-bcm6368", if BCM63268 is already supported by brcmnand, why should it match with your new driver? > + "brcm,brcmnand-v4.0", > + "brcm,brcmnand"; > + reg-names = "nand", > + "nand-cache", > + "nand-int-base"; > + reg = <0x1200 0x180>, > + <0x1600 0x200>, > + <0x10b0 0x10>; > + clocks = <&periph_clk BCM63268_CLK_NAND>; > + clock-names = "nand"; > + > + status = "disabled"; > + }; > + > periph_pwr: power-controller@1000184c { > compatible = "brcm,bcm6328-power-domain"; > reg = <0x1000184c 0x4>; > -- - Daniel ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH 5/6] bmips: bcm63268: add support for brcmnand
BCM63268 uses 4.0 HW nand controller, which is currently supported by brcmnand driver. Signed-off-by: Álvaro Fernández Rojas --- arch/mips/dts/brcm,bcm63268.dtsi | 18 ++ 1 file changed, 18 insertions(+) diff --git a/arch/mips/dts/brcm,bcm63268.dtsi b/arch/mips/dts/brcm,bcm63268.dtsi index f8a72ef535..5294242529 100644 --- a/arch/mips/dts/brcm,bcm63268.dtsi +++ b/arch/mips/dts/brcm,bcm63268.dtsi @@ -141,6 +141,24 @@ status = "disabled"; }; + nand: nand-controller@1200 { + #address-cells = <1>; + #size-cells = <0>; + compatible = "brcm,nand-bcm6368", +"brcm,brcmnand-v4.0", +"brcm,brcmnand"; + reg-names = "nand", + "nand-cache", + "nand-int-base"; + reg = <0x1200 0x180>, + <0x1600 0x200>, + <0x10b0 0x10>; + clocks = <&periph_clk BCM63268_CLK_NAND>; + clock-names = "nand"; + + status = "disabled"; + }; + periph_pwr: power-controller@1000184c { compatible = "brcm,bcm6328-power-domain"; reg = <0x1000184c 0x4>; -- 2.20.1 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot