* [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata @ 2017-02-15 3:06 Jacob Chen 2017-02-15 6:13 ` Kever Yang 0 siblings, 1 reply; 7+ messages in thread From: Jacob Chen @ 2017-02-15 3:06 UTC (permalink / raw) To: u-boot We should remove config_spl_of_platdata to build u-boot-spl-dtb.bin rather than u-boot-spl-nodtb.bin since we use spl_back_to_brom. I miss it because i forget to clean build-dir.. Signed-off-by: Jacob Chen <jacob-chen@iotwrt.com> --- configs/firefly-rk3288_defconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/configs/firefly-rk3288_defconfig b/configs/firefly-rk3288_defconfig index 18e050d..e5222da 100644 --- a/configs/firefly-rk3288_defconfig +++ b/configs/firefly-rk3288_defconfig @@ -32,7 +32,6 @@ CONFIG_CMD_FAT=y CONFIG_CMD_FS_GENERIC=y CONFIG_SPL_OF_CONTROL=y CONFIG_OF_SPL_REMOVE_PROPS="pinctrl-0 pinctrl-names clock-names interrupt-parent assigned-clocks assigned-clock-rates assigned-clock-parents" -CONFIG_SPL_OF_PLATDATA=y CONFIG_REGMAP=y CONFIG_SPL_REGMAP=y CONFIG_SYSCON=y -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata 2017-02-15 3:06 [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata Jacob Chen @ 2017-02-15 6:13 ` Kever Yang 2017-02-15 6:41 ` Jacob Chen 0 siblings, 1 reply; 7+ messages in thread From: Kever Yang @ 2017-02-15 6:13 UTC (permalink / raw) To: u-boot Hi Jacob, On 02/15/2017 11:06 AM, Jacob Chen wrote: > We should remove config_spl_of_platdata to build u-boot-spl-dtb.bin rather than u-boot-spl-nodtb.bin > since we use spl_back_to_brom. Have you try with CONFIG_SPL_OF_PLATDATA on and without SPL_BACK_TO_BROM? If this works on firefly, then we can still leave it as an example to show how both way works. Thanks, - Kever > > I miss it because i forget to clean build-dir.. > > Signed-off-by: Jacob Chen <jacob-chen@iotwrt.com> > --- > > configs/firefly-rk3288_defconfig | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/configs/firefly-rk3288_defconfig b/configs/firefly-rk3288_defconfig > index 18e050d..e5222da 100644 > --- a/configs/firefly-rk3288_defconfig > +++ b/configs/firefly-rk3288_defconfig > @@ -32,7 +32,6 @@ CONFIG_CMD_FAT=y > CONFIG_CMD_FS_GENERIC=y > CONFIG_SPL_OF_CONTROL=y > CONFIG_OF_SPL_REMOVE_PROPS="pinctrl-0 pinctrl-names clock-names interrupt-parent assigned-clocks assigned-clock-rates assigned-clock-parents" > -CONFIG_SPL_OF_PLATDATA=y > CONFIG_REGMAP=y > CONFIG_SPL_REGMAP=y > CONFIG_SYSCON=y ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata 2017-02-15 6:13 ` Kever Yang @ 2017-02-15 6:41 ` Jacob Chen 2017-02-16 20:44 ` Simon Glass 0 siblings, 1 reply; 7+ messages in thread From: Jacob Chen @ 2017-02-15 6:41 UTC (permalink / raw) To: u-boot Hi Kever, Kever Yang wrote on 2017?02?15? 14:13: > Hi Jacob, > > On 02/15/2017 11:06 AM, Jacob Chen wrote: >> We should remove config_spl_of_platdata to build u-boot-spl-dtb.bin >> rather than u-boot-spl-nodtb.bin >> since we use spl_back_to_brom. > > Have you try with CONFIG_SPL_OF_PLATDATA on and without SPL_BACK_TO_BROM? > > If this works on firefly, then we can still leave it as an example to > show how both way works. > Firefly had been using CONFIG_SPL_OF_PLATDATA and it works well. Just it will make trouble for other things to generate the u-boot blob, like yocto, sdk build scripts, guides... At first, I also want to keep CONFIG_SPL_OF_PLATDATA in firefly, but later I find that i have to do a lot of unnecessary work to deal with it, so now I would like one chip use one way to generate the u-boot blob. > Thanks, > - Kever >> >> I miss it because i forget to clean build-dir.. >> >> Signed-off-by: Jacob Chen <jacob-chen@iotwrt.com> >> --- >> >> configs/firefly-rk3288_defconfig | 1 - >> 1 file changed, 1 deletion(-) >> >> diff --git a/configs/firefly-rk3288_defconfig >> b/configs/firefly-rk3288_defconfig >> index 18e050d..e5222da 100644 >> --- a/configs/firefly-rk3288_defconfig >> +++ b/configs/firefly-rk3288_defconfig >> @@ -32,7 +32,6 @@ CONFIG_CMD_FAT=y >> CONFIG_CMD_FS_GENERIC=y >> CONFIG_SPL_OF_CONTROL=y >> CONFIG_OF_SPL_REMOVE_PROPS="pinctrl-0 pinctrl-names clock-names >> interrupt-parent assigned-clocks assigned-clock-rates >> assigned-clock-parents" >> -CONFIG_SPL_OF_PLATDATA=y >> CONFIG_REGMAP=y >> CONFIG_SPL_REGMAP=y >> CONFIG_SYSCON=y > > > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata 2017-02-15 6:41 ` Jacob Chen @ 2017-02-16 20:44 ` Simon Glass 2017-02-20 10:28 ` Jacob Chen 0 siblings, 1 reply; 7+ messages in thread From: Simon Glass @ 2017-02-16 20:44 UTC (permalink / raw) To: u-boot Hi, On 14 February 2017 at 23:41, Jacob Chen <jacob2.chen@rock-chips.com> wrote: > Hi Kever, > > > Kever Yang wrote on 2017?02?15? 14:13: >> >> Hi Jacob, >> >> On 02/15/2017 11:06 AM, Jacob Chen wrote: >>> >>> We should remove config_spl_of_platdata to build u-boot-spl-dtb.bin >>> rather than u-boot-spl-nodtb.bin >>> since we use spl_back_to_brom. >> >> >> Have you try with CONFIG_SPL_OF_PLATDATA on and without SPL_BACK_TO_BROM? >> >> If this works on firefly, then we can still leave it as an example to show >> how both way works. >> > > Firefly had been using CONFIG_SPL_OF_PLATDATA and it works well. > Just it will make trouble for other things to generate the u-boot blob, > like yocto, sdk build scripts, guides... > > At first, I also want to keep CONFIG_SPL_OF_PLATDATA in firefly, but later > I find that i have to do a lot of unnecessary work to deal with it, > so now I would like one chip use one way to generate the u-boot blob. How about having a separate board config like firefly-rk3288-dt so that we keep the non-of-platdata alive? It only needs a separate defconfig, which is pretty esay. Regards, Simon ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata 2017-02-16 20:44 ` Simon Glass @ 2017-02-20 10:28 ` Jacob Chen 2017-02-23 3:35 ` Simon Glass 0 siblings, 1 reply; 7+ messages in thread From: Jacob Chen @ 2017-02-20 10:28 UTC (permalink / raw) To: u-boot Hi, 2017-02-17 4:44 GMT+08:00 Simon Glass <sjg@chromium.org>: > Hi, > > On 14 February 2017 at 23:41, Jacob Chen <jacob2.chen@rock-chips.com> wrote: >> Hi Kever, >> >> >> Kever Yang wrote on 2017?02?15? 14:13: >>> >>> Hi Jacob, >>> >>> On 02/15/2017 11:06 AM, Jacob Chen wrote: >>>> >>>> We should remove config_spl_of_platdata to build u-boot-spl-dtb.bin >>>> rather than u-boot-spl-nodtb.bin >>>> since we use spl_back_to_brom. >>> >>> >>> Have you try with CONFIG_SPL_OF_PLATDATA on and without SPL_BACK_TO_BROM? >>> >>> If this works on firefly, then we can still leave it as an example to show >>> how both way works. >>> >> >> Firefly had been using CONFIG_SPL_OF_PLATDATA and it works well. >> Just it will make trouble for other things to generate the u-boot blob, >> like yocto, sdk build scripts, guides... >> >> At first, I also want to keep CONFIG_SPL_OF_PLATDATA in firefly, but later >> I find that i have to do a lot of unnecessary work to deal with it, >> so now I would like one chip use one way to generate the u-boot blob. > > How about having a separate board config like firefly-rk3288-dt so > that we keep the non-of-platdata alive? It only needs a separate > defconfig, which is pretty esay. > > Regards, > Simon > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot I send a patch "rockchip: firefly: configs: add a new config to keep non-of-platdata" to save old config. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata 2017-02-20 10:28 ` Jacob Chen @ 2017-02-23 3:35 ` Simon Glass 2017-02-23 16:19 ` Simon Glass 0 siblings, 1 reply; 7+ messages in thread From: Simon Glass @ 2017-02-23 3:35 UTC (permalink / raw) To: u-boot On 20 February 2017 at 03:28, Jacob Chen <jacobchen110@gmail.com> wrote: > Hi, > > 2017-02-17 4:44 GMT+08:00 Simon Glass <sjg@chromium.org>: >> Hi, >> >> On 14 February 2017 at 23:41, Jacob Chen <jacob2.chen@rock-chips.com> wrote: >>> Hi Kever, >>> >>> >>> Kever Yang wrote on 2017?02?15? 14:13: >>>> >>>> Hi Jacob, >>>> >>>> On 02/15/2017 11:06 AM, Jacob Chen wrote: >>>>> >>>>> We should remove config_spl_of_platdata to build u-boot-spl-dtb.bin >>>>> rather than u-boot-spl-nodtb.bin >>>>> since we use spl_back_to_brom. >>>> >>>> >>>> Have you try with CONFIG_SPL_OF_PLATDATA on and without SPL_BACK_TO_BROM? >>>> >>>> If this works on firefly, then we can still leave it as an example to show >>>> how both way works. >>>> >>> >>> Firefly had been using CONFIG_SPL_OF_PLATDATA and it works well. >>> Just it will make trouble for other things to generate the u-boot blob, >>> like yocto, sdk build scripts, guides... >>> >>> At first, I also want to keep CONFIG_SPL_OF_PLATDATA in firefly, but later >>> I find that i have to do a lot of unnecessary work to deal with it, >>> so now I would like one chip use one way to generate the u-boot blob. >> >> How about having a separate board config like firefly-rk3288-dt so >> that we keep the non-of-platdata alive? It only needs a separate >> defconfig, which is pretty esay. >> >> Regards, >> Simon >> _______________________________________________ >> U-Boot mailing list >> U-Boot at lists.denx.de >> http://lists.denx.de/mailman/listinfo/u-boot > > > I send a patch "rockchip: firefly: configs: add a new config to keep > non-of-platdata" to save old config. OK thanks. Acked-by: Simon Glass <sjg@chromium.org> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata 2017-02-23 3:35 ` Simon Glass @ 2017-02-23 16:19 ` Simon Glass 0 siblings, 0 replies; 7+ messages in thread From: Simon Glass @ 2017-02-23 16:19 UTC (permalink / raw) To: u-boot On 22 February 2017 at 20:35, Simon Glass <sjg@chromium.org> wrote: > On 20 February 2017 at 03:28, Jacob Chen <jacobchen110@gmail.com> wrote: >> Hi, >> >> 2017-02-17 4:44 GMT+08:00 Simon Glass <sjg@chromium.org>: >>> Hi, >>> >>> On 14 February 2017 at 23:41, Jacob Chen <jacob2.chen@rock-chips.com> wrote: >>>> Hi Kever, >>>> >>>> >>>> Kever Yang wrote on 2017?02?15? 14:13: >>>>> >>>>> Hi Jacob, >>>>> >>>>> On 02/15/2017 11:06 AM, Jacob Chen wrote: >>>>>> >>>>>> We should remove config_spl_of_platdata to build u-boot-spl-dtb.bin >>>>>> rather than u-boot-spl-nodtb.bin >>>>>> since we use spl_back_to_brom. >>>>> >>>>> >>>>> Have you try with CONFIG_SPL_OF_PLATDATA on and without SPL_BACK_TO_BROM? >>>>> >>>>> If this works on firefly, then we can still leave it as an example to show >>>>> how both way works. >>>>> >>>> >>>> Firefly had been using CONFIG_SPL_OF_PLATDATA and it works well. >>>> Just it will make trouble for other things to generate the u-boot blob, >>>> like yocto, sdk build scripts, guides... >>>> >>>> At first, I also want to keep CONFIG_SPL_OF_PLATDATA in firefly, but later >>>> I find that i have to do a lot of unnecessary work to deal with it, >>>> so now I would like one chip use one way to generate the u-boot blob. >>> >>> How about having a separate board config like firefly-rk3288-dt so >>> that we keep the non-of-platdata alive? It only needs a separate >>> defconfig, which is pretty esay. >>> >>> Regards, >>> Simon >>> _______________________________________________ >>> U-Boot mailing list >>> U-Boot at lists.denx.de >>> http://lists.denx.de/mailman/listinfo/u-boot >> >> >> I send a patch "rockchip: firefly: configs: add a new config to keep >> non-of-platdata" to save old config. > > OK thanks. > > Acked-by: Simon Glass <sjg@chromium.org> Applied to u-boot-rockchip, thanks! ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-02-23 16:19 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-02-15 3:06 [U-Boot] [PATCH] rockchip: firefly: configs: remove config_spl_of_platdata Jacob Chen 2017-02-15 6:13 ` Kever Yang 2017-02-15 6:41 ` Jacob Chen 2017-02-16 20:44 ` Simon Glass 2017-02-20 10:28 ` Jacob Chen 2017-02-23 3:35 ` Simon Glass 2017-02-23 16:19 ` Simon Glass
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.