diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2024-10-25 17:26:34 +0300 |
|---|---|---|
| committer | Uwe Kleine-König <ukleinek@kernel.org> | 2024-10-25 22:52:45 +0200 |
| commit | fdb62922ae89c17963f80abdd14b4d9f053bc962 (patch) | |
| tree | 74cf92263d869880c74f4fc08c58cd71aadd243d /drivers/pwm | |
| parent | 664b5e466f915ad7fce87215ccfb038c47ace4fb (diff) | |
pwm: core: use device_match_name() instead of strcmp(dev_name(...
Use the dedicated helper for comparing device names against strings.
Note, the current code has a check for the dev_name() against NULL.
With the current implementations of the device_add() and dev_set_name()
it most likely a theoretical assumption that that might happen, while
I don't see how. Hence, that check has simply been removed.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20241025142704.405340-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Uwe Kleine-König <ukleinek@kernel.org>
Diffstat (limited to 'drivers/pwm')
| -rw-r--r-- | drivers/pwm/core.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/pwm/core.c b/drivers/pwm/core.c index 634be56e204b..4399e793efaf 100644 --- a/drivers/pwm/core.c +++ b/drivers/pwm/core.c @@ -852,9 +852,7 @@ static struct pwm_chip *pwmchip_find_by_name(const char *name) guard(mutex)(&pwm_lock); idr_for_each_entry_ul(&pwm_chips, chip, tmp, id) { - const char *chip_name = dev_name(pwmchip_parent(chip)); - - if (chip_name && strcmp(chip_name, name) == 0) + if (device_match_name(pwmchip_parent(chip), name)) return chip; } |