On Wed, Mar 09, 2022 at 04:19:31PM +0000, John Keeping wrote:
On Wed, Mar 09, 2022 at 03:56:12PM +0000, Mark Brown wrote:
I'm still not seeing the functional change here. The actual state of the GPIO is identical in both cases, all that's changing is the logical view internally to the kernel.
Ah, sorry, I'm considering it functional since it changes the device tree ABI.
Used with the same device tree with, say, GPIO_ACTIVE_HIGH the physical state of the GPIO will change as a result of this patch and the device tree needs to be updated to use GPIO_ACTIVE_LOW.
I think the device tree binding needs to be clarified here to be explicit about this since there's obviously some room for user confusion here. We can probably get away with a change at this point since it's not hit a release but we do need to try to avoid the situation where any other implementations use active high polarity for the bindings.