On 04-01-24, 13:56, Konrad Dybcio wrote: > > > On 4.01.2024 13:53, Konrad Dybcio wrote: > > > > On 3.01.2024 11:48, Viresh Kumar wrote: > >> The OPP core needs to take care of a special case, where the OPPs aren't > >> available for a device, but in order to keep the same unified interface > >> for the driver, the same OPP core API must take care of performing a > >> simple clk_set_rate() for the device. > >> > >> This required the extra argument, but that is used only within the OPP > >> core and the drivers don't need to take care of that. > >> > >> Simplify the external API and handle it differently within the OPP core. > >> > >> This shouldn't result in any functional change. > > Hi, so this apparently breaks serial on Qualcomm platforms using > > "qcom,geni-debug-uart".. I'm seeing garbage on the console, likely > > meaning that ratesetting wasn't done. > > +CC Bjorn, Dmitry > > Probably also worth noting it only happens when an OPP table is present > in the device tree. Found the issue. Dropped the patch for now. Not sure if there is a clean way of handling it right now. -- viresh
