Messages in this thread | | | Subject | Re: [PATCH 0/4] nvmem: meson: efuse updates | From | Srinivas Kandagatla <> | Date | Mon, 12 Nov 2018 13:08:49 +0000 |
| |
On 30/10/18 10:22, Jerome Brunet wrote: > The first change of this patchset just adds add error message in case > of failure. If there is problem with the secure monitor, the > SM_EFUSE_USER_MAX call will be first one to fail so it is better if it > give us a clue to help debugging, instead af silently failing. > > Next this series adds the peripheral clock missing in this driver. > Like many other device in amlogic's SoC, the efuse requires a > peripheral clock to operate. ATM, the clock controller has > CLK_IGNORE_UNUSED on this clock and we have been lucky enough that the > bootloader left the clock enabled > > At some point, we would like to remove those CLK_IGNORE_UNUSED, so if a > driver needs a clock, it needs to properly claim it. > > Srinivas, Kevin, > The dts change needs to land before the actual driver change, to avoid > breaking the efuse on our users. If there an agreement on this series, > Kevin could you provide a tag to Srinivas ?
These are not fixes to any bugs/regressions, so its new material which can only go in next dev cycle!
I guess that should also address the patch sequencing issue!
Also I need ack from dt-maintaners on clk bindings to pick patch 2 and 4.
thanks, srini
> > Cheers > Jerome
| |