diff mbox

[v4,2/4] DT: Vendor prefixes: Add ricoh, ssi and synology

Message ID 1392673537-17308-3-git-send-email-klightspeed@killerwolves.net
State Superseded, archived
Headers show

Commit Message

klightspeed@killerwolves.net Feb. 17, 2014, 9:45 p.m. UTC
From: Andrew Lunn <andrew@lunn.ch>

The following patches make use of vendor names ricoh, ssi and
synology.  Add them to the vendor prefix list.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Acked-by: Jason Cooper <jason@lakedaemon.net>
---
v2:
Use stock ticker for Ricoh as vendor name
s/Richoh/Ricoh/
---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 3 +++
 1 file changed, 3 insertions(+)

Comments

Josh Cartwright Feb. 17, 2014, 10:34 p.m. UTC | #1
On Tue, Feb 18, 2014 at 07:45:35AM +1000, klightspeed@killerwolves.net wrote:
> From: Andrew Lunn <andrew@lunn.ch>
> 
> The following patches make use of vendor names ricoh, ssi and
> synology.  Add them to the vendor prefix list.
> 
> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> Acked-by: Jason Cooper <jason@lakedaemon.net>
> ---
> v2:
> Use stock ticker for Ricoh as vendor name
> s/Richoh/Ricoh/
> ---
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index 40ce2df..8427681 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -72,6 +72,7 @@ ralink	Mediatek/Ralink Technology Corp.
>  ramtron	Ramtron International
>  realtek Realtek Semiconductor Corp.
>  renesas	Renesas Electronics Corporation
> +ricoy	Ricoh Co. Ltd.
>  rockchip	Fuzhou Rockchip Electronics Co., Ltd
>  samsung	Samsung Semiconductor
>  sbs	Smart Battery System
> @@ -79,11 +80,13 @@ schindler	Schindler
>  sil	Silicon Image
>  silabs	Silicon Laboratories
>  simtek
> +sii	Seiko Instruments, Inc.
>  sirf	SiRF Technology, Inc.
>  snps 	Synopsys, Inc.
>  st	STMicroelectronics
>  ste	ST-Ericsson
>  stericsson	ST-Ericsson
> +synology	Synology, Inc.

Oh, great!  Looks like this one is added (see my comment on patch 1).
I'm wondering if 'qnap' should be added as well.
Sebastian Reichel Feb. 17, 2014, 10:40 p.m. UTC | #2
On Tue, Feb 18, 2014 at 07:45:35AM +1000, klightspeed@killerwolves.net wrote:
> From: Andrew Lunn <andrew@lunn.ch>
> 
> The following patches make use of vendor names ricoh, ssi and
> synology.  Add them to the vendor prefix list.

You have a typo in the patch description + subject: ssi vs sii.

-- Sebastian
klightspeed@killerwolves.net Feb. 18, 2014, 12:07 a.m. UTC | #3
On 18/02/2014 8:34 AM, Josh Cartwright wrote:
> On Tue, Feb 18, 2014 at 07:45:35AM +1000, klightspeed@killerwolves.net wrote:
>> From: Andrew Lunn <andrew@lunn.ch>
>>
>> The following patches make use of vendor names ricoh, ssi and
>> synology.  Add them to the vendor prefix list.
>>
>> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
>> Acked-by: Jason Cooper <jason@lakedaemon.net>
>> ---
>> v2:
>> Use stock ticker for Ricoh as vendor name
>> s/Richoh/Ricoh/
>> ---
>>  Documentation/devicetree/bindings/vendor-prefixes.txt | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
>> index 40ce2df..8427681 100644
>> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
>> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
>> @@ -72,6 +72,7 @@ ralink	Mediatek/Ralink Technology Corp.
>>  ramtron	Ramtron International
>>  realtek Realtek Semiconductor Corp.
>>  renesas	Renesas Electronics Corporation
>> +ricoy	Ricoh Co. Ltd.
>>  rockchip	Fuzhou Rockchip Electronics Co., Ltd
>>  samsung	Samsung Semiconductor
>>  sbs	Smart Battery System
>> @@ -79,11 +80,13 @@ schindler	Schindler
>>  sil	Silicon Image
>>  silabs	Silicon Laboratories
>>  simtek
>> +sii	Seiko Instruments, Inc.
>>  sirf	SiRF Technology, Inc.
>>  snps 	Synopsys, Inc.
>>  st	STMicroelectronics
>>  ste	ST-Ericsson
>>  stericsson	ST-Ericsson
>> +synology	Synology, Inc.
> 
> Oh, great!  Looks like this one is added (see my comment on patch 1).
> I'm wondering if 'qnap' should be added as well.

