From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: Fri, 3 Apr 2020 16:32:53 +0300
Subject: [PATCH v3 27/29] acpi: Put table-setup code in its own function
In-Reply-To: <20200330171226.v3.27.I34e9fcd28119cc2fcb87ad8679efb582a4c611df@changeid>
References: <20200330231305.130488-1-sjg@chromium.org>
 <20200330171226.v3.27.I34e9fcd28119cc2fcb87ad8679efb582a4c611df@changeid>
Message-ID: <20200403133253.GK3676135@smile.fi.intel.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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);

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)

-- 
With Best Regards,
Andy Shevchenko