]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
net: phy: marvell: Use HWMON_CHANNEL_INFO macro to simplify code
authorHuisong Li <lihuisong@huawei.com>
Mon, 10 Feb 2025 05:47:08 +0000 (13:47 +0800)
committerJakub Kicinski <kuba@kernel.org>
Wed, 12 Feb 2025 01:07:02 +0000 (17:07 -0800)
Use HWMON_CHANNEL_INFO macro to simplify code.

Signed-off-by: Huisong Li <lihuisong@huawei.com>
Link: https://patch.msgid.link/20250210054710.12855-4-lihuisong@huawei.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/phy/marvell.c

index 44e1927de4997c6f40aaa03af89c401035aef1f1..dd254e36ca8ae44c4ebfc143d880b37c976a8e9f 100644 (file)
@@ -3124,33 +3124,13 @@ static umode_t marvell_hwmon_is_visible(const void *data,
        }
 }
 
-static u32 marvell_hwmon_chip_config[] = {
-       HWMON_C_REGISTER_TZ,
-       0
-};
-
-static const struct hwmon_channel_info marvell_hwmon_chip = {
-       .type = hwmon_chip,
-       .config = marvell_hwmon_chip_config,
-};
-
 /* we can define HWMON_T_CRIT and HWMON_T_MAX_ALARM even though these are not
  * defined for all PHYs, because the hwmon code checks whether the attributes
  * exists via the .is_visible method
  */
-static u32 marvell_hwmon_temp_config[] = {
-       HWMON_T_INPUT | HWMON_T_CRIT | HWMON_T_MAX_ALARM,
-       0
-};
-
-static const struct hwmon_channel_info marvell_hwmon_temp = {
-       .type = hwmon_temp,
-       .config = marvell_hwmon_temp_config,
-};
-
 static const struct hwmon_channel_info * const marvell_hwmon_info[] = {
-       &marvell_hwmon_chip,
-       &marvell_hwmon_temp,
+       HWMON_CHANNEL_INFO(chip, HWMON_C_REGISTER_TZ),
+       HWMON_CHANNEL_INFO(temp, HWMON_T_INPUT | HWMON_T_CRIT | HWMON_T_MAX_ALARM),
        NULL
 };