Re: [PATCH] ARM: vexpress: initial device tree support

Stephen Warren wrote:

Mitch Bradley wrote at Wednesday, January 11, 2012 4:16 PM:

quoted

Perhaps I'm missing something, but it appears to me that the model is to
set the correct GPIO state before each use, instead of a
save-set-use-restore model.

That's true, but the select action happens implicitly inside the I2C
core for any and all transactions, AIUI, so the two modes are equivalent.

If there is some reason why it would be desirable to deselect
the DDC I2C bus after transactions, that could be added as an option
to the GPIO I2C mux, but I can't think of a reason.

For an I2C mux that is controlled via I2C, you can just add the mux
node as a child of the I2C controller, since it has an I2C address,
and so putting it there makes sense.

Only if the mux's I2C slave is on the near side of the mux so that
it's always addressable regardless of mux state, and if it's on the
same I2C.

But for an I2C mux that's controlled using GPIOs or pinmux, there's no
I2C address so I guess the mux shouldn't be directly underneath the I2C
controller.

It's connected to: (a) an I2C, and (b) a GPIO.

It can't be child of both, but I don't see why the GPIO can't be
referred by phandle and the I2C it's muxing be the direct parent.
That seems more natural to me.

All the best,
-- Jamie