Re: [PATCH v4 1/2] dt-bindings: can: m_can: add ti,am62-m_can compatible

[Kendall Willis]

Hi Markus,
On 10:28-20260903, Markus Schneider-Pargmann wrote:
> Hi Kendall,
> 
> On Wed Sep 2, 2026 at 11:03 PM CEST, Kendall Willis wrote:
> > Add "ti,am62-m_can" to the list of compatible strings for the M_CAN
> > controller binding. This compatible identifies TI AM62 SoCs which
> > support out-of-band wakeup capability.
> >
> > Signed-off-by: Kendall Willis <k-willis@xxxxxx>
> > ---
> >  Documentation/devicetree/bindings/net/can/bosch,m_can.yaml | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> > index 2c9d37975bedd652b3060ab11ba75c37565edaad..89e5a44453372c57506e9cf3a009b55a70cc4446 100644
> > --- a/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> > +++ b/Documentation/devicetree/bindings/net/can/bosch,m_can.yaml
> > @@ -16,7 +16,9 @@ allOf:
> >  
> >  properties:
> >    compatible:
> > -    const: bosch,m_can
> > +    enum:
> > +      - bosch,m_can
> > +      - ti,am62-m_can
> 
> I think you should use oneOf here to allow the use of
> 
>   compatible = "ti,am62-m_can", "bosch,m_can";
> 

Oops, yes this was a mistake on my part. Will fix in the next version.

Best,
Kendall