1 comment:
File src/soc/intel/alderlake/chip.c:
else if (dev->path.type == DEVICE_PATH_GPIO)
dev->ops = &soc_gpio_ops;
I agree on what Nico says. It's not a chip and we shouldn't fake it. I will try to move the struct.
Having a separate chip is helpful especially when you want to add some configs which are specific to that chip. You don't need to add all these configs to soc chip. I don't think there is a problem in providing a chip for the GPIO block. It also relieves each SoC from duplicating the same code to set up GPIO device ops.
To view, visit change 48583. To unsubscribe, or for help writing mail filters, visit settings.