Hi,
> > > + addr <= GPIO_MONID + 2 && addr + size > GPIO_MONID + 2) {
> >
> > Hmm, isn't this just "addr == GPIO_MONID + 2" ?
>
> No because there could be all kinds of unalligned or multibyte access and we
> only want to trigger this when the byte with the enable bits are touched.
> (The MacOS NDRV accesses this 1 byte at a time so this is needed to avoid
> spurious i2c bit banging but other drivers write 4 bytes so then addr is not
> equal but covering above byte which is what this test allows.
Can you add a comment explaining this (no need to respin, incremental patch is
fine)?
thanks,
Gerd