Do we want to use "qnap" or their ticker "qnapsz"?

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Andrew Lunn Feb. 18, 2014, 9:29 a.m. UTC | #4
On Tue, Feb 18, 2014 at 10:07:04AM +1000, Ben Peddell wrote:
> On 18/02/2014 8:34 AM, Josh Cartwright wrote:
> > On Tue, Feb 18, 2014 at 07:45:35AM +1000, klightspeed@killerwolves.net wrote:
> >> From: Andrew Lunn <andrew@lunn.ch>
> >>
> >> The following patches make use of vendor names ricoh, ssi and
> >> synology.  Add them to the vendor prefix list.
> >>
> >> Signed-off-by: Andrew Lunn <andrew@lunn.ch>
> >> Acked-by: Jason Cooper <jason@lakedaemon.net>
> >> ---
> >> v2:
> >> Use stock ticker for Ricoh as vendor name
> >> s/Richoh/Ricoh/
> >> ---
> >>  Documentation/devicetree/bindings/vendor-prefixes.txt | 3 +++
> >>  1 file changed, 3 insertions(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> >> index 40ce2df..8427681 100644
> >> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> >> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> >> @@ -72,6 +72,7 @@ ralink	Mediatek/Ralink Technology Corp.
> >>  ramtron	Ramtron International
> >>  realtek Realtek Semiconductor Corp.
> >>  renesas	Renesas Electronics Corporation
> >> +ricoy	Ricoh Co. Ltd.
> >>  rockchip	Fuzhou Rockchip Electronics Co., Ltd
> >>  samsung	Samsung Semiconductor
> >>  sbs	Smart Battery System
> >> @@ -79,11 +80,13 @@ schindler	Schindler
> >>  sil	Silicon Image
> >>  silabs	Silicon Laboratories
> >>  simtek
> >> +sii	Seiko Instruments, Inc.
> >>  sirf	SiRF Technology, Inc.
> >>  snps 	Synopsys, Inc.
> >>  st	STMicroelectronics
> >>  ste	ST-Ericsson
> >>  stericsson	ST-Ericsson
> >> +synology	Synology, Inc.
> > 
> > Oh, great!  Looks like this one is added (see my comment on patch 1).
> > I'm wondering if 'qnap' should be added as well.
> 
> Do we want to use "qnap" or their ticker "qnapsz"?
> 

Hi Ben

Ticker.

It might also make sense to re-arrange the order of these patches, so
that this one comes first and then the power off patch. However it is
not too important, because the patches are probably going to get split
up and go thought different maintainers.

   Andrew
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 40ce2df..8427681 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -72,6 +72,7 @@  ralink	Mediatek/Ralink Technology Corp.
 ramtron	Ramtron International
 realtek Realtek Semiconductor Corp.
 renesas	Renesas Electronics Corporation
+ricoy	Ricoh Co. Ltd.
 rockchip	Fuzhou Rockchip Electronics Co., Ltd
 samsung	Samsung Semiconductor
 sbs	Smart Battery System
@@ -79,11 +80,13 @@  schindler	Schindler
 sil	Silicon Image
 silabs	Silicon Laboratories
 simtek
+sii	Seiko Instruments, Inc.
 sirf	SiRF Technology, Inc.
 snps 	Synopsys, Inc.
 st	STMicroelectronics
 ste	ST-Ericsson
 stericsson	ST-Ericsson
+synology	Synology, Inc.
 ti	Texas Instruments
 tlm	Trusted Logic Mobility
 toshiba	Toshiba Corporation