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

[Daniel Lezcano]

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 |
<http://www.linaro.org/linaro-blog/> Blog