Re: [PATCH v5 4/8] dt-bindings: iio: imu: Add bindings for ADI ADIS16607

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1183 bytes --]

On Tue, Nov 03, 2026 at 12:15:50PM +0000, Sabau, Radu bogdan wrote:
> > -----Original Message-----
> > From: Krzysztof Kozlowski <krzk@kernel.org>
> > Sent: Tuesday, November 3, 2026 12:55 PM
> 
> ...
> 
> > > > The binding exposes optional properties for:
> > > >   - An external clock for sync mode (falling back to the internal 8 kHz
> > > >     clock when absent)
> > > >   - A hardware reset GPIO (active low)
> > > >
> > > > Signed-off-by: Radu Sabau <radu.sabau@analog.com>
> > >
> > > Why did you drop Rob's tag?
> > 
> > Explanation in cover letter is confusing. There is no difference between
> > v4 and v5. There is some description change betweeb v3 and v5. So was
> > the description change something forcing to re-review?
> > 
> 
> Perhaps the explanation can get a bit confusing around that "since v3".
> The quick history of it is that v3 to v4 saw a change in the GPIO description
> and so Jonathan suggested I should've dropped the tag in v4, therefore
> I did so in v5.

tbh, I don't really think that justified removal but w/e.
Since I can't re-provide rob's tag
Acked-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]