Re: [PATCH 3/3] PCI/AER: Use explicit register sizes for struct members

From: Jonathan Cameron
Date: Fri Dec 08 2023 - 09:39:18 EST


On Wed, 6 Dec 2023 16:42:31 -0600
Bjorn Helgaas <helgaas@xxxxxxxxxx> wrote:

> From: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
>
> aer_irq() reads the AER Root Error Status and Error Source Identification
> (PCI_ERR_ROOT_STATUS and PCI_ERR_ROOT_ERR_SRC) registers directly into
> struct aer_err_source. Both registers are 32 bits, so declare the members
> explicitly as "u32" instead of "unsigned int".
>
> Similarly, aer_get_device_error_info() reads the AER Header Log
> (PCI_ERR_HEADER_LOG) registers, which are also 32 bits, into struct
> aer_header_log_regs. Declare those members as "u32" as well.
>
> No functional changes intended.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>

Another sensible cleanup. FWIW on such simple patches
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>