On Wed, 2025-01-29 at 18:03 +0100, Cédric Le Goater wrote: > On 1/21/25 08:04, Jamin Lin wrote: > > The design of the INTC has significant changes in the AST2700 A1. > > In the > > AST2700 A0, there was one INTC controller, whereas in the AST2700 > > A1, > > there were two INTC controllers: INTC0 (CPU DIE) and INTC1 (I/O > > DIE). > > > > The previous INTC model only supported the AST2700 A0 and was > > implemented for > > the INTC0 (CPU DIE). To support the future INTC1 (I/O DIE) model, > > rename INTC > > to INTC0. > > > Why not introduce definitions with _INTC_IO_ and leave alone the > current > instead ? Do we expect to have more than 2 INTC controllers ? >
There was similar discussion on the devicetree bindings for the SCU a while back: https://lore.kernel.org/all/94efc2d4ff280a112b869124fc9d7e35ac031596.ca...@codeconstruct.com.au/ Ryan didn't like deviating from their internal documentation :( Andrew
