Message ID | 20190507182152.4254-3-jagan@amarulasolutions.com |
---|---|
State | New |
Headers | show |
Series |
|
Related | show |
On 05/08/2019 02:21 AM, Jagan Teki wrote: > Devicetree files in RK3399 platform is synced from Linux, like other > platforms does. Apart from these u-boot in rk3399 would also require > some u-boot specific node like dmc. > > dmc node has big chunk of DDR timing parameters which are specific > to specific board, and maintained with rk3399-sdram*.dtsi. > > So, create board specific -u-boot.dtsi files and move these sdram dtsi > files accordingly. This would help of maintain u-boot specific changes > separately without touching Linux dts(i) files which indeed easy for > syncing from Linux between releases. > > These board specific -u-boot.dtsi can be extendible to add more u-boot > specific nodes or properties in future. > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> Reviewed-by: Kever Yang <kever.yang@rock-chips.com> Thanks, - Kever > --- > arch/arm/dts/rk3399-evb-u-boot.dtsi | 6 ++++++ > arch/arm/dts/rk3399-evb.dts | 1 - > arch/arm/dts/rk3399-ficus-u-boot.dtsi | 6 ++++++ > arch/arm/dts/rk3399-ficus.dts | 1 - > arch/arm/dts/rk3399-firefly-u-boot.dtsi | 6 ++++++ > arch/arm/dts/rk3399-firefly.dts | 1 - > arch/arm/dts/rk3399-gru-bob-u-boot.dtsi | 6 ++++++ > arch/arm/dts/rk3399-gru-bob.dts | 1 - > arch/arm/dts/rk3399-rock960-u-boot.dtsi | 6 ++++++ > arch/arm/dts/rk3399-rock960.dts | 1 - > 10 files changed, 30 insertions(+), 5 deletions(-) > create mode 100644 arch/arm/dts/rk3399-evb-u-boot.dtsi > create mode 100644 arch/arm/dts/rk3399-ficus-u-boot.dtsi > create mode 100644 arch/arm/dts/rk3399-firefly-u-boot.dtsi > create mode 100644 arch/arm/dts/rk3399-gru-bob-u-boot.dtsi > create mode 100644 arch/arm/dts/rk3399-rock960-u-boot.dtsi > > diff --git a/arch/arm/dts/rk3399-evb-u-boot.dtsi b/arch/arm/dts/rk3399-evb-u-boot.dtsi > new file mode 100644 > index 0000000000..7e2c57af22 > --- /dev/null > +++ b/arch/arm/dts/rk3399-evb-u-boot.dtsi > @@ -0,0 +1,6 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> > + */ > + > +#include "rk3399-sdram-lpddr3-4GB-1600.dtsi" > diff --git a/arch/arm/dts/rk3399-evb.dts b/arch/arm/dts/rk3399-evb.dts > index 9162f3dd50..a506e8da37 100644 > --- a/arch/arm/dts/rk3399-evb.dts > +++ b/arch/arm/dts/rk3399-evb.dts > @@ -7,7 +7,6 @@ > #include <dt-bindings/pwm/pwm.h> > #include <dt-bindings/pinctrl/rockchip.h> > #include "rk3399.dtsi" > -#include "rk3399-sdram-lpddr3-4GB-1600.dtsi" > > / { > model = "Rockchip RK3399 Evaluation Board"; > diff --git a/arch/arm/dts/rk3399-ficus-u-boot.dtsi b/arch/arm/dts/rk3399-ficus-u-boot.dtsi > new file mode 100644 > index 0000000000..eab86bdb30 > --- /dev/null > +++ b/arch/arm/dts/rk3399-ficus-u-boot.dtsi > @@ -0,0 +1,6 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> > + */ > + > +#include "rk3399-sdram-ddr3-1600.dtsi" > diff --git a/arch/arm/dts/rk3399-ficus.dts b/arch/arm/dts/rk3399-ficus.dts > index 4af0e4e383..4b2dd82b67 100644 > --- a/arch/arm/dts/rk3399-ficus.dts > +++ b/arch/arm/dts/rk3399-ficus.dts > @@ -8,7 +8,6 @@ > > /dts-v1/; > #include "rk3399-rock960.dtsi" > -#include "rk3399-sdram-ddr3-1600.dtsi" > > / { > model = "96boards RK3399 Ficus"; > diff --git a/arch/arm/dts/rk3399-firefly-u-boot.dtsi b/arch/arm/dts/rk3399-firefly-u-boot.dtsi > new file mode 100644 > index 0000000000..eab86bdb30 > --- /dev/null > +++ b/arch/arm/dts/rk3399-firefly-u-boot.dtsi > @@ -0,0 +1,6 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> > + */ > + > +#include "rk3399-sdram-ddr3-1600.dtsi" > diff --git a/arch/arm/dts/rk3399-firefly.dts b/arch/arm/dts/rk3399-firefly.dts > index 46f2ffaf8d..a4cb64f8bd 100644 > --- a/arch/arm/dts/rk3399-firefly.dts > +++ b/arch/arm/dts/rk3399-firefly.dts > @@ -7,7 +7,6 @@ > #include <dt-bindings/pwm/pwm.h> > #include <dt-bindings/pinctrl/rockchip.h> > #include "rk3399.dtsi" > -#include "rk3399-sdram-ddr3-1600.dtsi" > > / { > model = "Firefly-RK3399 Board"; > diff --git a/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi b/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi > new file mode 100644 > index 0000000000..9edb8cf841 > --- /dev/null > +++ b/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi > @@ -0,0 +1,6 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> > + */ > + > +#include "rk3399-sdram-lpddr3-samsung-4GB-1866.dtsi" > diff --git a/arch/arm/dts/rk3399-gru-bob.dts b/arch/arm/dts/rk3399-gru-bob.dts > index 0e3d91fc28..1ee0dc0d9f 100644 > --- a/arch/arm/dts/rk3399-gru-bob.dts > +++ b/arch/arm/dts/rk3399-gru-bob.dts > @@ -7,7 +7,6 @@ > > /dts-v1/; > #include "rk3399-gru-chromebook.dtsi" > -#include "rk3399-sdram-lpddr3-samsung-4GB-1866.dtsi" > > / { > model = "Google Bob"; > diff --git a/arch/arm/dts/rk3399-rock960-u-boot.dtsi b/arch/arm/dts/rk3399-rock960-u-boot.dtsi > new file mode 100644 > index 0000000000..5256f6d3f2 > --- /dev/null > +++ b/arch/arm/dts/rk3399-rock960-u-boot.dtsi > @@ -0,0 +1,6 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> > + */ > + > +#include "rk3399-sdram-lpddr3-2GB-1600.dtsi" > diff --git a/arch/arm/dts/rk3399-rock960.dts b/arch/arm/dts/rk3399-rock960.dts > index 25c58b4261..7e06bc97e5 100644 > --- a/arch/arm/dts/rk3399-rock960.dts > +++ b/arch/arm/dts/rk3399-rock960.dts > @@ -5,7 +5,6 @@ > > /dts-v1/; > #include "rk3399-rock960.dtsi" > -#include "rk3399-sdram-lpddr3-2GB-1600.dtsi" > > / { > model = "96boards Rock960";
On 05/08/2019 11:39 AM, Kever Yang wrote: > > On 05/08/2019 02:21 AM, Jagan Teki wrote: >> Devicetree files in RK3399 platform is synced from Linux, like other >> platforms does. Apart from these u-boot in rk3399 would also require >> some u-boot specific node like dmc. >> >> dmc node has big chunk of DDR timing parameters which are specific >> to specific board, and maintained with rk3399-sdram*.dtsi. >> >> So, create board specific -u-boot.dtsi files and move these sdram dtsi >> files accordingly. This would help of maintain u-boot specific changes >> separately without touching Linux dts(i) files which indeed easy for >> syncing from Linux between releases. >> >> These board specific -u-boot.dtsi can be extendible to add more u-boot >> specific nodes or properties in future. >> >> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > Reviewed-by: Kever Yang <kever.yang@rock-chips.com> Applied to u-boot-rockchip, thanks! > > Thanks, > - Kever >> --- >> arch/arm/dts/rk3399-evb-u-boot.dtsi | 6 ++++++ >> arch/arm/dts/rk3399-evb.dts | 1 - >> arch/arm/dts/rk3399-ficus-u-boot.dtsi | 6 ++++++ >> arch/arm/dts/rk3399-ficus.dts | 1 - >> arch/arm/dts/rk3399-firefly-u-boot.dtsi | 6 ++++++ >> arch/arm/dts/rk3399-firefly.dts | 1 - >> arch/arm/dts/rk3399-gru-bob-u-boot.dtsi | 6 ++++++ >> arch/arm/dts/rk3399-gru-bob.dts | 1 - >> arch/arm/dts/rk3399-rock960-u-boot.dtsi | 6 ++++++ >> arch/arm/dts/rk3399-rock960.dts | 1 - >> 10 files changed, 30 insertions(+), 5 deletions(-) >> create mode 100644 arch/arm/dts/rk3399-evb-u-boot.dtsi >> create mode 100644 arch/arm/dts/rk3399-ficus-u-boot.dtsi >> create mode 100644 arch/arm/dts/rk3399-firefly-u-boot.dtsi >> create mode 100644 arch/arm/dts/rk3399-gru-bob-u-boot.dtsi >> create mode 100644 arch/arm/dts/rk3399-rock960-u-boot.dtsi >> >> diff --git a/arch/arm/dts/rk3399-evb-u-boot.dtsi b/arch/arm/dts/rk3399-evb-u-boot.dtsi >> new file mode 100644 >> index 0000000000..7e2c57af22 >> --- /dev/null >> +++ b/arch/arm/dts/rk3399-evb-u-boot.dtsi >> @@ -0,0 +1,6 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> >> + */ >> + >> +#include "rk3399-sdram-lpddr3-4GB-1600.dtsi" >> diff --git a/arch/arm/dts/rk3399-evb.dts b/arch/arm/dts/rk3399-evb.dts >> index 9162f3dd50..a506e8da37 100644 >> --- a/arch/arm/dts/rk3399-evb.dts >> +++ b/arch/arm/dts/rk3399-evb.dts >> @@ -7,7 +7,6 @@ >> #include <dt-bindings/pwm/pwm.h> >> #include <dt-bindings/pinctrl/rockchip.h> >> #include "rk3399.dtsi" >> -#include "rk3399-sdram-lpddr3-4GB-1600.dtsi" >> >> / { >> model = "Rockchip RK3399 Evaluation Board"; >> diff --git a/arch/arm/dts/rk3399-ficus-u-boot.dtsi b/arch/arm/dts/rk3399-ficus-u-boot.dtsi >> new file mode 100644 >> index 0000000000..eab86bdb30 >> --- /dev/null >> +++ b/arch/arm/dts/rk3399-ficus-u-boot.dtsi >> @@ -0,0 +1,6 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> >> + */ >> + >> +#include "rk3399-sdram-ddr3-1600.dtsi" >> diff --git a/arch/arm/dts/rk3399-ficus.dts b/arch/arm/dts/rk3399-ficus.dts >> index 4af0e4e383..4b2dd82b67 100644 >> --- a/arch/arm/dts/rk3399-ficus.dts >> +++ b/arch/arm/dts/rk3399-ficus.dts >> @@ -8,7 +8,6 @@ >> >> /dts-v1/; >> #include "rk3399-rock960.dtsi" >> -#include "rk3399-sdram-ddr3-1600.dtsi" >> >> / { >> model = "96boards RK3399 Ficus"; >> diff --git a/arch/arm/dts/rk3399-firefly-u-boot.dtsi b/arch/arm/dts/rk3399-firefly-u-boot.dtsi >> new file mode 100644 >> index 0000000000..eab86bdb30 >> --- /dev/null >> +++ b/arch/arm/dts/rk3399-firefly-u-boot.dtsi >> @@ -0,0 +1,6 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> >> + */ >> + >> +#include "rk3399-sdram-ddr3-1600.dtsi" >> diff --git a/arch/arm/dts/rk3399-firefly.dts b/arch/arm/dts/rk3399-firefly.dts >> index 46f2ffaf8d..a4cb64f8bd 100644 >> --- a/arch/arm/dts/rk3399-firefly.dts >> +++ b/arch/arm/dts/rk3399-firefly.dts >> @@ -7,7 +7,6 @@ >> #include <dt-bindings/pwm/pwm.h> >> #include <dt-bindings/pinctrl/rockchip.h> >> #include "rk3399.dtsi" >> -#include "rk3399-sdram-ddr3-1600.dtsi" >> >> / { >> model = "Firefly-RK3399 Board"; >> diff --git a/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi b/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi >> new file mode 100644 >> index 0000000000..9edb8cf841 >> --- /dev/null >> +++ b/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi >> @@ -0,0 +1,6 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> >> + */ >> + >> +#include "rk3399-sdram-lpddr3-samsung-4GB-1866.dtsi" >> diff --git a/arch/arm/dts/rk3399-gru-bob.dts b/arch/arm/dts/rk3399-gru-bob.dts >> index 0e3d91fc28..1ee0dc0d9f 100644 >> --- a/arch/arm/dts/rk3399-gru-bob.dts >> +++ b/arch/arm/dts/rk3399-gru-bob.dts >> @@ -7,7 +7,6 @@ >> >> /dts-v1/; >> #include "rk3399-gru-chromebook.dtsi" >> -#include "rk3399-sdram-lpddr3-samsung-4GB-1866.dtsi" >> >> / { >> model = "Google Bob"; >> diff --git a/arch/arm/dts/rk3399-rock960-u-boot.dtsi b/arch/arm/dts/rk3399-rock960-u-boot.dtsi >> new file mode 100644 >> index 0000000000..5256f6d3f2 >> --- /dev/null >> +++ b/arch/arm/dts/rk3399-rock960-u-boot.dtsi >> @@ -0,0 +1,6 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> >> + */ >> + >> +#include "rk3399-sdram-lpddr3-2GB-1600.dtsi" >> diff --git a/arch/arm/dts/rk3399-rock960.dts b/arch/arm/dts/rk3399-rock960.dts >> index 25c58b4261..7e06bc97e5 100644 >> --- a/arch/arm/dts/rk3399-rock960.dts >> +++ b/arch/arm/dts/rk3399-rock960.dts >> @@ -5,7 +5,6 @@ >> >> /dts-v1/; >> #include "rk3399-rock960.dtsi" >> -#include "rk3399-sdram-lpddr3-2GB-1600.dtsi" >> >> / { >> model = "96boards Rock960"; > > > > _______________________________________________ > Linux-rockchip mailing list > Linux-rockchip@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-rockchip >
diff --git a/arch/arm/dts/rk3399-evb-u-boot.dtsi b/arch/arm/dts/rk3399-evb-u-boot.dtsi new file mode 100644 index 0000000000..7e2c57af22 --- /dev/null +++ b/arch/arm/dts/rk3399-evb-u-boot.dtsi @@ -0,0 +1,6 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> + */ + +#include "rk3399-sdram-lpddr3-4GB-1600.dtsi" diff --git a/arch/arm/dts/rk3399-evb.dts b/arch/arm/dts/rk3399-evb.dts index 9162f3dd50..a506e8da37 100644 --- a/arch/arm/dts/rk3399-evb.dts +++ b/arch/arm/dts/rk3399-evb.dts @@ -7,7 +7,6 @@ #include <dt-bindings/pwm/pwm.h> #include <dt-bindings/pinctrl/rockchip.h> #include "rk3399.dtsi" -#include "rk3399-sdram-lpddr3-4GB-1600.dtsi" / { model = "Rockchip RK3399 Evaluation Board"; diff --git a/arch/arm/dts/rk3399-ficus-u-boot.dtsi b/arch/arm/dts/rk3399-ficus-u-boot.dtsi new file mode 100644 index 0000000000..eab86bdb30 --- /dev/null +++ b/arch/arm/dts/rk3399-ficus-u-boot.dtsi @@ -0,0 +1,6 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> + */ + +#include "rk3399-sdram-ddr3-1600.dtsi" diff --git a/arch/arm/dts/rk3399-ficus.dts b/arch/arm/dts/rk3399-ficus.dts index 4af0e4e383..4b2dd82b67 100644 --- a/arch/arm/dts/rk3399-ficus.dts +++ b/arch/arm/dts/rk3399-ficus.dts @@ -8,7 +8,6 @@ /dts-v1/; #include "rk3399-rock960.dtsi" -#include "rk3399-sdram-ddr3-1600.dtsi" / { model = "96boards RK3399 Ficus"; diff --git a/arch/arm/dts/rk3399-firefly-u-boot.dtsi b/arch/arm/dts/rk3399-firefly-u-boot.dtsi new file mode 100644 index 0000000000..eab86bdb30 --- /dev/null +++ b/arch/arm/dts/rk3399-firefly-u-boot.dtsi @@ -0,0 +1,6 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> + */ + +#include "rk3399-sdram-ddr3-1600.dtsi" diff --git a/arch/arm/dts/rk3399-firefly.dts b/arch/arm/dts/rk3399-firefly.dts index 46f2ffaf8d..a4cb64f8bd 100644 --- a/arch/arm/dts/rk3399-firefly.dts +++ b/arch/arm/dts/rk3399-firefly.dts @@ -7,7 +7,6 @@ #include <dt-bindings/pwm/pwm.h> #include <dt-bindings/pinctrl/rockchip.h> #include "rk3399.dtsi" -#include "rk3399-sdram-ddr3-1600.dtsi" / { model = "Firefly-RK3399 Board"; diff --git a/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi b/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi new file mode 100644 index 0000000000..9edb8cf841 --- /dev/null +++ b/arch/arm/dts/rk3399-gru-bob-u-boot.dtsi @@ -0,0 +1,6 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> + */ + +#include "rk3399-sdram-lpddr3-samsung-4GB-1866.dtsi" diff --git a/arch/arm/dts/rk3399-gru-bob.dts b/arch/arm/dts/rk3399-gru-bob.dts index 0e3d91fc28..1ee0dc0d9f 100644 --- a/arch/arm/dts/rk3399-gru-bob.dts +++ b/arch/arm/dts/rk3399-gru-bob.dts @@ -7,7 +7,6 @@ /dts-v1/; #include "rk3399-gru-chromebook.dtsi" -#include "rk3399-sdram-lpddr3-samsung-4GB-1866.dtsi" / { model = "Google Bob"; diff --git a/arch/arm/dts/rk3399-rock960-u-boot.dtsi b/arch/arm/dts/rk3399-rock960-u-boot.dtsi new file mode 100644 index 0000000000..5256f6d3f2 --- /dev/null +++ b/arch/arm/dts/rk3399-rock960-u-boot.dtsi @@ -0,0 +1,6 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright (C) 2019 Jagan Teki <jagan@amarulasolutions.com> + */ + +#include "rk3399-sdram-lpddr3-2GB-1600.dtsi" diff --git a/arch/arm/dts/rk3399-rock960.dts b/arch/arm/dts/rk3399-rock960.dts index 25c58b4261..7e06bc97e5 100644 --- a/arch/arm/dts/rk3399-rock960.dts +++ b/arch/arm/dts/rk3399-rock960.dts @@ -5,7 +5,6 @@ /dts-v1/; #include "rk3399-rock960.dtsi" -#include "rk3399-sdram-lpddr3-2GB-1600.dtsi" / { model = "96boards Rock960";
Devicetree files in RK3399 platform is synced from Linux, like other platforms does. Apart from these u-boot in rk3399 would also require some u-boot specific node like dmc. dmc node has big chunk of DDR timing parameters which are specific to specific board, and maintained with rk3399-sdram*.dtsi. So, create board specific -u-boot.dtsi files and move these sdram dtsi files accordingly. This would help of maintain u-boot specific changes separately without touching Linux dts(i) files which indeed easy for syncing from Linux between releases. These board specific -u-boot.dtsi can be extendible to add more u-boot specific nodes or properties in future. Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> --- arch/arm/dts/rk3399-evb-u-boot.dtsi | 6 ++++++ arch/arm/dts/rk3399-evb.dts | 1 - arch/arm/dts/rk3399-ficus-u-boot.dtsi | 6 ++++++ arch/arm/dts/rk3399-ficus.dts | 1 - arch/arm/dts/rk3399-firefly-u-boot.dtsi | 6 ++++++ arch/arm/dts/rk3399-firefly.dts | 1 - arch/arm/dts/rk3399-gru-bob-u-boot.dtsi | 6 ++++++ arch/arm/dts/rk3399-gru-bob.dts | 1 - arch/arm/dts/rk3399-rock960-u-boot.dtsi | 6 ++++++ arch/arm/dts/rk3399-rock960.dts | 1 - 10 files changed, 30 insertions(+), 5 deletions(-) create mode 100644 arch/arm/dts/rk3399-evb-u-boot.dtsi create mode 100644 arch/arm/dts/rk3399-ficus-u-boot.dtsi create mode 100644 arch/arm/dts/rk3399-firefly-u-boot.dtsi create mode 100644 arch/arm/dts/rk3399-gru-bob-u-boot.dtsi create mode 100644 arch/arm/dts/rk3399-rock960-u-boot.dtsi