Re: [PATCH 1/4] dt-bindings: serial: document esp32-uart bindings

From: Max Filippov
Date: Thu Sep 14 2023 - 16:02:49 EST


On Thu, Sep 14, 2023 at 7:48 AM Conor Dooley <conor@xxxxxxxxxx> wrote:
> On Thu, Sep 14, 2023 at 07:55:35AM +0200, Krzysztof Kozlowski wrote:
> > On 13/09/2023 23:14, Max Filippov wrote:
> > > +description: |
> > > + ESP32 UART controller is a part of ESP32 SoC series.
> >
> > 1. Company name?
> > 2. ESP32 SoC series suggests esp32 is a series.
> >
> > > +
> > > +properties:
> > > + compatible:
> > > + oneOf:
> > > + - description: UART controller for the ESP32 SoC
> > > + const: esp,esp32-uart
> >
> > Also, the vendor prefix looks incorrect, so again - what is the company
> > name?
>
> esp32 is made by expressif, which would match with "esp" as a vendor
> prefix.

It's 'Espressif', but otherwise yes, this is a registered vendor prefix. See
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/vendor-prefixes.yaml?h=v6.6-rc1#n443

--
Thanks.
-- Max