[PATCH v3 27/29] acpi: Put table-setup code in its own function

Simon Glass sjg at chromium.org
Wed Apr 8 04:57:52 CEST 2020


Hi Andy,

On Fri, 3 Apr 2020 at 07:32, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> On Mon, Mar 30, 2020 at 05:13:03PM -0600, Simon Glass wrote:
> > We always write three basic tables to ACPI at the start. Move this into
> > its own function, along with acpi_fill_header(), so we can write a test
> > for this code.
>
> ...
>
> >       /* Re-calculate checksum */
> >       rsdt->header.checksum = 0;
> > -     rsdt->header.checksum = table_compute_checksum((u8 *)rsdt,
> > +     rsdt->header.checksum = table_compute_checksum(rsdt,
> >                                                      rsdt->header.length);

Please can you keep the filenames / functions in your response?
Fragments make it harder to find the code.

>
> Why suddenly casting is not needed in this patch?
> Same question to the rest.
>
> (If it's a valid change, it should be in a separate patch)

It was never needed. See the prototype for table_compute_checksum().

But I can put it back in.

Regards,
Simon


More information about the U-Boot mailing list