[U-Boot] [PATCH v2 09/10] x86: coreboot: Enable LPC TPM and CONFIG_NO_RESET_CODE
Graeme Russ
graeme.russ at gmail.com
Thu Oct 11 05:47:37 CEST 2012
Hi Marek,
On Thu, Oct 11, 2012 at 2:45 PM, Marek Vasut <marek.vasut at gmail.com> wrote:
> Dear Graeme Russ,
>
>> Hi Simon,
>>
>> On Thu, Oct 11, 2012 at 10:13 AM, Simon Glass <sjg at chromium.org> wrote:
>> > Coreboot boards have an LPC TPM connected, so enable this. We also need
>> > to skip the reset code.
>> >
>> > Signed-off-by: Simon Glass <sjg at chromium.org>
>> > ---
>> >
>> > include/configs/coreboot.h | 6 +++++-
>> > 1 files changed, 5 insertions(+), 1 deletions(-)
>> >
>> > diff --git a/include/configs/coreboot.h b/include/configs/coreboot.h
>> > index 2c65d74..75db176 100644
>> > --- a/include/configs/coreboot.h
>> > +++ b/include/configs/coreboot.h
>> > @@ -37,7 +37,7 @@
>> >
>> > #define CONFIG_SYS_COREBOOT
>> > #undef CONFIG_SHOW_BOOT_PROGRESS
>> > #define CONFIG_LAST_STAGE_INIT
>> >
>> > -
>> > +#define CONFIG_NO_RESET_CODE
>> >
>> > /*----------------------------------------------------------------------
>> > -
>> >
>> > * Watchdog Configuration
>> >
>> > @@ -45,6 +45,10 @@
>> >
>> > #undef CONFIG_WATCHDOG
>> > #undef CONFIG_HW_WATCHDOG
>> >
>> > +/* Generic TPM interfaced through LPC bus */
>> > +#define CONFIG_GENERIC_LPC_TPM
>> > +#define CONFIG_TPM_TIS_BASE_ADDRESS 0xfed40000
>> > +
>> >
>> > /*----------------------------------------------------------------------
>> > -
>> >
>> > * Real Time Clock Configuration
>> > */
>> >
>> > --
>> > 1.7.7.3
>>
>> Acked-by: Graeme Russ <graeme.russ at gmail.com>
>
> Well, spliting it into two patches -- since each hunk does something else --
> won't hurt. But I think it doesn't matter.
It's board level config - I see no reason to atomise it. If it was
global / arch / SoC / CPU level then I would to help maintain
bisectability.
Regards,
Graeme
More information about the U-Boot
mailing list