Re: [PATCH 5/9] drivers: gpio: add QIXIS FPGA GPIO controller
From: Andrew Lunn <andrew@lunn.ch>
Date: 2025-07-10 14:12:16
Also in:
linux-devicetree, linux-gpio, lkml
From: Andrew Lunn <andrew@lunn.ch>
Date: 2025-07-10 14:12:16
Also in:
linux-devicetree, linux-gpio, lkml
The FPGA does have an ID register that we could verify and match against the board type that we expect. On the other hand, I am not 100% on board with the idea to check this from the GPIO driver which teoretically should only touch its one register. Maybe from the parent's driver we could do that and prevent the probing of children if things don't match up. But this does prove to be complicated since those drivers are simple-mfd (for LS1046AQDS) and simple-mfd-i2c (for LX2160ARDB). And I don't think it would be wise to add some specific board logic into of/platform.c.
My experience is, DT authors will mess up and put in the wrong compatible. And the wrong compatible might be enough for it to mostly work, so you end up with deployed systems with wrong compatibles. It then becomes difficult to actually extend the use of the compatible, without causing regressions. Also, checking will catch putting the wrong bitstream into the FPGA. So if you can check it, do check it, and return -ENODEV. Andrew