|To:||"'Chris Wedgwood'" <firstname.lastname@example.org>|
|Subject:||[PATCH 2.6.16-rc5 1/5] IDT Interprise Processor Support for Linu x 2.6.x|
|From:||"Tiwari, Rakesh" <Rakesh.Tiwari@idt.com>|
|Date:||Tue, 14 Mar 2006 16:23:50 -0800|
|Cc:||"'Ralf Baechle'" <email@example.com>, "'firstname.lastname@example.org'" <email@example.com>|
The attached patch adds support for the IDT Interprise processor RC32438
Additional information regarding IDT processor's can be found at
Signed-off-by: Rakesh Tiwari <firstname.lastname@example.org>
Appreciate your feedback.
Look forward for additional comments/suggestions, if any.
* Firstly, it's really great to see this!
* A single 1.6MB patch is far from ideal, please try to break it
[rkt] Agreed, 1.6MB is a huge patch. I will try to break it down into
* You shouldn't be removing .gitignore :-)
[rkt] I think these are still there.
[rkt] The Ethernet interface/driver is integral to each processor
[rkt] Chris you hit the bulls eye. This is the only function which
[rkt] Since all the IDT's processors are primarily SoC's, the UARTS are
* There is quite a bit of extraneous white-space that could be
[rkt] Will try to clean up as much as possible...
Sorry this is a little vague and 'hand-wavy', if you post smaller logically complete patches I think you'll get better feedback where people can comment more easily. Ideally inline to the email if you can, m$ lookout/$exchange as that just makes a mess, if you have to use that then attach them as you did.
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:||RE: BCM91x80A/B PCI DMA problems, Mark E Mason|
|Next by Date:||Building GCC for BCM1480 SiByte, email@example.com|
|Previous by Thread:||[PATCH] Alchemy: make OSS I2S driver build, Sergei Shtylylov|
|Next by Thread:||Building GCC for BCM1480 SiByte, firstname.lastname@example.org|
|Indexes:||[Date] [Thread] [Top] [All Lists]|