Re: [PATCH POWERPC] i2c: adds support for i2c bus on Frescale CPM1/CPM2 controllers

Hi Vitaly,

I had an attempt a while ago to do this but haven't had enough time to get it completed, so
I am glad to see it finally picked up. There was some sort of discussion that time, you seem to have some of those points
addressed but  something not, please
check: http://lkml.org/lkml/2007/5/8/45

(for instance, cpm_i2c_shutdown() and i2c_cpm_del_bus() are void because they cant' fail)

I'll resend a fixed patch.

Also, why this was directed to Scott? This is lm-sensors stuff and is
Jean Delvare <[email protected]> material..

Scott reviewed this driver last time and i hope to get some further comments from him. But thanks for adding

Jean as well.

Thanks,
Jochen

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• References:
  • [PATCH POWERPC] i2c: adds support for i2c bus on Frescale CPM1/CPM2 controllers
    • From: Jochen Friedrich <[email protected]>
  • Re: [PATCH POWERPC] i2c: adds support for i2c bus on Frescale CPM1/CPM2 controllers
    • From: Vitaly Bordug <[email protected]>

• Prev by Date: Re: [PATCH] kthread: run kthreadd with max priority SCHED_FIFO
• Next by Date: Re: Major regression on hackbench with SLUB (more numbers)
• Previous by thread: Re: [PATCH POWERPC] i2c: adds support for i2c bus on Frescale CPM1/CPM2 controllers
• Next by thread: [PATCH] (linux-2.6-block.git) xsysace: fix size misconversion of blk_end_request
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]