Messages in this thread | | | Date | Tue, 28 Mar 2023 11:23:10 +0200 | Subject | Re: [PATCH v2] drivers/gpio : Remove redundant platform_set_drvdata(). | From | "Enrico Weigelt, metux IT consult" <> |
| |
On 19.03.23 12:59, Lizhe wrote:
Hi,
> platform_set_drvdata() is redundant in these functions. > the purpose of calling this function is to place data > in *driver_data. but the data is not retrieved in these > functions
I appreciate your contribution, but I have to NAK for the gpio-amd-fch driver: it would at least create a memleak (driver core can't free the driver_data when the driver is unloaded).
In general, patches to drivers should be splitted per driver, unless there's a hard reason to do otherwise. That's important for the individual maintainers being able to review patches to their drivers. If each maintainer can only ack on pieces of the patch, somebody else would need to split out those things that could be picked. Really too complicated.
--mtx
-- --- Hinweis: unverschlüsselte E-Mails können leicht abgehört und manipuliert werden ! Für eine vertrauliche Kommunikation senden Sie bitte ihren GPG/PGP-Schlüssel zu. --- Enrico Weigelt, metux IT consult Free software and Linux embedded engineering info@metux.net -- +49-151-27565287
| |