Message ID | 1439450957-23197-9-git-send-email-bmeng.cn@gmail.com |
---|---|
State | Accepted |
Delegated to: | Simon Glass |
Headers | show |
Hi Bin, On 13 August 2015 at 01:29, Bin Meng <bmeng.cn@gmail.com> wrote: > When running U-Boot bare-metal, the cbfs command is useless. > > Signed-off-by: Bin Meng <bmeng.cn@gmail.com> > --- > > include/configs/x86-common.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h > index 349b06c..3d07cc0 100644 > --- a/include/configs/x86-common.h > +++ b/include/configs/x86-common.h > @@ -87,7 +87,9 @@ > #define CONFIG_ISO_PARTITION /* Experimental */ > > #define CONFIG_CMD_PART > +#ifdef CONFIG_SYS_COREBOOT > #define CONFIG_CMD_CBFS > +#endif > #define CONFIG_CMD_EXT4 > #define CONFIG_CMD_EXT4_WRITE > #define CONFIG_PARTITION_UUIDS > -- > 1.8.2.1 > Acked-by: Simon Glass <sjg@chromium.org> We should move that to Kconfig. Have you tried Masahiro's moveconfig script? Regards, Simon
Hi Simon, On Mon, Aug 17, 2015 at 5:27 AM, Simon Glass <sjg@chromium.org> wrote: > Hi Bin, > > On 13 August 2015 at 01:29, Bin Meng <bmeng.cn@gmail.com> wrote: >> When running U-Boot bare-metal, the cbfs command is useless. >> >> Signed-off-by: Bin Meng <bmeng.cn@gmail.com> >> --- >> >> include/configs/x86-common.h | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h >> index 349b06c..3d07cc0 100644 >> --- a/include/configs/x86-common.h >> +++ b/include/configs/x86-common.h >> @@ -87,7 +87,9 @@ >> #define CONFIG_ISO_PARTITION /* Experimental */ >> >> #define CONFIG_CMD_PART >> +#ifdef CONFIG_SYS_COREBOOT >> #define CONFIG_CMD_CBFS >> +#endif >> #define CONFIG_CMD_EXT4 >> #define CONFIG_CMD_EXT4_WRITE >> #define CONFIG_PARTITION_UUIDS >> -- >> 1.8.2.1 >> > > Acked-by: Simon Glass <sjg@chromium.org> > > We should move that to Kconfig. Have you tried Masahiro's moveconfig script? > I haven't got a chance to try moveconfig scripts, but I will try some time. I believe there are lots of work to get all config.h defines converted to Kconfig options first, and some day we will achieve this :) Regards, Bin
On 16 August 2015 at 19:25, Bin Meng <bmeng.cn@gmail.com> wrote: > Hi Simon, > > On Mon, Aug 17, 2015 at 5:27 AM, Simon Glass <sjg@chromium.org> wrote: >> Hi Bin, >> >> On 13 August 2015 at 01:29, Bin Meng <bmeng.cn@gmail.com> wrote: >>> When running U-Boot bare-metal, the cbfs command is useless. >>> >>> Signed-off-by: Bin Meng <bmeng.cn@gmail.com> >>> --- >>> >>> include/configs/x86-common.h | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h >>> index 349b06c..3d07cc0 100644 >>> --- a/include/configs/x86-common.h >>> +++ b/include/configs/x86-common.h >>> @@ -87,7 +87,9 @@ >>> #define CONFIG_ISO_PARTITION /* Experimental */ >>> >>> #define CONFIG_CMD_PART >>> +#ifdef CONFIG_SYS_COREBOOT >>> #define CONFIG_CMD_CBFS >>> +#endif >>> #define CONFIG_CMD_EXT4 >>> #define CONFIG_CMD_EXT4_WRITE >>> #define CONFIG_PARTITION_UUIDS >>> -- >>> 1.8.2.1 >>> >> >> Acked-by: Simon Glass <sjg@chromium.org> >> >> We should move that to Kconfig. Have you tried Masahiro's moveconfig script? >> > > I haven't got a chance to try moveconfig scripts, but I will try some > time. I believe there are lots of work to get all config.h defines > converted to Kconfig options first, and some day we will achieve this > :) > > Regards, > Bin Applied to u-boot-x86, thanks!
diff --git a/include/configs/x86-common.h b/include/configs/x86-common.h index 349b06c..3d07cc0 100644 --- a/include/configs/x86-common.h +++ b/include/configs/x86-common.h @@ -87,7 +87,9 @@ #define CONFIG_ISO_PARTITION /* Experimental */ #define CONFIG_CMD_PART +#ifdef CONFIG_SYS_COREBOOT #define CONFIG_CMD_CBFS +#endif #define CONFIG_CMD_EXT4 #define CONFIG_CMD_EXT4_WRITE #define CONFIG_PARTITION_UUIDS
When running U-Boot bare-metal, the cbfs command is useless. Signed-off-by: Bin Meng <bmeng.cn@gmail.com> --- include/configs/x86-common.h | 2 ++ 1 file changed, 2 insertions(+)