[patch 3/3] mpc8349emitx.dts: Add ds1339 RTC

Scott Wood scottwood at freescale.com
Tue Sep 25 00:52:31 EST 2007


David Gibson wrote:
>>  		i2c at 3100 {
>> +			#address-cells = <1>;
>> +			#size-cells = <0>;
>>  			device_type = "i2c";
> 
> Hrm... we probably want an "i2c" device_type class, but I don't think
> we've actually defined one, which is a problem

Right... but we need to get the kernel to stop expecting the device type 
to be there before we yell at people for including it. :-)

> The fact that NVRAM+RTC chips are so common is a bit of an issue from
> the point of view of defining a device class binding - a device can't
> have type "rtc" and "nvram".

This is one of the reasons that I'd prefer to use compatible for such 
things.

-Scott



More information about the Linuxppc-dev mailing list