Hi Saket, The patch/commit title is: x86: Added ACPI support as a configuration option.
Please use simple tense in the title. So it's 'Add', not 'Added'. Also please do not add any ending period. On Thu, Aug 13, 2015 at 11:01 AM, Saket Sinha <saket.sinh...@gmail.com> wrote: Please add some commit message to describe what this commit is for. > Signed-off-by: Saket Sinha <saket.sinh...@gmail.com> > --- > > arch/x86/Kconfig | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig > index 01ed760..36eae6d 100644 > --- a/arch/x86/Kconfig > +++ b/arch/x86/Kconfig > @@ -346,6 +346,12 @@ config GENERATE_MP_TABLE > > endmenu > > +config GENERATE_ACPI_TABLE Please move this Kconfig option into the "System tables" menu. > + bool "Generate ACPI(Advanced Configuration and Power Interface) table" Please add 'an' before ACPI, and one space after ACPI. > + default y Default should be n. > + help > + The Advanced Configuration and Power Interface (ACPI) specification > provides an open standard for device configuration and management > by the operating system. It defines platform-independent interfaces for > configuration, power management monitoring. Please rewrite this paragraph, make sure they do not exceed 80 cols. > + > config MAX_PIRQ_LINKS > int > default 8 > -- Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
