Re: Why do we need reset_control_get_optional() ?

From: Masahiro Yamada
Date: Tue Aug 16 2016 - 05:41:45 EST


Hi Arnd,



2016-08-06 0:35 GMT+09:00 Arnd Bergmann <arnd@xxxxxxxx>:

>
> config RESET_FOO
> bool "FOO reset controller" if COMPILE_TEST && !ARCH_FOO
> default ARCH_FOO
>
> then I think we get both: you won't be able to turn it off
> but also get the build testing.


This looks good to me, too.
I do not know if we need "&& !ARCH_FOO", though.

When we are compile-testing, we do not care if it works on run-time,
so perhaps it makes sense to allow to enable/disable RESET_FOO
regardless ARCH_FOO.


Philipp also agreed, so I guess the permission for this rework
is granted.
Would you do it?



--
Best Regards
Masahiro Yamada