Re: [PATCH] lib: devres: Mark devm_request_and_ioremap() as deprecated

From: 'Greg Kroah-Hartman'
Date: Wed May 14 2014 - 10:30:50 EST


On Wed, May 14, 2014 at 02:38:11PM +0900, Jingoo Han wrote:
> devm_request_and_ioremap() was obsoleted by devm_ioremap_resource().
> (7509657 "lib: devres: Introduce devm_ioremap_resource()") So,
> let's mark it as deprecated in order to stop using it.
>
> Suggested-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
> Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx>
> Cc: Thierry Reding <thierry.reding@xxxxxxxxx>
> ---
> include/linux/device.h | 4 ++--
> lib/devres.c | 1 +
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/include/linux/device.h b/include/linux/device.h
> index 67f003a..769c86b 100644
> --- a/include/linux/device.h
> +++ b/include/linux/device.h
> @@ -627,8 +627,8 @@ extern void *devm_kmemdup(struct device *dev, const void *src, size_t len,
> gfp_t gfp);
>
> void __iomem *devm_ioremap_resource(struct device *dev, struct resource *res);
> -void __iomem *devm_request_and_ioremap(struct device *dev,
> - struct resource *res);
> +void __deprecated __iomem *devm_request_and_ioremap(struct device *dev,
> + struct resource *res);

If no one uses it, just delete it entirely from the tree, no need to
mark __deprecated, that's not going to stop new people from using it.

Also, I see a number of existing users in Linus's tree, please remove
those usages as well.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/