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

[Adrian Chadd]

On 25 May 2016 at 18:57, Patrick Kelsey <pkelsey at freebsd.org> wrote:
>
> 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.

loos and i are thinking about it. I may start landing some of the
spibus changes tonight just to lay the foundation.



-adrian