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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <54EB99CE.1030806@linaro.org>
Date:	Mon, 23 Feb 2015 22:21:18 +0100
From:	Daniel Lezcano <daniel.lezcano@...aro.org>
To:	Oleksij Rempel <linux@...pel-privat.de>,
	Guenter Roeck <linux@...ck-us.net>
CC:	linux-kernel@...r.kernel.org, tglx@...utronix.de
Subject: Re: [PATCH] clockevents: asm9260: Fix compilation error with sparc/sparc64
 allyesconfig

On 02/23/2015 09:46 PM, 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@...aro.org>
>>>>
>>>> Acked-by: Guenter Roeck <linux@...ck-us.net>
>>>>
>>>
>>> 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?

The Kconfig policy for the timers is to let the arch to select the timer 
they need and not enable them from the clocksource/Kconfig file. That is 
the general policy even there are a couple of exceptions.

So if you are ok with the patch I sent I will merge it as a fix.


-- 
  <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/