Messages in this thread | | | Subject | Re: [PATCH v2 1/2] dt-bindings: Add vendor prefix for Catalyst Semiconductor | From | Andreas Färber <> | Date | Sat, 19 Jan 2019 12:30:14 +0100 |
| |
Hi Martyn,
Am 18.01.19 um 20:48 schrieb Martyn Welch: > Add vendor prefix "catalyst" for Catalyst Semiconductor which is > already in use but undocumented. > > Signed-off-by: Martyn Welch <martyn.welch@collabora.com> > Reviewed-by: Rob Herring <robh@kernel.org> > --- > > Changes in v2: None > > Documentation/devicetree/bindings/vendor-prefixes.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt > index 389508584f48..d80b23b7771f 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.txt > +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt > @@ -65,6 +65,7 @@ bticino Bticino International > calxeda Calxeda > capella Capella Microsystems, Inc > cascoda Cascoda, Ltd. > +catalyst Catalyst Semiconductor, Inc
"Inc" surely is an abbreviation for "Incorporated", so can you please follow the below examples of writing "Inc." with trailing dot?
> cavium Cavium, Inc. > cdns Cadence Design Systems Inc. > cdtech CDTech(H.K.) Electronics Limited
Otherwise looks good. Thanks for fixing the undocumented use.
Regards, Andreas
-- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg)
| |