ACK: [PATCH 08/12] acpi/dmar: use PRIu8 specifier for uint8_t type

Alex Hung alex.hung at canonical.com
Thu Sep 6 21:39:31 UTC 2018


On 2018-09-06 07:19 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/acpi/dmar/dmar.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/acpi/dmar/dmar.c b/src/acpi/dmar/dmar.c
> index 7685f35d..a1181c65 100644
> --- a/src/acpi/dmar/dmar.c
> +++ b/src/acpi/dmar/dmar.c
> @@ -109,7 +109,7 @@ static int read_pci_device_secondary_bus_number(const uint8_t seg,
>   	size_t count;
>   
>   	snprintf(path, sizeof(path),
> -		"/sys/bus/pci/devices/%04x:%02x:%02x.%d/config",
> +		"/sys/bus/pci/devices/%04x:%02x:%02x.%" PRIu8 "/config",
>   		seg, bus, dev, fn);
>   	if ((file = fopen(path, "r")) == NULL)
>   		return -1;
> 



Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list