]> git.karo-electronics.de Git - karo-tx-linux.git/commit
net: tulip: turn compile-time warning into dev_warn()
authorArnd Bergmann <arnd@arndb.de>
Thu, 19 Nov 2015 10:42:26 +0000 (11:42 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 20 Nov 2015 16:02:48 +0000 (11:02 -0500)
commitde92718883ddbcd11b738d36ffcf57617b97fa12
tree77924f5e39d1e7aae1a951c84be631d5d97fc214
parent425d3d83707d378d97d1f7d460ce3083a1948c20
net: tulip: turn compile-time warning into dev_warn()

The tulip driver causes annoying build-time warnings for allmodconfig
builds for all recent architectures:

dec/tulip/winbond-840.c:910:2: warning: #warning Processor architecture undefined
dec/tulip/tulip_core.c:101:2: warning: #warning Processor architecture undefined!

This is the last remaining warning for arm64, and I'd like to get rid of
it. We don't really know the cache line size, architecturally it would
be at least 16 bytes, but all implementations I found have 64 or 128
bytes. Configuring tulip for 32-byte lines as we do on ARM32 seems to
be the safe but slow default, and nobody who cares about performance these
days would use a tulip chip anyway, so we can just use that.

To save the next person the job of trying to find out what this is for
and picking a default for their architecture just to kill off the warning,
I'm now removing the preprocessor #warning and turning it into a pr_warn
or dev_warn that prints the equivalent information when the driver gets
loaded.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Grant Grundler <grundler@parisc-linux.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/dec/tulip/tulip_core.c
drivers/net/ethernet/dec/tulip/winbond-840.c