Re: [PATCH v2 09/10] arm/dts: omap3-beagle: Add twl4030 and i2c EEPROM

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 1/10/2012 12:24 AM, Tabi Timur-B04825 wrote:
On Fri, Dec 9, 2011 at 8:02 AM, Benoit Cousson<b-cousson@xxxxxx>  wrote:

+       eeprom@50 {
+               compatible = "ti,eeprom";
+               reg =<0x50>;
+       };

Why is this "ti,"?  For EDID, isn't the I2C device actually on the
monitor itself, and DVI cable just connects to the I2C bus?

Yes, in fact I didn't know what to put here, since I do not know what kind of eeprom the monitor will use.

And then I realized that the i2c driver is using the compatible name without the prefix, so any prefix can be used there... OK, that's maybe not a very valid reason... but I did not find any better prefix at that time:-)

The reason I ask is that I'm trying to do the same thing for a PowerPC
board.  I need to defined an EDID I2C node.

Well, this seems to be generic i2c eeprom, so maybe a "ddc," prefix or "edid," will be a little bit better.

If later, we want to access that node from the display driver, we can just add a phandle to the node's label: "edid-eeprom: eeprom@50:"

Regards,
Benoit

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux