Re: [PATCH v2 2/2] devicetree: Add devicetree bindings documentation for Cadence SPI

From: Mark Brown
Date: Fri Apr 04 2014 - 10:45:25 EST


On Fri, Apr 04, 2014 at 07:38:14PM +0530, Harini Katakam wrote:

> OK. I understand.
> Can you comment on the case where a decoder is used?

> There is support for adding a decoder where extended slaves can be selected
> through the IP's control register.
> (This is not currently implemented in the driver but will be in the future.)

> How should the driver know whether it is 4 or 16 select lines for example?
> This has to be written to master->num_chipselect.

That's the sort of case where it's useful yes - depending on how the
implementation is done it may be sensible to just have a property
specifying if a decoder is there (if it just changes the encoding in the
register for example).

Attachment: signature.asc
Description: Digital signature