|To:||Jordan Crouse <firstname.lastname@example.org>|
|Subject:||Re: I2C troubles with Au1550|
|From:||Sergei Shtylyov <email@example.com>|
|Date:||Fri, 19 May 2006 21:11:10 +0400|
|Cc:||Linux-MIPS <firstname.lastname@example.org>, email@example.com|
|Organization:||MontaVista Software Inc.|
|References:||<firstname.lastname@example.org> <20060519143247.GC9596@cosmic.amd.com> <446DDABE.email@example.com> <20060519150851.GD9596@cosmic.amd.com>|
|User-agent:||Mozilla/5.0 (X11; U; Linux i686; rv:1.7.2) Gecko/20040803|
Jordan Crouse wrote:
On 19/05/06 18:48 +0400, Sergei Shtylyov wrote:Alas, I have to NAK this. DBAu1200 code should be in arch/mips/au1000/pb1200/...
if this was DB1200 code only, I would be inclined to agree, but its not - so this code is well placed.
It's under #ifdef CONFIG_MIPS_DB1200, so is *completely* misplaced. This file is not even compiled for DBAu1200. Therefore, this code will never execute.
Thou wait... that hunk won't even aplly to the current git tree...
Well, it does apply - latest GIT tree, right from l-m.o
Hmm, indeed it applies with fuzz (because of PSC redefinitions)... So, I'm taking this back. :-)
Looks like this patch is trying to redeclare PSC base addresses for Au1200
Yeah, it does a redundant declaration - I'll pull that part of it. The rest of the patch is still valid though - I see no reason why you should NAK it, especially when this was posted by popular request.
Because I caught a defect in it with naked eye. Maybe NAKing it was indeed too much. :-)
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:||[PATCH] I2C: Add Au1200 I2C support, Jordan Crouse|
|Next by Date:||[PATCH] NEC EMMA2RH support, revisited, dmitry pervushin|
|Previous by Thread:||Re: I2C troubles with Au1550, Jordan Crouse|
|Next by Thread:||Re: I2C troubles with Au1550, Ralf Baechle|
|Indexes:||[Date] [Thread] [Top] [All Lists]|