[U-Boot] [PATCH v6 00/15] Add PSCI support for Jetson TK1/Tegra124 + CNTFRQ fix

Stephen Warren
2015-04-14T14:21:47Z

On 04/14/2015 08:12 AM, Jan Kiszka wrote:
> On 2015-04-14 16:06, Stephen Warren wrote:
>> On 04/14/2015 07:46 AM, Tom Rini wrote:
>>> On Mon, Apr 13, 2015 at 06:48:05AM +0200, Jan Kiszka wrote:
>>>
>>>> Changes in v6:
>>>> - rebased over master
>>>> - included Thierry's SMMU enabling patch
>>>> - moved activation patch at the end so that it can be held back
>>>>
>>>> This version can also be found at
>>>> https://github.com/siemens/u-boot/tree/jetson-tk1-v6.
>>>
>>> So what level of coordination do we need on applying this series so that
>>> kernels (both old and new) can continue to function? And perhaps README
>>> updates or similar? Thanks!
>>
>> Hopefully this series doesn't change anything by default, and simply
>> allows people to turn on support for booting kernels in non-secure mode
>> if they want to? If so, there shouldn't be any co-ordination required.
>> If it changes the default behaviour, co-ordination is probably required,
>> and that'd be a bad thing.
>
> Sorry, forgot to mention: I can't flip the default behaviour to leave
> virtualization support off only for the TK1. That's a generic default.
>
> So we can either leave out the last patch for now and wait until the
> cpu-idle fix is upstream or tell the people picking upstream u-boot to
> turn either virt support in u-boot on or CPU_IDLE in the kernel off
> (just tell me where to put such a note).

We should definitely not enable support by default if it requires kernel
changes. If someone wants to use the feature, they should edit the
source to enable it.