Hi Mark, On Friday 23 of August 2013 15:14:33 Mark Rutland wrote: > On Wed, Aug 21, 2013 at 05:53:34PM +0100, Andrzej Hajda wrote: > > This patch adds only of_match_table. > > There are no device specific properties. > > Could you clarify what functionality this enables and what it doesn't, > please? This patch simply adds explicit OF match table for this device. Before, the driver could be matched only by a fallback to i2c_device_id table. > This doesn't seem to enable support for the regulators described in the > binding [1] (which from the looks of it needs proof-reading and possibly > rework). > > Are there any changes we might need in future to either support new > functionality or to generalise the binding. e.g. do we need a regulator > for the LED? > > Given the binding has never been supported, are we happy now that it > best represents the hardware, or are there avenues of improvement > *before* it becomes ABI? Well, documentation of the binding has been present in kernel tree since June, but I too think that we should review it and make sure it makes sense. I'll try to get some information on this chip at work, on Monday. Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html