On Fri, 2026-01-23 at 17:03 +1030, Andrew Jeffery wrote:
Going down this path for pinctrl to fix the SCU situation will require
some rework of what's already merged for the AST2700. However, we've
not yet merged either a DTS or DTSI using the compatibles (and by
extension, aren't using the AST2700 support from the drivers), so I
hope that allows us to do a course-correction without too much
collateral damage.
A possible path forward is to:
* Move AST2700 definitions out of mfd/aspeed,ast2x00-scu.yaml into one
of:
- soc/aspeed/aspeed,ast2700-scu.yaml: Follow the example of
mobileye,eyeq5-olb?
- arm/aspeed/aspeed,ast2700-scu.yaml: We already have e.g. the secure
boot controller documented under arm/aspeed
* Retain compatible strings but require simple-mfd is not specified
* Rework AST2700 support introduced in:
- drivers/irqchip/irq-aspeed-scu-ic.c
- drivers/soc/aspeed/aspeed-socinfo.c
... but perhaps that is too much churn for this stage.
Krzysztof: If you think the rework is worthwhile, let me know and we
can look at that design, otherwise I think Billy can proceed by
incorporating your current feedback for a v4.
Andrew