Thread (9 messages) 9 messages, 2 authors, 2013-09-30
DORMANTno replies

[PATCH 3/4] spi: davinci: Allow SPI to be build without EDMA support

From: Santosh Shilimkar <hidden>
Date: 2013-09-30 15:23:48

On Monday 30 September 2013 10:38 AM, Mark Brown wrote:
On Mon, Sep 30, 2013 at 09:36:23AM -0400, Santosh Shilimkar wrote:
quoted
On Sunday 29 September 2013 08:31 AM, Mark Brown wrote:
quoted
quoted
If there's no hard dependency on EDMA then the driver probably ought to
be marked as COMPILE_TEST so that it can be build tested on other
systems.  Is that possible?
quoted
EDMA is already getting tested for DAVINCI builds and there are patches
coming AMXXXX SOC support. So probably marking it COMPILE_TEST isn't
needed since it has been in use already for other SOCs.
It'd make life easier for general subsystem development if we were able
to get build coverage without having to build a specific config, even if
it's never actually going to be run.
So after your comments, I actually checked whether the driver works with
EDMA enabled but no EDMA node/data information from DT for SPI driver.
It seems to work.

So I am going to drop this patch and rather allow the EDMA to be built
for Keystone SOCs as well. Not sure why I didn't looked at this as an
option first place.

Thanks for your comments Mark.

Regards,
Santosh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help