svn commit: r298274 - head/sys/dev/spibus

[Patrick Kelsey]

On Fri, May 20, 2016 at 2:35 AM, Adrian Chadd <adrian at freebsd.org> wrote:

> I've reviewed the patches from luiz, and these look fine. but indeed,
> this patchset does set/release the bus each transaction so we can do
> multiple transactions whilst holding the bus.
>
> Which is fine, but it means I have to undo ruslan's removal in the below
> commit.
>
> I'm happy to run through and do each of the spi bus drivers (as there
> are more now than the two you patched) but it's a bit close to the
> 11.0-release cycle to go and churn the spibus code.
>
> But, if people think it's worth doing it so we can try to get mmcspi
> into the tree before 11.0-rel is cut, I'm happy to run through and do
> it. I actually have some AR9331 stuff now that could use mmcspi. :)
>
> What do people think?
>
>
Thanks, Adrian.  My opinion is that the changes to the spi bus interface
and the corresponding changes to the spi bus drivers you are talking about
are very straightforward and low risk.  There is non-zero other interest
out there in using the mmcspi driver, based on the couple of emails I've
received seeking help on the subject in the last six months or so, so I
don't think this would be just a commit for the sake of getting some
interesting new code in.

-Patrick