[PATCH v2] spi: orion.c: Add direct access mode

Mark Brown broonie at kernel.org
Wed Mar 23 06:36:12 PDT 2016


On Wed, Mar 23, 2016 at 02:26:37PM +0100, Andrew Lunn wrote:

> The number of windows is limited. On i think the Armada XP, if you put
> a PCIe device on every available PCIe bus, you can run out of
> windows. This is why Thomas implemented dynamic allocation of the
> Windows, so that only those that are needed are used.

> So i would not statically and globally allocate as many windows as
> possible SPI devices.

> The fact that SPI can fall back to another mechanism if there are no
> available windows, were as PCIe cannot, suggests that SPI should
> dynamically allocate a window, and be prepared for it to fail.

> Since only one SPI device can be active at once on a SPI bus, one
> window per bus makes sense, and keeps the required number of windows
> to a minimum.

If we're under pressure for windows I'd go further and say that we
should be dynamically allocating the windows only when they're actually
in use (and modifying other drivers to do the same if that makes sense
for them), unless it's somehow expensive to allocate windows that means
that we should reduce the overall pressure.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160323/a891cc14/attachment.sig>


More information about the linux-arm-kernel mailing list