lkml.org 
[lkml]   [2019]   [Jan]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 1/2] dt-bindings: Add vendor prefix for Catalyst Semiconductor
From
Date
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)

\
 
 \ /
  Last update: 2019-01-19 12:31    [W:0.059 / U:0.956 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site