Re: [PATCH v3 08/11] gpio: sim: Remove gpio_sim_dev_match_fwnode()

Zijun Hu <[email protected]>
Newsgroups gmane.linux.kernel.gpio,gmane.linux.kernel,gmane.linux.sound,gmane.linux.ports.sparc,gmane.linux.block,gmane.linux.kernel.firewire.devel,gmane.linux.kernel.efi,gmane.comp.video.dri.devel,gmane.linux.ports.arm.mediatek,gmane.linux.kernel.hwmon,gmane.linux.drivers.video-input-infrastructure,gmane.linux.pwm,gmane.linux.scsi,gmane.linux.iscsi.open-iscsi,gmane.linux.usb.general,gmane.linux.serial,gmane.linux.network
Message-ID <[email protected]>
On 2024/12/6 00:24, Bartosz Golaszewski wrote:
> On Thu, Dec 5, 2024 at 1:15 AM Zijun Hu <[email protected]> wrote:
>>
>> From: Zijun Hu <[email protected]>
>>
>> gpio_sim_dev_match_fwnode() is a simple wrapper of device_match_fwnode()
>> Remvoe the unnecessary wrapper.
>>
>> Signed-off-by: Zijun Hu <[email protected]>
>> ---
>>  drivers/gpio/gpio-sim.c | 7 +------
>>  1 file changed, 1 insertion(+), 6 deletions(-)
>>
>> diff --git a/drivers/gpio/gpio-sim.c b/drivers/gpio/gpio-sim.c
>> index 370b71513bdb529112e157fa22a5451e02502a17..b1f33cbaaaa78aca324f99c45a868e7e79a9d672 100644
>> --- a/drivers/gpio/gpio-sim.c
>> +++ b/drivers/gpio/gpio-sim.c
>> @@ -413,11 +413,6 @@ static int gpio_sim_setup_sysfs(struct gpio_sim_chip *chip)
>>         return devm_add_action_or_reset(dev, gpio_sim_sysfs_remove, chip);
>>  }
>>
>> -static int gpio_sim_dev_match_fwnode(struct device *dev, const void *data)
>> -{
>> -       return device_match_fwnode(dev, data);
>> -}
>> -
>>  static int gpio_sim_add_bank(struct fwnode_handle *swnode, struct device *dev)
>>  {
>>         struct gpio_sim_chip *chip;
>> @@ -503,7 +498,7 @@ static int gpio_sim_add_bank(struct fwnode_handle *swnode, struct device *dev)
>>         if (ret)
>>                 return ret;
>>
>> -       chip->dev = device_find_child(dev, swnode, gpio_sim_dev_match_fwnode);
>> +       chip->dev = device_find_child(dev, swnode, device_match_fwnode);
>>         if (!chip->dev)
>>                 return -ENODEV;
>>
>>
>> --
>> 2.34.1
>>
>>
> 
> Please use get_maintainers.pl to get the complete list of addresses to Cc.

thanks for code review.
will fix it in v4.

> 
> Bartosz
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.