Re: [PATCH v8 2/2] clocksource: Add Intel Keem Bay timer support

Daniel Lezcano
2022-03-02T10:24:26Z

On 02/03/2022 11:12, Sanil, Shruthi wrote:

[ ... ]

>>> + if (!(val & TIM_CONFIG_PRESCALER_ENABLE)) {
>>> + pr_err("%pOF: Prescaler is not enabled\n", np);
>>> + ret = -ENODEV;
>>> + }
>>
>> Why bail out instead of enabling the prescalar ?
>
> Because it is a secure register and it would be updated by the bootloader.
Should it be considered as a firmware bug ?

--
<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 |