Messages in this thread | | | From | Tomasz Figa <> | Subject | Re: [PATCH v3 2/2] max77693: added device tree support | Date | Fri, 23 Aug 2013 22:36:10 +0200 |
| |
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
| |