Re: [PATCH] hwmon: (fam15h_power) Fix pci_device_id array

[Jean Delvare <khali@xxxxxxxxxxxx>]

On Wed, 25 Apr 2012 13:50:09 -0700, Guenter Roeck wrote:
> pci_match_id() takes an *array* of IDs which must be properly zero-
> terminated.

Good catch, sorry for not seeing it.

> Reported-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx>
> Signed-off-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx>
> ---
>  drivers/hwmon/fam15h_power.c |    9 ++++++---
>  1 files changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/hwmon/fam15h_power.c b/drivers/hwmon/fam15h_power.c
> index 37a8fc9..e8e18ca 100644
> --- a/drivers/hwmon/fam15h_power.c
> +++ b/drivers/hwmon/fam15h_power.c
> @@ -128,17 +128,20 @@ static bool __devinit fam15h_power_is_internal_node0(struct pci_dev *f4)
>   * counter saturations resulting in bogus power readings.
>   * We correct this value ourselves to cope with older BIOSes.
>   */
> +static DEFINE_PCI_DEVICE_TABLE(affected_device) = {
> +	{ PCI_VDEVICE(AMD, PCI_DEVICE_ID_AMD_15H_NB_F4) },
> +	{ 0 }
> +};
> +

Not sure why you move it here, you didn't have to, did you?

Other than this:

Acked-by: Jean Delvare <khali@xxxxxxxxxxxx>

>  static void __devinit tweak_runavg_range(struct pci_dev *pdev)
>  {
>  	u32 val;
> -	const struct pci_device_id affected_device = {
> -		PCI_VDEVICE(AMD, PCI_DEVICE_ID_AMD_15H_NB_F4) };
>  
>  	/*
>  	 * let this quirk apply only to the current version of the
>  	 * northbridge, since future versions may change the behavior
>  	 */
> -	if (!pci_match_id(&affected_device, pdev))
> +	if (!pci_match_id(affected_device, pdev))
>  		return;
>  
>  	pci_bus_read_config_dword(pdev->bus,

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors