[PATCH v2 2/2] x86: acpi: Add memset to initialize SPCR table

[Bin Meng]

On Mon, Sep 21, 2020 at 9:43 AM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> On Wed, Sep 16, 2020 at 11:12 PM Andy Shevchenko
> <andriy.shevchenko at linux.intel.com> wrote:
> >
> > On Wed, Sep 16, 2020 at 04:57:53PM +0200, Wolfgang Wallner wrote:
> > > Add a missing memset to acpi_create_spcr().
> > >
> > > The other acpi_create_xxxx() functions perform a memset on their
> > > structures, acpi_create_spcr() does not and as a result the contents of
> > > this table are partly uninitialized (and thus random after every reset).
> >
> > > Fixes: commit b288cd960072 ("x86: acpi: Generate SPCR table")
> > >
> > > Signed-off-by: Wolfgang Wallner <wolfgang.wallner at br-automation.com>
> >
> > In tag block should not be any blank line, but I guess maintainer can fix this
> > when applying (i.o.w. no need to resend).
>
> Yep, normally I will fix such kinds of issues when applying. Thanks
> for pointing it out.

Fixed the tags format, and

>
> >
> > > Reviewed-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > > Reviewed-by: Simon Glass <sjg at chromium.org>
> > >
> > > ---
> > >
> > > Changes in v2:
> > >  - Removed unrelated whitespace change
> > >  - Added Reviewed-by tags
> > >
> > >  arch/x86/lib/acpi_table.c | 2 ++
> > >  1 file changed, 2 insertions(+)
> > >
> > > diff --git a/arch/x86/lib/acpi_table.c b/arch/x86/lib/acpi_table.c
> > > index 6b827bfa3f..c445aa6870 100644
> > > --- a/arch/x86/lib/acpi_table.c
> > > +++ b/arch/x86/lib/acpi_table.c
> > > @@ -252,6 +252,8 @@ static void acpi_create_spcr(struct acpi_spcr *spcr)
> > >       int space_id;
> > >       int ret = -ENODEV;
> > >
> > > +     memset((void *)spcr, 0, sizeof(struct acpi_spcr));
> > > +
> > >       /* Fill out header fields */
> > >       acpi_fill_header(header, "SPCR");
> > >       header->length = sizeof(struct acpi_spcr);
> > > --
>
> Reviewed-by: Bin Meng <bmeng.cn at gmail.com>

applied to u-boot-x86, thanks!