Messages in this thread | | | Date | Wed, 26 Jul 2023 20:35:01 +0200 | From | Stephan Gerhold <> | Subject | Re: [PATCH 1/2] clk: qcom: reset: Increase max reset delay |
| |
On Wed, Jul 26, 2023 at 08:24:33PM +0200, Konrad Dybcio wrote: > On 26.07.2023 20:21, Stephan Gerhold wrote: > > On Wed, Jul 26, 2023 at 03:26:19PM +0200, Konrad Dybcio wrote: > >> u8 limits us to 255 microseconds of delay. Promote the delay variable to > >> u16 to hold bigger values. > >> > >> Signed-off-by: Konrad Dybcio <konrad.dybcio@linaro.org> > > > > It would be clearer to change this together with an actual user that > > needs > 255us. AFAICT atm MSM8909 is the only user of this and it has > > just 15us. > Some LPASS resets ask for 500, but I'm still working on that driver. >
Maybe send it together with that driver then? This feels close to the typical "no API changes without also adding the user" rule.
Thanks, Stephan
| |