linux-kernel - Re: [RFC PATCH 1/4] dt-binding: i2c: add generic properties for GPIO bus recovery

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Sun, 5 Jul 2020 23:19:18 +0200
From:   Wolfram Sang <wsa@...nel.org>
To:     Codrin Ciubotariu <codrin.ciubotariu@...rochip.com>,
        Russell King <linux@...linux.org.uk>
Cc:     linux-i2c@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        robh+dt@...nel.org, ludovic.desroches@...rochip.com,
        nicolas.ferre@...rochip.com, alexandre.belloni@...tlin.com,
        linux@...linux.org.uk, kamel.bouhara@...tlin.com
Subject: Re: [RFC PATCH 1/4] dt-binding: i2c: add generic properties for GPIO
 bus recovery

> +- pinctrl
> +	add extra pinctrl to configure SCL/SDA pins to GPIO function for bus
> +	recovery, call it "gpio" or "recovery" state

I think we should stick with "gpio" only. That is what at91 and imx have
in their bindings. pxa uses "recovery" as a pinctrl state name but I
can't find any further use or documentation of that. PXA is not fully
converted to the best of my knowledge, so maybe it is no problem for PXA
to switch to "gpio", too? We should ask Russell King (cced).

Russell, do you object naming the pinctrl state for bus recovery in
the pxa i2c driver from "recovery" to "gpio"?

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)