Skip to content

Commit a8bc416

Browse files
yghannambp3tk0v
authored andcommitted
hwmon: (k10temp) Reduce k10temp_get_ccd_support() parameters
Currently, k10temp_get_ccd_support() takes as input "pdev" and "data". However, "pdev" is already included in "data". Furthermore, the "pdev" parameter is no longer used in k10temp_get_ccd_support(), since its use was moved into read_ccd_temp_reg(). Drop the "pdev" input parameter as it is no longer needed. No functional change is intended. Signed-off-by: Yazen Ghannam <[email protected]> Signed-off-by: Borislav Petkov (AMD) <[email protected]> Reviewed-by: Mario Limonciello <[email protected]> Acked-by: Guenter Roeck <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent cc66126 commit a8bc416

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

drivers/hwmon/k10temp.c

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -385,8 +385,7 @@ static const struct hwmon_chip_info k10temp_chip_info = {
385385
.info = k10temp_info,
386386
};
387387

388-
static void k10temp_get_ccd_support(struct pci_dev *pdev,
389-
struct k10temp_data *data, int limit)
388+
static void k10temp_get_ccd_support(struct k10temp_data *data, int limit)
390389
{
391390
u32 regval;
392391
int i;
@@ -456,18 +455,18 @@ static int k10temp_probe(struct pci_dev *pdev, const struct pci_device_id *id)
456455
case 0x11: /* Zen APU */
457456
case 0x18: /* Zen+ APU */
458457
data->ccd_offset = 0x154;
459-
k10temp_get_ccd_support(pdev, data, 4);
458+
k10temp_get_ccd_support(data, 4);
460459
break;
461460
case 0x31: /* Zen2 Threadripper */
462461
case 0x60: /* Renoir */
463462
case 0x68: /* Lucienne */
464463
case 0x71: /* Zen2 */
465464
data->ccd_offset = 0x154;
466-
k10temp_get_ccd_support(pdev, data, 8);
465+
k10temp_get_ccd_support(data, 8);
467466
break;
468467
case 0xa0 ... 0xaf:
469468
data->ccd_offset = 0x300;
470-
k10temp_get_ccd_support(pdev, data, 8);
469+
k10temp_get_ccd_support(data, 8);
471470
break;
472471
}
473472
} else if (boot_cpu_data.x86 == 0x19) {
@@ -481,21 +480,21 @@ static int k10temp_probe(struct pci_dev *pdev, const struct pci_device_id *id)
481480
case 0x21: /* Zen3 Ryzen Desktop */
482481
case 0x50 ... 0x5f: /* Green Sardine */
483482
data->ccd_offset = 0x154;
484-
k10temp_get_ccd_support(pdev, data, 8);
483+
k10temp_get_ccd_support(data, 8);
485484
break;
486485
case 0x40 ... 0x4f: /* Yellow Carp */
487486
data->ccd_offset = 0x300;
488-
k10temp_get_ccd_support(pdev, data, 8);
487+
k10temp_get_ccd_support(data, 8);
489488
break;
490489
case 0x60 ... 0x6f:
491490
case 0x70 ... 0x7f:
492491
data->ccd_offset = 0x308;
493-
k10temp_get_ccd_support(pdev, data, 8);
492+
k10temp_get_ccd_support(data, 8);
494493
break;
495494
case 0x10 ... 0x1f:
496495
case 0xa0 ... 0xaf:
497496
data->ccd_offset = 0x300;
498-
k10temp_get_ccd_support(pdev, data, 12);
497+
k10temp_get_ccd_support(data, 12);
499498
break;
500499
}
501500
} else if (boot_cpu_data.x86 == 0x1a) {

0 commit comments

Comments
 (0)