Re: [PATCH 2/2] ARM: multi_v7_defconfig: Don't attempt to enable schedutil governor as module

From: Javier Martinez Canillas
Date: Fri Aug 19 2016 - 09:46:42 EST


Hello Krzysztof,

On 08/19/2016 09:40 AM, Krzysztof Kozlowski wrote:
> On 08/19/2016 03:12 PM, Javier Martinez Canillas wrote:
>> The schedutil CPUFreq governor could be built as a module but the commit
>> 58919e83c85c ("cpufreq / sched: Pass flags to cpufreq_update_util()")
>> made it non-modular so attempting to enable as module leads to a warning:
>>
>> warning: symbol value 'm' invalid for CPU_FREQ_GOV_SCHEDUTIL
>>
>> Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
>>
>> ---
>>
>> arch/arm/configs/multi_v7_defconfig | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Looks good.
>
> Applied... wait, no, these are not in samsung-soc list so they miss also
> from patchwork. For reference:
> Reviewed-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx>
>

Sigh, get_maintainers.pl suggested me an insanely large cc list so I added
it manually... but I forgot linux-samsung-soc. I'll post v2.

> Please post it to linux-samsung-soc, unless it goes through arm-soc or
> other tree.
>

I think is OK if both patches go through your tree.

> Best regards,
> Krzysztof
>

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America