mbox series

[0/7] dt-bindings: i2c: few fixes and cleanups

Message ID 20240620-dt-bindings-i2c-clean-v1-0-3a1016a95f9d@linaro.org
Headers show
Series dt-bindings: i2c: few fixes and cleanups | expand

Message

Krzysztof Kozlowski June 20, 2024, 11:34 a.m. UTC 
Few fixes for I2C controller schemas. The third patch (atmel,at91sam)
depends on first, so I suggest not splitting this into fixes branch but
take as is via next branch.

Best regards,
Krzysztof

---
Krzysztof Kozlowski (7):
      dt-bindings: i2c: atmel,at91sam: correct path to i2c-controller schema
      dt-bindings: i2c: google,cros-ec-i2c-tunnel: correct path to i2c-controller schema
      dt-bindings: i2c: atmel,at91sam: drop unneeded address/size-cells
      dt-bindings: i2c: nvidia,tegra20: drop unneeded address/size-cells
      dt-bindings: i2c: samsung,s3c2410: drop unneeded address/size-cells
      dt-bindings: i2c: ti,omap4: reference i2c-controller.yaml schema
      dt-bindings: i2c: adjust indentation in DTS example to coding style

 .../devicetree/bindings/i2c/atmel,at91sam-i2c.yaml |  10 +-
 .../devicetree/bindings/i2c/brcm,brcmstb-i2c.yaml  |  28 +++---
 .../bindings/i2c/google,cros-ec-i2c-tunnel.yaml    |   2 +-
 .../devicetree/bindings/i2c/i2c-demux-pinctrl.yaml | 106 ++++++++++-----------
 .../bindings/i2c/nvidia,tegra20-i2c.yaml           |   6 --
 .../devicetree/bindings/i2c/renesas,iic-emev2.yaml |  14 +--
 .../devicetree/bindings/i2c/renesas,rcar-i2c.yaml  |  20 ++--
 .../devicetree/bindings/i2c/renesas,riic.yaml      |  34 +++----
 .../bindings/i2c/renesas,rmobile-iic.yaml          |  24 ++---
 .../bindings/i2c/samsung,s3c2410-i2c.yaml          |   6 --
 .../devicetree/bindings/i2c/st,stm32-i2c.yaml      |  66 ++++++-------
 .../devicetree/bindings/i2c/ti,omap4-i2c.yaml      |  64 +++++--------
 12 files changed, 174 insertions(+), 206 deletions(-)
---
base-commit: 76db4c64526c5e8ba0f56ad3d890dce8f9b00bbc
change-id: 20240620-dt-bindings-i2c-clean-b5a0b2dfdece

Best regards,

Comments

Conor Dooley June 20, 2024, 4:06 p.m. UTC | #1 
On Thu, Jun 20, 2024 at 01:34:48PM +0200, Krzysztof Kozlowski wrote:
> Few fixes for I2C controller schemas. The third patch (atmel,at91sam)
> depends on first, so I suggest not splitting this into fixes branch but
> take as is via next branch.

Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
Andi Shyti June 20, 2024, 11:05 p.m. UTC | #2 
Cześć Krzysztof,

On Thu, Jun 20, 2024 at 01:34:48PM GMT, Krzysztof Kozlowski wrote:
> Few fixes for I2C controller schemas. The third patch (atmel,at91sam)
> depends on first, so I suggest not splitting this into fixes branch but
> take as is via next branch.
> 
> Best regards,
> Krzysztof
> 
> ---
> Krzysztof Kozlowski (7):
>       dt-bindings: i2c: atmel,at91sam: correct path to i2c-controller schema
>       dt-bindings: i2c: google,cros-ec-i2c-tunnel: correct path to i2c-controller schema

merged to i2c/i2c-host-fixes

>       dt-bindings: i2c: atmel,at91sam: drop unneeded address/size-cells
>       dt-bindings: i2c: nvidia,tegra20: drop unneeded address/size-cells
>       dt-bindings: i2c: samsung,s3c2410: drop unneeded address/size-cells
>       dt-bindings: i2c: ti,omap4: reference i2c-controller.yaml schema
>       dt-bindings: i2c: adjust indentation in DTS example to coding style

merged to i2c/i2c-host

Thanks,
Andi
Krzysztof Kozlowski June 21, 2024, 6:02 a.m. UTC | #3 
On 21/06/2024 01:05, Andi Shyti wrote:
> Cześć Krzysztof,
> 
> On Thu, Jun 20, 2024 at 01:34:48PM GMT, Krzysztof Kozlowski wrote:
>> Few fixes for I2C controller schemas. The third patch (atmel,at91sam)
>> depends on first, so I suggest not splitting this into fixes branch but
>> take as is via next branch.
>>
>> Best regards,
>> Krzysztof
>>
>> ---
>> Krzysztof Kozlowski (7):
>>       dt-bindings: i2c: atmel,at91sam: correct path to i2c-controller schema
>>       dt-bindings: i2c: google,cros-ec-i2c-tunnel: correct path to i2c-controller schema
> 
> merged to i2c/i2c-host-fixes
> 
>>       dt-bindings: i2c: atmel,at91sam: drop unneeded address/size-cells
>>       dt-bindings: i2c: nvidia,tegra20: drop unneeded address/size-cells
>>       dt-bindings: i2c: samsung,s3c2410: drop unneeded address/size-cells
>>       dt-bindings: i2c: ti,omap4: reference i2c-controller.yaml schema
>>       dt-bindings: i2c: adjust indentation in DTS example to coding style
> 
> merged to i2c/i2c-host

So you broke the binding... Why openly ignoring my first sentence?

Best regards,
Krzysztof
Andi Shyti June 21, 2024, 9:27 a.m. UTC | #4 
On Fri, Jun 21, 2024 at 08:02:03AM GMT, Krzysztof Kozlowski wrote:
> On 21/06/2024 01:05, Andi Shyti wrote:
> > Cześć Krzysztof,
> > 
> > On Thu, Jun 20, 2024 at 01:34:48PM GMT, Krzysztof Kozlowski wrote:
> >> Few fixes for I2C controller schemas. The third patch (atmel,at91sam)
> >> depends on first, so I suggest not splitting this into fixes branch but
> >> take as is via next branch.
> >>
> >> Best regards,
> >> Krzysztof
> >>
> >> ---
> >> Krzysztof Kozlowski (7):
> >>       dt-bindings: i2c: atmel,at91sam: correct path to i2c-controller schema
> >>       dt-bindings: i2c: google,cros-ec-i2c-tunnel: correct path to i2c-controller schema
> > 
> > merged to i2c/i2c-host-fixes
> > 
> >>       dt-bindings: i2c: atmel,at91sam: drop unneeded address/size-cells
> >>       dt-bindings: i2c: nvidia,tegra20: drop unneeded address/size-cells
> >>       dt-bindings: i2c: samsung,s3c2410: drop unneeded address/size-cells
> >>       dt-bindings: i2c: ti,omap4: reference i2c-controller.yaml schema
> >>       dt-bindings: i2c: adjust indentation in DTS example to coding style
> > 
> > merged to i2c/i2c-host
> 
> So you broke the binding... Why openly ignoring my first sentence?

It's not an issue. We can get the fixes first and apply the rest
later, I can keep them in my -next branch and reapply next week.

Otherwise I would wait for everything at the merge window, but I
don't want to wait so long.

Thanks,
Andi