[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHp75VcLLjjc5T+Qq8du4QgnQ-rtzFLh=5Jt-DPjigb+QUWOSg@mail.gmail.com>
Date: Fri, 10 Apr 2020 13:55:26 +0300
From: Andy Shevchenko <andy.shevchenko@...il.com>
To: Alexandre Belloni <alexandre.belloni@...tlin.com>
Cc: Aishwarya R <aishwaryarj100@...il.com>,
Jonathan Cameron <jic23@...nel.org>,
Hartmut Knaack <knaack.h@....de>,
Lars-Peter Clausen <lars@...afoo.de>,
Peter Meerwald-Stadler <pmeerw@...erw.net>,
Nicolas Ferre <nicolas.ferre@...rochip.com>,
Ludovic Desroches <ludovic.desroches@...rochip.com>,
Thomas Gleixner <tglx@...utronix.de>,
Allison Randal <allison@...utok.net>,
Kefeng Wang <wangkefeng.wang@...wei.com>,
Stephen Boyd <swboyd@...omium.org>,
linux-iio <linux-iio@...r.kernel.org>,
linux-arm Mailing List <linux-arm-kernel@...ts.infradead.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] iio: adc: at91-adc: Use devm_platform_ioremap_resource
On Thu, Apr 9, 2020 at 7:00 PM Alexandre Belloni
<alexandre.belloni@...tlin.com> wrote:
> On 09/04/2020 20:41:23+0530, Aishwarya R wrote:
> > Use the helper function that wraps the calls to
> > platform_get_resource() and devm_ioremap_resource()
> > together.
> Please elaborate the actual value of doing that.
Please, elaborate actual value of not doing that.
Yes, I know that you are p* off of these changes, but why you not
going further and forbid all clean ups we are doing in the code?
To the point. Above change is reducing code base and showing the new
comers modern APIs to use.
--
With Best Regards,
Andy Shevchenko
Powered by blists - more mailing lists