Hello,
Thanks for spending your time replying to this.
On Sun, Jul 16, 2023 at 06:35:17PM +0200, Krzysztof Kozlowski wrote:
> On 15/07/2023 13:25, Jookia wrote:
> > From: John Watts <contact@xxxxxxxxxx>
> >
> > The Allwinner D1, T113 provide two CAN controllers that are variants
> > of the R40 controller.
> >
> > I have tested support for these controllers on two boards:
> >
>
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC (and consider --no-git-fallback argument). It might
> happen, that command when run on an older kernel, gives you outdated
> entries. Therefore please be sure you base your patches on recent Linux
> kernel.
I ran this script and selected some of the emails and CCed them, at least I
believe that's what I did. Maybe I lost them when copying them to vim to make
the command line arguments.
> > +
> > + /omit-if-no-ref/
> > + can0_pins: can0_pins {
>
> Wrong node naming. Underscores are not allowed.
>
> It does not look like you tested the DTS against bindings. Please run
> `make dtbs_check` (see
> Documentation/devicetree/bindings/writing-schema.rst or
> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
> for instructions).
Hmm. I spent a few hours struggling and testing with make dtbs_check and
dt_binding_check. It doesn't seem to pick up on this specific issue, or at
least not in this case.
But yes I do see this, thanks for the catch. Will fix in v2.
I'm basing this patch series on 6.5-rc1, does that make a difference?
> Best regards,
> Krzysztof
Thanks,
John.
