Re: [PATCH v5] iio: magnetometer: rm3100: Modernize locking and refactor control flow
From: Jonathan Cameron
Date: Wed May 06 2026 - 14:34:44 EST
On Wed, 06 May 2026 10:28:29 -0500
"Maxwell Doose" <m32285159@xxxxxxxxx> wrote:
> On Wed May 6, 2026 at 9:43 AM CDT, Jonathan Cameron wrote:
> > On Tue, 5 May 2026 12:05:11 -0500
> > Maxwell Doose <m32285159@xxxxxxxxx> wrote:
> >
> >> On Tue, May 5, 2026 at 7:45 AM Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
> >> >
> >> > Applied to the testing branch of iio.git with that tweak.
> >> >
> >> > Thanks,
> >> >
> >> > Jonathan
> >> >
> >>
> >> Hey Jonathan, I just noticed that I made a mistake in the commit
> >> message, it says "Add new helper-wrapper function
> >> rm3100_guarded_regmap_bulk_read()..." when it really should be "Add
> >> new helper-wrapper function rm3100_regmap_bulk_read_locked()..." I'm
> >> unfortunately not in a position to fix it myself since I'm away, but I
> >> just wanted to point it out so it could be fixed.
> > fixed up.
> >>
> >> best regards,
> >> max
>
> Gosh, I'm so sorry to keep bothering you, but I noticed a stray change
> here:
>
> @@ -7,7 +7,7 @@
> * User Manual available at
> * <https://www.pnicorp.com/download/rm3100-user-manual/>
> *
> - * TODO: event generation, pm.
> + * TODO: event generation, pm.da
> */
>
> wishing vs code would quit messing with me,
> max
Doh. That wasn't there in the original so I must have messed up.
Good spot.
Fixed up.