Re: [PATCH V2] cpuidle: dt: bail out if the idle-state DT node is not compatible

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 01, 2019 at 10:16:39AM +0800, Joseph Lo wrote:
> Currently, the DT of the idle states will be parsed first whether it's
> compatible or not. This could cause a warning message that comes from if
> the CPU doesn't support identical idle states. E.g. Tegra186 can run
> with 2 Cortex-A57 and 2 Denver cores with different idle states on
> different types of these cores.
> 
> So fix it by checking the match node earlier, then it can make sure it
> only goes through the idle states that the CPU supported.
> 
> Signed-off-by: Joseph Lo <josephl@xxxxxxxxxx>
> ---
> v2:
>  * fix it by moving of_match_node earlier, so we don't need to match the
>    node twice like the v1 did.

Neat, thanks for the quick patch. FWIW,

Reviewed-by: Sudeep Holla <sudeep.holla@xxxxxxx>

--
Regards,
Sudeep



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux