[PATCH 08/12] mpc5121: Added I2C support.

Grant Likely grant.likely at secretlab.ca
Thu May 7 12:41:47 EST 2009


On Wed, May 6, 2009 at 4:51 PM, Grant Likely <grant.likely at secretlab.ca> wrote:
> On Wed, May 6, 2009 at 4:19 PM, Wolfgang Denk <wd at denx.de> wrote:
>> Dear Grant Likely,
>>
>> In message <fa686aa40905061401k319313c5q89fd3e245c30808f at mail.gmail.com> you wrote:
>>> On Wed, May 6, 2009 at 2:15 PM, Wolfgang Denk <wd at denx.de> wrote:
>>> > From: Piotr Ziecik <kosmo at semihalf.com>
>>> >
>>> > - Enabled I2C interrupts on MPC5121.
>>> > - Updated Kconfig for i2c-mpc driver.
>>>
>>> I think this workaround belongs in the driver itself.
>>
>> Sorry, I don't get it. Which workaround? What exactly should I change?
>
> Sorry, I misread the patch.  Never mind.

Actually, on 3rd reading, I think my first impression was correct
(even if I was wrong about it being a workaround).  This code in
mpc512x_init_i2c() is only relevant for i2c busses (it isn't shared
with any other drivers).  Therefore, it belongs with the i2c bus
itself.  It does not belong in platform code.

Cheers,
g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.



More information about the Linuxppc-dev mailing list