Thread (14 messages) 14 messages, 3 authors, 2014-08-29

[PATCH 4/4] gpio: pca953x: Add DT binding for reset gpio

From: p.zabel@pengutronix.de (Philipp Zabel)
Date: 2014-08-08 14:11:51
Also in: linux-gpio

Am Freitag, den 08.08.2014, 15:14 +0200 schrieb Linus Walleij:
On Tue, Jul 29, 2014 at 9:24 AM, Markus Pargmann [off-list ref] wrote:
quoted
The pca953x has a negated reset input. This patch adds a DT binding for
the reset gpio and resets the chip when it is probed. This will reset
the device and leave the gpio in the correct state so reset is not
triggered.

Signed-off-by: Markus Pargmann <redacted>
Why on earth should this be in the GPIO driver?

The driver should be in drivers/reset/reset-gpio.c and you
should provide a separate driver for it.
I still think we should keep using the reset-gpios binding for simple
cases like this; I see no reason to add a separate device to the device
tree for a single GPIO.
As it happens, Houcheng Lin has already proposed such a
driver:
http://marc.info/?l=linux-kernel&m=140309916607115&w=2
That is a different issue, as there the device does not appear on the
bus until the reset is released.

Here the I2C device will be probed from the device tree, so the reset
can be released or triggered from the probe function.

regards
Philipp
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help