diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2025-03-10 17:54:53 +0300 |
|---|---|---|
| committer | Lee Jones <lee@kernel.org> | 2025-04-10 12:12:24 +0300 |
| commit | 53762bb44b0659e79a3e3177372e823ec4afcc8a (patch) | |
| tree | dd76d2862e0ee361e88d8e0fe7dde91563bbd40e | |
| parent | 4623cc4e9a5f1b7ad7e6599dfc2a5a4d9d4f5d72 (diff) | |
| download | linux-53762bb44b0659e79a3e3177372e823ec4afcc8a.tar.xz | |
leds: ncp5623: Use fwnode_get_child_node_count()
Since fwnode_get_child_node_count() was split from its device property
counterpart, we may utilise it in the driver and drop custom implementation.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Link: https://lore.kernel.org/r/20250310150835.3139322-4-andriy.shevchenko@linux.intel.com
Signed-off-by: Lee Jones <lee@kernel.org>
| -rw-r--r-- | drivers/leds/rgb/leds-ncp5623.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/leds/rgb/leds-ncp5623.c b/drivers/leds/rgb/leds-ncp5623.c index f18156683375..7c7d44623a9e 100644 --- a/drivers/leds/rgb/leds-ncp5623.c +++ b/drivers/leds/rgb/leds-ncp5623.c @@ -155,9 +155,9 @@ static int ncp5623_probe(struct i2c_client *client) struct device *dev = &client->dev; struct fwnode_handle *mc_node, *led_node; struct led_init_data init_data = { }; - int num_subleds = 0; struct ncp5623 *ncp; struct mc_subled *subled_info; + unsigned int num_subleds; u32 color_index; u32 reg; int ret; @@ -172,8 +172,7 @@ static int ncp5623_probe(struct i2c_client *client) if (!mc_node) return -EINVAL; - fwnode_for_each_child_node(mc_node, led_node) - num_subleds++; + num_subleds = fwnode_get_child_node_count(mc_node); subled_info = devm_kcalloc(dev, num_subleds, sizeof(*subled_info), GFP_KERNEL); if (!subled_info) { |