Re: [PATCH] dt-bindings: clock: renesas: div6: Use ZT/ZTR trace clock in R-Mobile APE6 example

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 2092 bytes --]

On Sat, May 23, 2026 at 07:40:47PM +0000, sashiko-bot@kernel.org wrote:
> Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> - [Low] Adding 18 items to `clock-output-names` in the example triggers a `dt_binding_check` schema validation error due to an un-updated top-level constraint in `renesas,cpg-clocks.yaml`.
> --
> 
> commit aaa0ff6ae07f5974071a547fef4a5b41bc484428
> Author: Marek Vasut <marek.vasut+renesas@mailbox.org>
> 
> dt-bindings: clock: renesas: div6: Use ZT/ZTR trace clock in R-Mobile APE6 example
> 
> This commit updates the R-Mobile APE6 clock node example to include
> the "zt" and "ztr" clock-output-names entries. This aligns the example
> with the expectations documented in a previous commit.
> 
> > diff --git a/Documentation/devicetree/bindings/clock/renesas,cpg-div6-clock.yaml b/Documentation/devicetree/bindings/clock/renesas,cpg-div6-clock.yaml
> > index 2197c952e21df..b6ee8c8efd46d 100644
> > --- a/Documentation/devicetree/bindings/clock/renesas,cpg-div6-clock.yaml
> > +++ b/Documentation/devicetree/bindings/clock/renesas,cpg-div6-clock.yaml
> > @@ -60,7 +60,7 @@ examples:
> >              clock-output-names = "main", "pll0", "pll1", "pll2",
> >                                    "pll2s", "pll2h", "z", "z2",
> >                                    "i", "m3", "b", "m1", "m2",
> > -                                  "zx", "zs", "hp";
> > +                                  "zx", "zs", "hp", "ztr", "zt";
> 
> [Severity: Low]
> Does adding these two items cause a schema validation failure during
> dt_binding_check?
> 
> The top-level clock-output-names property in renesas,cpg-clocks.yaml has a
> maxItems constraint of 17. Adding "ztr" and "zt" here brings the total
> item count to 18.
> 
> While the underlying limit omission occurred in the earlier commit
> 2abdc3dcf978, modifying this example to 18 items will now directly
> trigger the overarching validation limit error.

What does renesas,cpg-clocks.yaml have to do with this?
This is just complete crap, right?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]