Re: [PATCH 06/14] mfd: Initialise the DB8500 PRCMU driver at core_initcalltime

[Lee Jones]

On 11/06/12 22:01, Linus Walleij wrote:
> On Mon, Jun 11, 2012 at 5:24 PM, Lee Jones<lee.jones@xxxxxxxxxx>  wrote:
>
> > Now the AB8500 has its own IRQ domain
>
> But that does not appear until patch 8 in this series? Are the patches in
> the wrong order? Or does this need rewording?

The patches are in the correct order. This needs to be done _before_ we 
provide the AB8500 with its own domain. I guess the line should 
reference future tense. We are only talking two patches in the future, 
is it that important? I will change the wording slightly if it is.

> > it needs to be initialised earlier
> > in the boot sequence. As the AB8500 relies on the DB8500 PRCMU we need to
> > reflect this change for the PRCMU driver too.
>
> Hm what shall we do when we run out of initlevels? I think this was the
> kind of thing that deferred probe should solve. Usually changing this kind
> of thing has side effects so I'm a bit hesitant.

Nothing seems to be broken by it. My Snowball still comes up and 
everything that worked before continues to do so. How would you like me 
to take this forward?

--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
M: +44 77 88 633 515
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
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/