[PATCH 2/4] mfd: dbx500-prcmu: Correctly reorder PRCMU clock identifiers

Linus Walleij linus.walleij at linaro.org
Fri Jun 7 09:13:19 EDT 2013


On Thu, Jun 6, 2013 at 1:21 PM, Lee Jones <lee.jones at linaro.org> wrote:

> ... as stipulated by the Hardware Specification document.
>
> Signed-off-by: Lee Jones <lee.jones at linaro.org>

I think you need Ulf (as ux500 clock author) and Loic (as PRCMU
developer) to have a look at this.

> -enum prcmu_clock {
> -       PRCMU_SGACLK,
(...)
> +#define ARMCLK                 0
> +#define PRCMU_ACLK             1
(...)

It seems the enum and the defines want to do two different things?

The first is a kernel-internal representation of the clocks, whereas
the latter is a HW-centric representation for a certain PRCMU
variant (I guess?)

Does one actually exclude the other?

Yours,
Linus Walleij



More information about the linux-arm-kernel mailing list