hwmon: drop unnecessary OF name NULL checks
Subsystems (2)
PATCH 1/1 hwmon: drop unnecessary OF name NULL checks
Checking the child node names is pointless as the DT node name can never be NULL, so remove it.
Cc Jean Delvare <[email protected]>
Cc Guenter Roeck <[email protected]>
Cc Benjamin Herrenschmidt <[email protected]>
Cc Paul Mackerras <[email protected]>
Cc Michael Ellerman <[email protected]>
Signed-off-by Rob Herring <[email protected]>
--- drivers/hwmon/ibmpowernv.c | 6 ------ 1 file changed, 6 deletions(-)
drivers/hwmon/ibmpowernv.c +0 -6
@@ -458,9 +458,6 @@ static int populate_attr_groups(struct platform_device *pdev)for_each_child_of_node (opal, np) {const char *label; -if (np->name ==NULL ) -continue ; - type =get_sensor_type (np);if (type ==MAX_SENSOR_TYPE )continue ;
@@ -589,9 +586,6 @@ static int create_device_attrs(struct platform_device *pdev)const char *label;enum sensors type; -if (np->name ==NULL ) -continue ; - type =get_sensor_type (np);if (type ==MAX_SENSOR_TYPE )continue ;
Signature · 1 line
2.17.1
Quoted 11 lines · Rob Herring
Checking the child node names is pointless as the DT node name can never be NULL, so remove it.
Cc Jean Delvare <[email protected]>Cc Guenter Roeck <[email protected]>Cc Benjamin Herrenschmidt <[email protected]>Cc Paul Mackerras <[email protected]>Cc Michael Ellerman <[email protected]>Signed-off-by Rob Herring <[email protected]>
Applied to hwmon-next.
Thanks,
Guenter
Quoted 31 lines
--- drivers/hwmon/ibmpowernv.c | 6 ------ 1 file changed, 6 deletions(-)drivers/hwmon/ibmpowernv.c +0
-6@@ -458,9 +458,6 @@ static int populate_attr_groups(struct platform_device *pdev)for_each_child_of_node (opal, np) {const char *label; -if (np->name ==NULL ) -continue ; - type =get_sensor_type (np);if (type ==MAX_SENSOR_TYPE )continue ;@@ -589,9 +586,6 @@ static int create_device_attrs(struct platform_device *pdev)const char *label;enum sensors type; -if (np->name ==NULL ) -continue ; - type =get_sensor_type (np);if (type ==MAX_SENSOR_TYPE )continue ;Signature · 1 line
2.17.1