Re: [PATCH v6 1/2] dt-bindings: iio: light: add as7343

[Jose A. Perez de Azpillaga]

On 10/13/26 1:56 PM, Chang Yu wrote:
[snip]
> +
> +  reg:
> +    description:
> +      I2C address of the device (0x39).
> +    maxItems: 1

on v4 Jonathan said "Might as well specify it as const given it can only
take that value. They you can drop the description as it won't add
anything." v6 still has the old form, looks like that got lost.

[snip]> +
> +  interrupts:
> +    description:
> +      Open drain output active low interrupt pin.
> +    maxItems: 1
> +
> +  vdd-supply: true
> +
> +  led-max-microamp:
> +    description:
> +      The driver current for the external LED connected to the LDR pin.

this wording dates from when it was ams,led-current-microamp.
leds/common.yaml defines led-max-microamp as a maximum ("risk of
hardware damage in case an excessive current is set"), i.e. the board
limit for LED_DRIVE, not the current program. please describe it as a
maximum.

-- 
cheers,
jose a. p-a