Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1417314 > unrolled thread

Re: [PATCH 4/4] mfd: db8500 stop passing around platform data

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH 4/4] mfd: db8500 stop passing around platform data Linus Walleij <linus.walleij@linaro.org> - 2016-06-08 13:40 +0200
    Re: [PATCH 4/4] mfd: db8500 stop passing around platform data Arnd Bergmann <arnd@arndb.de> - 2016-06-08 18:00 +0200
      Re: [PATCH 4/4] mfd: db8500 stop passing around platform data Lee Jones <lee.jones@linaro.org> - 2016-06-09 17:00 +0200

#1417314 — Re: [PATCH 4/4] mfd: db8500 stop passing around platform data

On Tue, May 10, 2016 at 4:02 PM, Arnd Bergmann <arnd@arndb.de> wrote:

> Except for the constant DB8500_PRCMU_FW_VERSION_OFFSET number, nothing
> is ever passed through the platform data and used in a driver, so we
> can simply stop passing it around.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Do you think it's relevant for Lee to take this or do you want to
have it in ARM SoC together with the other cleanups (that I
haven't had time to look at...)?

In the latter case I need an ACK from Lee to apply it for a pull
request to ARM SoC.

The functionality it pertains to is out-of-tree but if we want to
reintroduce it, it should happen through DT anyway.

Yours,
Linus Walleij

[toc] | [next] | [standalone]

#1417609

On Wednesday, June 8, 2016 1:37:12 PM CEST Linus Walleij wrote:
> On Tue, May 10, 2016 at 4:02 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> 
> > Except for the constant DB8500_PRCMU_FW_VERSION_OFFSET number, nothing
> > is ever passed through the platform data and used in a driver, so we
> > can simply stop passing it around.
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> 
> Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> 
> Do you think it's relevant for Lee to take this or do you want to
> have it in ARM SoC together with the other cleanups (that I
> haven't had time to look at...)?
> 
> In the latter case I need an ACK from Lee to apply it for a pull
> request to ARM SoC.
> 
> The functionality it pertains to is out-of-tree but if we want to
> reintroduce it, it should happen through DT anyway.

I think most of the changes are in mfd, so it would be appropriate
for Lee to take them all, but I could also put them into arm-soc
if he prefers.

	Arnd

[toc] | [prev] | [next] | [standalone]

#1418409

On Wed, 08 Jun 2016, Arnd Bergmann wrote:

> On Wednesday, June 8, 2016 1:37:12 PM CEST Linus Walleij wrote:
> > On Tue, May 10, 2016 at 4:02 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> > 
> > > Except for the constant DB8500_PRCMU_FW_VERSION_OFFSET number, nothing
> > > is ever passed through the platform data and used in a driver, so we
> > > can simply stop passing it around.
> > >
> > > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > 
> > Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
> > 
> > Do you think it's relevant for Lee to take this or do you want to
> > have it in ARM SoC together with the other cleanups (that I
> > haven't had time to look at...)?
> > 
> > In the latter case I need an ACK from Lee to apply it for a pull
> > request to ARM SoC.
> > 
> > The functionality it pertains to is out-of-tree but if we want to
> > reintroduce it, it should happen through DT anyway.
> 
> I think most of the changes are in mfd, so it would be appropriate
> for Lee to take them all, but I could also put them into arm-soc
> if he prefers.

Up to you.  If you want a pull-request, I can provide you with one.
If you are happy to 'risk it', I'm good with that too.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web