Re: [PATCH 3/2] checkpatch: document the function renaming and deprecation around devm_ioremap_resource
From: Joe Perches
Date: Fri Nov 13 2020 - 11:36:49 EST
On Fri, 2020-11-13 at 10:11 +0100, Uwe Kleine-König wrote:
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>
> ---
> Hello,
>
> this can also be squashed into the respective patches instead.
>
> Best regards
> Uwe
>
> scripts/checkpatch.pl | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
[]
> @@ -615,6 +615,11 @@ our %deprecated_apis = (
> "rcu_barrier_sched" => "rcu_barrier",
> "get_state_synchronize_sched" => "get_state_synchronize_rcu",
> "cond_synchronize_sched" => "cond_synchronize_rcu",
> + "devm_platform_get_and_ioremap_resource" => "devm_platform_get_request_and_ioremap_resource",
Do we really need 46 character length function names?
> + "devm_platform_ioremap_resource" => "devm_platform_request_ioremap_resource",
> + "devm_platform_ioremap_resource_wc" => "devm_platform_request_ioremap_resource_wc",
> + "devm_ioremap_resource" => "devm_request_ioremap_resource",
> + "devm_ioremap_resource_wc" => "devm_request_ioremap_resource_wc",
> );
>
>
> #Create a search pattern for all these strings to speed up a loop below
And do please send your proposed patches to the appropriate maintainers.