Thread (1 message) 1 message, 1 author, 2002-05-28

Re: [PATCH] enhanced i2c driver for MPC8xx/MPC8260 CPM ...

From: Matt Porter <hidden>
Date: 2002-05-28 13:27:18

On Tue, May 28, 2002 at 09:39:31PM +1000, Murray Jensen wrote:
On Tue, 28 May 2002 07:01:25 -0400, Dan Malek [off-list ref] writes:
quoted
Murray Jensen wrote:
quoted
enhanced i2c driver for MPC8xx/MPC8260 CPM ...
These have to be submitted through the i2c project on sourceforge (or
where ever it is).  The i2c isn't part of the kernel source tree, it is
applied on top from their repository from time to time.
Tom said this to me as well, and I agree in principle, but I thought it should
first be reviewed by the Linux/PPC Embedded community. It's actually quite a
hacky driver, but in my defence, I could claim the original driver was hacky
to start with - I really didn't change it a lot, conceptually (what I mean by
this is that it could be much improved in the way it does things - more
correct, more efficient, etc, but hey it works reliably for me, so there isn't
a lot of incentive to fix it - its only i2c after all :-).

Also, the i2c people probably wouldn't have any hope of testing this driver,
since nothing else is going to have a CPM, and in fact I haven't even tested
it myself on the 8xx platform yet. I propose we get it into linuxppc_2_4_devel
and have it hammered on by others, and when/if it gets the thumbs up from the
Linux/PPC Embedded community, we/I contribute it back to the i2c project -
really just for the sake of completeness. Comments? Cheers!
FWIW, that's why I pushed the 4xx IIC driver to _devel recently.  It
certainly allows a wider audience to easily test and submit changes
to the maintainer, allowing for faster turnaround in improvements.

Regards,
--
Matt Porter
porter@cox.net
This is Linux Country. On a quiet night, you can hear Windows reboot.

** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help