Re: [RFC PATCH 2/5] gnss: ubx: use new helper to remove open coded regulator handling

[Johan Hovold]

On Tue, Jun 20, 2023 at 11:04:27AM +0200, Wolfram Sang wrote:
> > I'm a bit torn about this one as I'm generally sceptical of devres and
> > especially helpers that enable or register resources, which just tends to
> > lead to subtle bugs.
> 
> It is good to think twice with devres, but I also really like this
> helper. En-/Disabling the regulator matches the life cycle of the device
> itself. The boilerplate code it removes tends also to be error prone.

So can the "trival" devres conversions be:

	https://lore.kernel.org/lkml/ZJFqCQ8bbBoX3l1g@xxxxxxxxxxxxxxxxxxxx

I meant to CC you on my reply there.

Johan

Attachment: signature.asc
Description: PGP signature