25 Jul
2012
25 Jul
'12
12:03 a.m.
Stephen Warren wrote:
The I2C code matches the compatible values from the device tree against two tables in the kernel. First, the of_match_table that this patch adds, and then if there's no match it falls back to the I2C device ID table, and matches that against the DT compatible value with the vendor prefix stripped off.
So while the second method does work as a fall-back, in the past I've received guidance (I think from Grant Likely) that we should still add the of_match_table to drivers in order to be explicit.
So you're saying that all I2C device drivers should have an of_match_table()? If so, then we should probably fix all codec drivers in one patch.
--
Timur Tabi
Linux kernel developer at Freescale