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
