Thread (95 messages) 95 messages, 32 authors, 2014-06-25

[PATCH] i2c/nuc900: fix ancient build error

From: arnd@arndb.de (Arnd Bergmann)
Date: 2014-05-14 18:14:30
Also in: linux-i2c, lkml

On Wednesday 14 May 2014 18:27:25 Wolfram Sang wrote:
On Thu, May 08, 2014 at 04:56:22PM +0200, Arnd Bergmann wrote:
quoted
As far as I can tell, this driver must have produced this error
for as long as it has been merged into the mainline kernel, but
it was never part of the normal build tests:

drivers/i2c/busses/i2c-nuc900.c: In function 'nuc900_i2c_probe':
drivers/i2c/busses/i2c-nuc900.c:601:17: error: request for member 'apbfreq' in something not a structure or union
  ret = (i2c->clk.apbfreq)/(pdata->bus_freq * 5) - 1;
                 ^

This is an attempt to get the driver to build and possibly
work correctly, although I do wonder whether we should just
remove it, as it has clearly never worked.
I'd go for removing. For this platform, the last patch which was not a
generic cleanup seems to be from late 2011?
Ah, you mean removing the entire platform? I guess we could do that
as well, but I was really thinking of just removing the i2c driver.

For the moment, I'd leave this up to Wan ZongShun. He has in the past
at least replied to emails about the platform, even though there hasn't
been any new development.

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