Re: [PATCH] clockevents: asm9260: Fix compilation error with sparc/sparc64 allyesconfig

From: Guenter Roeck
Date: Mon Feb 23 2015 - 15:49:36 EST


On Mon, Feb 23, 2015 at 09:46:51PM +0100, Oleksij Rempel wrote:
> Am 23.02.2015 um 21:34 schrieb Guenter Roeck:
> > On Mon, Feb 23, 2015 at 09:27:01PM +0100, Oleksij Rempel wrote:
> >> Am 23.02.2015 um 20:10 schrieb Guenter Roeck:
> >>> On Mon, Feb 23, 2015 at 08:00:51PM +0100, Daniel Lezcano wrote:
> >>>> The Kconfig options for the asm9260 timer is wrong as it can be selected by
> >>>> another platform with allyes config and thus leading to a compilation failure
> >>>> as some non arch related code is pulled by the compilation.
> >>>>
> >>>> Fix this by having the platform Kconfig to select the timer as it is done for
> >>>> the others drivers.
> >>>>
> >>>> Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
> >>>
> >>> Acked-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> >>>
> >>
> >> the thing is, this SoC or this company has no own IP cores. All of them
> >> are from other companies and providers, including this timer. This is
> >> why all modules for this SoC are selectable.
> >>
> > Maybe the modules should be selected by the architectures using the IP cores ?
>
> At the moment of pushing patches to arch/arm, maintainers preferred to
> have minimalistic Kconfig.
>
> > Either case, we'll need _some_ fix upstream, whatever is acceptable for the
> > maintainer.
>
> What is with initial white list patch?
>
I don't know. All I know is that the upstream kernel still has the problem,
and that it affects both sparc64-allmodconfig and sparc-allmodconfig.

Guenter
--
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/