On Sat, 22 Mar 2025 at 01:21, Heinrich Schuchardt
<heinrich.schucha...@canonical.com> wrote:
>
> Use acpi_update_checksum() to update table header.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
Reviewed-by: Ilias Apalodimas <ilias.apalodi...@linaro.org>
> ---
> arch/arm/mach-bcm283x/bcm2711_acpi.c | 4 ++--
> board/raspberrypi/rpi/rpi.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-bcm283x/bcm2711_acpi.c
> b/arch/arm/mach-bcm283x/bcm2711_acpi.c
> index 79b283353cf..58f8ee232b9 100644
> --- a/arch/arm/mach-bcm283x/bcm2711_acpi.c
> +++ b/arch/arm/mach-bcm283x/bcm2711_acpi.c
> @@ -81,7 +81,7 @@ static int acpi_write_pptt(struct acpi_ctx *ctx, const
> struct acpi_writer *entry
> }
>
> header->length = ctx->current - ctx->tab_start;
> - header->checksum = table_compute_checksum(header, header->length);
> + acpi_update_checksum(header);
>
> acpi_inc(ctx, header->length);
> acpi_add_table(ctx, header);
> @@ -116,7 +116,7 @@ static int rpi_write_gtdt(struct acpi_ctx *ctx, const
> struct acpi_writer *entry)
> gtdt->el2_flags = GTDT_FLAG_INT_ACTIVE_LOW;
> gtdt->cnt_read_base = 0xffffffffffffffff;
>
> - header->checksum = table_compute_checksum(header, header->length);
> + acpi_update_checksum(header);
>
> acpi_add_table(ctx, gtdt);
>
> diff --git a/board/raspberrypi/rpi/rpi.c b/board/raspberrypi/rpi/rpi.c
> index 70d3c35499b..6ecd3eb120f 100644
> --- a/board/raspberrypi/rpi/rpi.c
> +++ b/board/raspberrypi/rpi/rpi.c
> @@ -791,7 +791,7 @@ static int rpi_acpi_write_ssdt(struct acpi_ctx *ctx,
> const struct acpi_writer *e
>
> /* (Re)calculate length and checksum */
> ssdt->length = ctx->current - (void *)ssdt;
> - ssdt->checksum = table_compute_checksum((void *)ssdt, ssdt->length);
> + acpi_update_checksum(ssdt);
> log_debug("SSDT at %p, length %x\n", ssdt, ssdt->length);
>
> /* Drop the table if it is empty */
> --
> 2.48.1
>
