Change in ...coreboot[master]: superio: Add ASpeed AST2400

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/23135 )

Change subject: superio: Add ASpeed AST2400
......................................................................

Patch Set 23:

(2 comments)

https://review.coreboot.org/#/c/23135/23/src/superio/aspeed/ast2400/superio.... 
File src/superio/aspeed/ast2400/superio.c:

https://review.coreboot.org/#/c/23135/23/src/superio/aspeed/ast2400/superio.... 
PS23, Line 49: &ops
these should be replaced with NULL, since all LDNs use &ops passed to pnp_enable_devices below and no special per-LDN override is needed in the pnp_info struct

https://review.coreboot.org/#/c/23135/23/src/superio/aspeed/ast2400/superio.... 
PS23, Line 55: 	{ &ops, AST2400_GPIO,     PNP_IRQ0, },
the GPIO LDN doesn't have an IO region?