Get rid of 'remove_new' relic from platform driver structThe continual trickle of small conversion patches is grating on me, andis really not helping. Just get rid of the 'remove_new' memberfunc
Get rid of 'remove_new' relic from platform driver structThe continual trickle of small conversion patches is grating on me, andis really not helping. Just get rid of the 'remove_new' memberfunction, which is just an alias for the plain 'remove', and had acomment to that effect: /* * .remove_new() is a relic from a prototype conversion of .remove(). * New drivers are supposed to implement .remove(). Once all drivers are * converted to not use .remove_new any more, it will be dropped. */This was just a tree-wide 'sed' script that replaced '.remove_new' with'.remove', with some care taken to turn a subsequent tab into two tabsto make things line up.I did do some minimal manual whitespace adjustment for places that usedspaces to line things up.Then I just removed the old (sic) .remove_new member function, and thisis the end result. No more unnecessary conversion noise.Signed-off-by: Linus Torvalds <[email protected]>
show more ...
pmdomain: core: constify of_phandle_args in xlateThe xlate callbacks are supposed to translate of_phandle_args to properprovider without modifying the of_phandle_args. Make the argumentpointer t
pmdomain: core: constify of_phandle_args in xlateThe xlate callbacks are supposed to translate of_phandle_args to properprovider without modifying the of_phandle_args. Make the argumentpointer to const for code safety and readability.Signed-off-by: Krzysztof Kozlowski <[email protected]>Link: https://lore.kernel.org/r/[email protected]Signed-off-by: Ulf Hansson <[email protected]>
pmdomain: xilinx/zynqmp: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume it's possible t
pmdomain: xilinx/zynqmp: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume it's possible to do error handling byreturning an error code. However the value returned is ignored (apartfrom emitting a warning) and this typically results in resource leaks.To improve here there is a quest to make the remove callback returnvoid. In the first step of this quest all drivers are converted to.remove_new(), which already returns void. Eventually after all driversare converted, .remove_new() will be renamed to .remove().Trivially convert this driver from always returning zero in the removecallback to the void returning variant.Signed-off-by: Uwe Kleine-König <[email protected]>Acked-by: Michal Simek <[email protected]>Link: https://lore.kernel.org/r/[email protected]Signed-off-by: Ulf Hansson <[email protected]>
pmdomain: xilinx: Move Kconfig option to the pmdomain subsystemThe Kconfig option belongs closer to the corresponding implementation,hence let's move it from the soc subsystem to the pmdomain subs
pmdomain: xilinx: Move Kconfig option to the pmdomain subsystemThe Kconfig option belongs closer to the corresponding implementation,hence let's move it from the soc subsystem to the pmdomain subsystem.Cc: Michal Simek <[email protected]>Signed-off-by: Ulf Hansson <[email protected]>
pmdomain: Rename the genpd subsystem to pmdomainIt has been pointed out that naming a subsystem "genpd" isn't veryself-explanatory and the acronym itself that means Generic PM Domain, isknown onl
pmdomain: Rename the genpd subsystem to pmdomainIt has been pointed out that naming a subsystem "genpd" isn't veryself-explanatory and the acronym itself that means Generic PM Domain, isknown only by a limited group of people.In a way to improve the situation, let's rename the subsystem to pmdomain,which ideally should indicate that this is about so called Power Domains or"PM domains" as we often also use within the Linux Kernel terminology.Suggested-by: Rafael J. Wysocki <[email protected]>Signed-off-by: Ulf Hansson <[email protected]>Reviewed-by: Linus Walleij <[email protected]>Acked-by: Arnd Bergmann <[email protected]>Acked-by: Heiko Stuebner <[email protected]>Acked-by: Rafael J. Wysocki <[email protected]>Acked-by: Geert Uytterhoeven <[email protected]>Link: https://lore.kernel.org/r/[email protected]