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

From: Wolfram Sang
Date: Tue Jun 20 2023 - 05:04:33 EST



> 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.

> Care to respin this one after dropping the merge patch?

Sure, I'll respin this series right away

Attachment: signature.asc
Description: PGP signature