ACK: [PATCH 13/27] acpi: crsdump: add in missing pointer constifications

[ivanhu]

On 08/15/2018 09:11 PM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> Fix up some build warnings because of missing const casting
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/acpi/crsdump/crsdump.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/src/acpi/crsdump/crsdump.c b/src/acpi/crsdump/crsdump.c
> index abf20ce1..706677c4 100644
> --- a/src/acpi/crsdump/crsdump.c
> +++ b/src/acpi/crsdump/crsdump.c
> @@ -74,7 +74,7 @@ static void crsdump_show_info(
>  			/*
>  			 *  CRS_BIT*() data
>  			 */
> -			val = (uint64_t)*(uint8_t*)(data + info->offset);
> +			val = (uint64_t)*(const uint8_t *)(data + info->offset);
>  			while (mask && ((mask & 1) == 0)) {
>  				val >>= 1;
>  				mask >>= 1;
> @@ -87,16 +87,16 @@ static void crsdump_show_info(
>  			 */
>  			switch (info->bitlength) {
>  			case 8:
> -				val = (uint64_t)*(uint8_t*)(data + info->offset);
> +				val = (uint64_t)*(const uint8_t*)(data + info->offset);
>  				break;
>  			case 16:
> -				val = (uint64_t)*(uint16_t*)(data + info->offset);
> +				val = (uint64_t)*(const uint16_t*)(data + info->offset);
>  				break;
>  			case 32:
> -				val = (uint64_t)*(uint32_t*)(data + info->offset);
> +				val = (uint64_t)*(const uint32_t*)(data + info->offset);
>  				break;
>  			case 64:
> -				val = (uint64_t)*(uint64_t*)(data + info->offset);
> +				val = (uint64_t)*(const uint64_t*)(data + info->offset);
>  				break;
>  			default:
>  				val = ~0;
Acked-by: Ivan Hu <ivan.hu at canonical.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20180816/8d6f97c4/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20180816/8d6f97c4/attachment.sig>