On Fri, Sep 5, 2025 at 12:02 PM Dmitry Torokhov
[off-list ref] wrote:

On Thu, Aug 21, 2025 at 01:56:24PM +0200, Linus Walleij wrote:

quoted

Hi Ariel,

thanks for your patch!

On Wed, Aug 20, 2025 at 7:17 PM Ariel D'Alessandro
[off-list ref] wrote:

quoted

+  ce-gpios:
+    description: GPIO connected to the CE (chip enable) pin of the chip
+    maxItems: 1

Mention that this should always have the flag GPIO_ACTIVE_HIGH
as this is required by the hardware.

Unfortunately we have no YAML syntax for enforcing flags :/

Theoretically there can be an inverter on the line, so from the AP point
of view the line is active low while from the peripheral POV the pin is
active high...

Yes, I think someone even proposed adding inverters to the
device tree and was nixed.

It's a matter of phrasing I would say:

"Mention that this should nominally have the flag GPIO_ACTIVE_HIGH
as this is required by the hardware."

s/always/nominally/g

Yours,
Linus Walleij