lkml.org 
[lkml]   [2016]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v2 1/2] devicetree: i2c-hid: Add Wacom digitizer + regulator support
    On Mon, Dec 12, 2016 at 12:34 PM, Brian Norris <briannorris@chromium.org> wrote:
    > Hi all,
    >
    > On Mon, Dec 12, 2016 at 08:47:06AM -0600, Rob Herring wrote:
    >
    > [Snip Benjamin's proposal; I agree we don't really want a multi-level DT
    > layout purely for making the driver look a little nicer (I'm not sure
    > this would really be nicer anyway). And I think, as Rob notes here, our
    > disagreement is smaller than appears. But I might be wrong.]
    >
    >> Anyway, we're only debating this:
    >
    > OK, so I think we might have a consensus of sorts? I'll describe it
    > here, in case I'm wrong. Otherwise, I'll send another rev.
    >
    >> i2c-hid-dev@2c {
    >> compatible = "wacom,w9013", "hid-over-i2c";
    >
    > I plan to document the above, but not treat "wacom,w9013" specially in
    > the driver, apart from possibly listing it in the driver of_match_table.
    > This was mentioned by Dmitry earlier, and I didn't see any objection.
    >
    > (Note that there are problems with module autoload when using multiple
    > compatible strings like above. May not be supremely relevant to the
    > documentation, but it *is* practically important.)

    I'm not sure what is not working here exactly. We emit all the
    compatible strings in the uevent. However, it looks like this is only
    called for platform devices. In the case of i2c, I don't think any
    compatible string is emitted. It looks to me like i2c_device_uevent
    just needs a call to of_device_get_modalias to fix this. There's some
    issues in the I2C core in how it does matching and maybe this is
    related? I would guess if it was that easy, then it would be fixed
    already. Maybe not.

    >> reg = <0x2c>;
    >> hid-descr-addr = <0x0020>;
    >> interrupt-parent = <&gpx3>;
    >> interrupts = <3 2>;
    >> vdd-supply = <sth>;
    >
    > Document and support 'vdd-supply', optionally.
    >
    >> init-delay-ms = <100>;
    >
    > Per Rob's mention below, support this as 'post-power-on-delay-ms',
    > optionally.
    >
    > We can use either of these properties on any device, with the
    > intention that if there are future needs for divergent bindings, the
    > aforementioned compatible property can help us differentiate.

    TBC, from a DT perspective (and what the binding should say), is the
    properties are only valid with a wacom compatible present (or any
    others you want to add). If the driver doesn't enforce that and
    supports having those properties with just "hid-over-i2c", then
    downstream dts's can use that for all I care.

    Rob

    \
     
     \ /
      Last update: 2016-12-13 23:11    [W:3.351 / U:0.472 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site