Hi Jacopo, On Fri, Sep 04, 2020 at 10:03:02AM +0200, Jacopo Mondi wrote: > On Fri, Sep 04, 2020 at 12:14:20AM +0300, Laurent Pinchart wrote: > > On Thu, Sep 03, 2020 at 01:51:40PM +0200, Jacopo Mondi wrote: > > > I know I've sent the previous version first as part of a longer list of patches, > > > later individually, and now grouped together again. Sorry for the fuss, hope > > > it's not too confusing. > > > > > > These three sensor bindings conversions go together as they all happen in a > > > single patch, so I deemed it was easier to just group them. > > > > > > In v4 after finalizing the discussion with Laurent and Rob on how to handle > > > of-graph endpoint, I have dropped them from mt9v111 and imx274. For imx214 as I > > > have endpoint properties to document I have also documented 'endpoint' (not > > > mandatory) and 'remote-endpoint' (mandatory). Hope I got the outcome of the > > > discussion right. For imx214 I also took in Laurent's suggestion on how to > > > document the 'data-lanes' supported values. > > > > I think you can drop remote-endpoint, it will be defined in > > of-graph.yaml. Apart from that, it sounds good to me. > > Will I ever get this right ? Should we record the policy to document > endpoints until we don't have an of-graph.yaml to avoid other to have > go through 5 iterations to do it right ? I'll give graph.yaml a go :-) > On this patch: > I owe an updated to the imx274 patch but I'm waiting for this > discussion to get somewhere (you're welcome to step-in :) > https://patchwork.linuxtv.org/project/linux-media/patch/1599012278-10203-3-git-send-email-skomatineni@xxxxxxxxxx/ > > Then I'll re-send a v5 > > > > Individual maintainers cc-ed per-patch. > > > > > > Jacopo Mondi (3): > > > dt-bindings: media: mt9v111: Convert to json-schema > > > dt-bindings: media: imx274: Convert to json-schema > > > dt-bindings: media: imx214: Convert to json-schema > > > > > > .../bindings/media/i2c/aptina,mt9v111.txt | 46 ------ > > > .../bindings/media/i2c/aptina,mt9v111.yaml | 75 ++++++++++ > > > .../devicetree/bindings/media/i2c/imx274.txt | 38 ----- > > > .../bindings/media/i2c/sony,imx214.txt | 53 ------- > > > .../bindings/media/i2c/sony,imx214.yaml | 137 ++++++++++++++++++ > > > .../bindings/media/i2c/sony,imx274.yaml | 59 ++++++++ > > > MAINTAINERS | 6 +- > > > 7 files changed, 274 insertions(+), 140 deletions(-) > > > delete mode 100644 Documentation/devicetree/bindings/media/i2c/aptina,mt9v111.txt > > > create mode 100644 Documentation/devicetree/bindings/media/i2c/aptina,mt9v111.yaml > > > delete mode 100644 Documentation/devicetree/bindings/media/i2c/imx274.txt > > > delete mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx214.txt > > > create mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx214.yaml > > > create mode 100644 Documentation/devicetree/bindings/media/i2c/sony,imx274.yaml -- Regards, Laurent Pinchart
