zephyr.js
zephyr.js copied to clipboard
intel
The GPIO “pullup”/”pulldown” feature doesn’t work on IO2, IO4, IO7-IO8 pins
The 'pull' for pullup/pulldown resistors is implemented but the feature doesn't work on IO2, IO4, IO7-IO8 pins. The other pins are not sure whether the pull feature are workable.
I've tried again and no it doesn't seem to work for me either. I only tried IO3/4 so far.
Later I found that the Zephyr QMSI (and older DW) GPIO drivers never even use the GPIO_PUD_PULL_UP flag, while some other drivers like K64F do. So it can't possibly work currently - I'm not sure if that's because the hardware doesn't support it or just the driver. There is some reference to pullup for A101 within the pinmux driver though, so there might be an alternate way to enable it. (No reference to pulldown there though.)
So basically, unless we get so serious about this particular issue that we attempt to go add the support to Zephyr ourselves (and figure out whether it's actually supported in the A101 hardware), it's just not going to work until they get around to it.
We're starting to run into this in other places: for example, it appears there is no I2C driver for K64F. So we're probably going to need to get used to missing Zephyr support and figure out what we're going to do about it. Unless we run out of work that's clearly "ours", I don't think writing Zephyr drivers is the answer. So we need to either clearly document where support is missing, or report it in the code with errors, or both.
I'm in the process of adding API documentation so I guess adding a section to each page on hardware support caveats would make sense.
Hmm I still haven't opened a Zephyr bug but talked to ktseng on IRC and that's what he told me to do. I should check first on a recent Zephyr commit.
I found that zephyrproject-rtos/zephyr#2288 may shed some light on this issue; it appears that for A101 at least, it is pinmux driver where pullups are controlled.
Lowering priority because this is A101-specific and pullup has been observed to work on other boards.