[Skiboot] [PATCH 1/3] pci-quirk: Re-order struct members

Michael Neuling mikey at neuling.org
Tue Jul 23 17:22:34 AEST 2019



On Fri, 2019-07-19 at 19:38 +1000, Oliver O'Halloran wrote:
> Signed-off-by: Oliver O'Halloran <oohall at gmail.com>

LGTM

Acked-by: Michael Neuling <mikey at neuling.org>

oohal: Will you release my family now? :-P

Mikey


> ---
>  core/pci-quirk.c    | 4 ++--
>  include/pci-quirk.h | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/core/pci-quirk.c b/core/pci-quirk.c
> index 1007e9621d71..0862e8dc4072 100644
> --- a/core/pci-quirk.c
> +++ b/core/pci-quirk.c
> @@ -52,8 +52,8 @@ static void quirk_astbmc_vga(struct phb *phb __unused,
>  /* Quirks are: {fixup function, vendor ID, (device ID or PCI_ANY_ID)} */
>  static const struct pci_quirk quirk_table[] = {
>  	/* ASPEED 2400 VGA device */
> -	{ &quirk_astbmc_vga, 0x1a03, 0x2000 },
> -	{ NULL, 0, 0 }
> +	{ 0x1a03, 0x2000, &quirk_astbmc_vga },
> +	{ 0, 0, NULL }
>  };
>  
>  static void __pci_handle_quirk(struct phb *phb, struct pci_device *pd,
> diff --git a/include/pci-quirk.h b/include/pci-quirk.h
> index 1883248af548..bea0c2f88601 100644
> --- a/include/pci-quirk.h
> +++ b/include/pci-quirk.h
> @@ -22,9 +22,9 @@
>  #define PCI_ANY_ID 0xFFFF
>  
>  struct pci_quirk {
> -	void (*fixup)(struct phb *, struct pci_device *);
>  	uint16_t vendor_id;
>  	uint16_t device_id;
> +	void (*fixup)(struct phb *, struct pci_device *);
>  };
>  
>  void pci_handle_quirk(struct phb *phb, struct pci_device *pd);



More information about the Skiboot mailing list