Hi Laurent, On Fri, Sep 04, 2020 at 12:14:20AM +0300, Laurent Pinchart wrote: > Hi Jacopo, > > 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 ? 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
