Thread (17 messages) 17 messages, 8 authors, 2013-10-30
STALE4596d
Revisions (2)
  1. v1 [diff vs current]
  2. v1 current

Re: [PATCH 00/19] Enable various Renesas drivers on all ARM platforms

From: Simon Horman <horms@verge.net.au>
Date: 2013-10-30 00:05:03
Also in: linux-arm-kernel, linux-i2c, linux-input, linux-iommu, linux-mmc, linux-pm, linux-pwm, linux-serial, linux-spi

On Tue, Oct 29, 2013 at 10:58:34AM -0700, Mark Brown wrote:
On Tue, Oct 29, 2013 at 06:29:59PM +0100, Laurent Pinchart wrote:
quoted
On Tuesday 29 October 2013 10:23:31 Mark Brown wrote:
quoted
On Tue, Oct 29, 2013 at 06:05:53PM +0100, Laurent Pinchart wrote:
quoted
The first one is that I can't compile-test all those drivers on all
architectures. The spi-sh-msiof driver, for instance, uses
io(read|write)(16|
quoted
quoted
Which architectures are these and is there not a symbol we can depend on
for them?
quoted
arch/cris for instance. We can use readl/writel instead (maybe it would be 
time to rationalize and document the I/O accessors across all architectures, 
but that's another topic).
It'd certainly be sensible, or adding a config option to depend on if
you rely on these functions.
quoted
My point is that there might be other issues that I won't be able to easily 
catch. This would break compilation for everybody for no reason, as the 
drivers are useless on non-SuperH, non-ARM platforms. That's why I believe 
COMPILE_TEST would be a better option as a first step.
Yes, it would - please do that.  Note that it won't stop anyone running
into build issues on other architectures though, it's just about
stopping Kconfig noise.
FWIW, I am happy with using COMPILE_TEST for this series.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help