On 09/11/2022 17:26, Nicholas Roth wrote: > Happy to reply inline next time. I'm still getting used to this format > :-). Here's the context around my clock frequency question-- I'd > really like to understand this better: > >>> + clock-names: >>> + description: >>> + Input clock for the sensor. >>> + items: >>> + - const: xvclk >>> + >>> + clock-frequency: >>> + description: >>> + Frequency of the xvclk clock in Hertz. >> >> The frequency of clock should go via common clock framework - you have >> get_rate and set_rate. Drop entire property. > > I am trying to be consistent with the ov8856 driver and bindings but > would be happy to change. I’m not familiar with that framework though. > Is there somewhere I could read about this, including the driver and > device-tree changes I need to use this? git grep clk_get_rate -- drivers/media/i2c/ For example imx219 or imx334. Best regards, Krzysztof
