Re: [PATCH] drivers: char: omap-uart: add "clock-hz" option

[Julien Grall <julien@xxxxxxx>]

Hi Amneesh,

Sorry for the late answer.

On 06/08/2024 12:50, Amneesh Singh wrote:
> > need to specify the clockspeed on the command line. I think we should
> > investigate other approaches such as implementing partially
> > clk_get_rate() (if this is how Linux manage to retrieve the clock speed
> > without any command line option).
> I guess, we can just ditch the entire idea, and just use the a default
> fallback and just print a message informing the user that there is no
> frequency in the DT. I do not think implementing clk_get_rate (or any
> clock API) is feasible or worth the effort to be completely honest.
> What do you think? I do not particularly have any issues with that. I
> feel like this is a niche case (at least now) anyway.
Michal, Stefano and I had a chat. We agree that implementing 
clk_get_rate() may be too complex. So the best next solution is to ask 
the user to update the DT and read the property from Xen.
Cheers,

--
Julien Grall