devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] i2c: mux: Add more warnings to i2c-arb-gpio-challenge docs
@ 2014-04-29 17:30 Doug Anderson
  2014-05-21 10:29 ` Wolfram Sang
  0 siblings, 1 reply; 2+ messages in thread
From: Doug Anderson @ 2014-04-29 17:30 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: olof, Doug Anderson, robh+dt, pawel.moll, mark.rutland,
	ijc+devicetree, galak, rdunlap, devicetree, linux-doc,
	linux-kernel

This adds some more warnings to the i2c-arb-gpio-challenge docs to
help encourage people not to use it in their designs unless they have
no choice.

Signed-off-by: Doug Anderson <dianders@chromium.org>
---
 Documentation/devicetree/bindings/i2c/i2c-arb-gpio-challenge.txt | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/i2c/i2c-arb-gpio-challenge.txt b/Documentation/devicetree/bindings/i2c/i2c-arb-gpio-challenge.txt
index 1ac8ea8..bfeabb8 100644
--- a/Documentation/devicetree/bindings/i2c/i2c-arb-gpio-challenge.txt
+++ b/Documentation/devicetree/bindings/i2c/i2c-arb-gpio-challenge.txt
@@ -8,6 +8,12 @@ the standard I2C multi-master rules.  Using GPIOs is generally useful in
 the case where there is a device on the bus that has errata and/or bugs
 that makes standard multimaster mode not feasible.
 
+Note that this scheme works well enough but has some downsides:
+* It is nonstandard (not using standard I2C multimaster)
+* Having two masters on a bus in general makes it relatively hard to debug
+  problems (hard to tell if i2c issues were caused by one master, another, or
+  some device on the bus).
+
 
 Algorithm:
 
-- 
1.9.1.423.g4596e3a


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] i2c: mux: Add more warnings to i2c-arb-gpio-challenge docs
  2014-04-29 17:30 [PATCH] i2c: mux: Add more warnings to i2c-arb-gpio-challenge docs Doug Anderson
@ 2014-05-21 10:29 ` Wolfram Sang
  0 siblings, 0 replies; 2+ messages in thread
From: Wolfram Sang @ 2014-05-21 10:29 UTC (permalink / raw)
  To: Doug Anderson
  Cc: olof, robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak,
	rdunlap, devicetree, linux-doc, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 416 bytes --]

On Tue, Apr 29, 2014 at 10:30:51AM -0700, Doug Anderson wrote:
> This adds some more warnings to the i2c-arb-gpio-challenge docs to
> help encourage people not to use it in their designs unless they have
> no choice.
> 
> Signed-off-by: Doug Anderson <dianders@chromium.org>

Woha, you forgot to add the i2c list. It nearly slipped through.
Luckily, I found it nonetheless :) Applied to for-next, thanks!


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-05-21 10:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-29 17:30 [PATCH] i2c: mux: Add more warnings to i2c-arb-gpio-challenge docs Doug Anderson
2014-05-21 10:29 ` Wolfram Sang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).