Re: [PATCH v4 2/4] dt-bindings: hwmon: Add Sensirion SHT30 series

[Conor Dooley <conor@kernel.org>]

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

On Thu, Mar 26, 2026 at 05:42:30PM +0000, Conor Dooley wrote:
> On Wed, Mar 25, 2026 at 06:05:22PM -0700, Guenter Roeck wrote:
> > On 3/25/26 11:20, Conor Dooley wrote:
> > > On Wed, Mar 25, 2026 at 05:08:08PM +0800, Zaixiang Xu wrote:
> > > > Add YAML devicetree binding schema for Sensirion SHT30 series.
> > > > Use fallback compatibles for compatible chips and add optional
> > > > interrupts and vdd-supply properties.
> > > > 
> > > > Reported-by: kernel test robot <lkp@intel.com>
> > > > Closes: https://lore.kernel.org/r/202603212044.BRPaiz86-lkp@intel.com/
> > > 
> > > The robot did not report that this binding was missing.
> > > It also told you not to add these tags.
> > > 
> > > You also ignored my and Krzysztof's reviews.
> > > 
> > > NAK.
> > > 
> > 
> > Maybe we should just point to AI feedback:
> > 
> > https://sashiko.dev/#/patchset/1774429690-129139-1-git-send-email-zaixiang.xu.dev%40gmail.com
> > 
> > and only get involved after AI does not report any problems.
> > 
> 
> The presentation of info in that is weird, it creates a pseudo-commit
> message, and then goes on to talk about things that the pseudo-commit
> message has had culled.

How good is this LLM stuff at figuring out if previous review feedback
has been resolved? Or is it not capable of looking at earlier revisions?

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