lkml.org 
[lkml]   [2020]   [May]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 0/2] Add TI tps6598x DT binding and probe
Hi,

On Thu, May 07, 2020 at 10:47:31PM +0100, Bryan O'Donoghue wrote:
> V2:
> - Put myself down as sole schema maintainer as suggested - Andy
> - Fixed whitespace typo - Andy
> - Removed ifdef and of_match_ptr() - Andy
>
> V1:
> This simple series adds DT binding yaml and a DT lookup table for the
> tps6598x.
>
> Its possible to use i2c id_table to match the 'compatible = "ti,tps6598x"
> and probe that way, however I think it is worthwhile adding a specific OF
> way of doing it and having an accompanying yaml as an example.
>
> Bryan O'Donoghue (2):
> dt-bindings: usb: Add TI tps6598x device tree binding documentation
> usb: typec: tps6598x: Add OF probe binding
>
> .../devicetree/bindings/usb/ti,tps6598x.yaml | 64 +++++++++++++++++++
> drivers/usb/typec/tps6598x.c | 7 ++
> 2 files changed, 71 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/usb/ti,tps6598x.yaml

There was already a series from Angus [1] for this. The bindings
looked a bit different, but I think we should use these, because in
the DT bindings from Angus there appeared to be definitions for OF
graph that was not used. Or maybe I got it wrong?

Angus, is it OK if we use these patches instead the ones from you?

[1] https://lore.kernel.org/linux-usb/20200506191718.2144752-1-angus@akkea.ca/

thanks,

--
heikki

\
 
 \ /
  Last update: 2020-05-08 16:01    [W:0.097 / U:0.712 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site