[PATCH v3 2/3] dt-bindings: pinctrl: Add aspeed,ast2700-soc1-pinctrl
Billy Tsai
billy_tsai at aspeedtech.com
Mon May 18 20:14:50 PDT 2026
> > + properties:
> > + function:
> > + enum:
> > + - ADC0
> > + - ADC1
> It'd be nice if you could use the other enum format I think so that
> there's not 700 lines taken up by functions/groups/pins.
I considered the inline format, but kept the one-item-per-line style
intentionally: with 200+ entries in each enum, any future addition or
rename produces a clean single-line diff. The inline format would
require re-wrapping the entire list on every change, making those diffs
significantly harder to review.
> Otherwise, I really don't like this approach but it seems to be standard
> on aspeed so whatever.
> Acked-by: Conor Dooley <conor.dooley at microchip.com>
> pw-bot: not-applicable
Thank you for the review and the ack.
Billy Tsai
More information about the linux-arm-kernel
mailing list