diff mbox

[PATCHv2,1/8] dt-bindings: use isil prefix for Intersil in vendor-prefixes.txt

Message ID e32fe98a511e3ab8aeed9849bcac33ba14fd8986.1418763328.git.arno@natisbad.org
State Accepted, archived
Commit e94fbe607f8c6bf87358ca84e9631a2fe5ebdc21
Headers show

Commit Message

Arnaud Ebalard Dec. 16, 2014, 9:18 p.m. UTC
This patch fixes DT vendor-prefixes.txt documentation to reference
isil (NASDAQ symbol and the most used prefix inside the kernel) for
Intersil.

It reverts 7c75c1d5e72b ("dt-bindings: Document deprecated device
vendor name to fix related warning").

Signed-off-by: Arnaud Ebalard <arno@natisbad.org>
---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Rob Herring Jan. 13, 2015, 8:02 p.m. UTC | #1
On Tue, Dec 16, 2014 at 3:18 PM, Arnaud Ebalard <arno@natisbad.org> wrote:
>
> This patch fixes DT vendor-prefixes.txt documentation to reference
> isil (NASDAQ symbol and the most used prefix inside the kernel) for
> Intersil.
>
> It reverts 7c75c1d5e72b ("dt-bindings: Document deprecated device
> vendor name to fix related warning").
>
> Signed-off-by: Arnaud Ebalard <arno@natisbad.org>

Applied for 3.20. Thanks.

While there are not interdepencies here, it probably would have been
better to keep this series all together. Other patches have already
been picked up, so I'm just taking this one.

Rob


> ---
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index 423d47418e72..585e5b5dca27 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -81,8 +81,7 @@ innolux       Innolux Corporation
>  intel  Intel Corporation
>  intercontrol   Inter Control Group
>  isee   ISEE 2007 S.L.
> -isil    Intersil (deprecated, use isl)
> -isl    Intersil
> +isil   Intersil
>  karo   Ka-Ro electronics GmbH
>  keymile        Keymile GmbH
>  lacie  LaCie
> --
> 2.1.1
>
>
--
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
Arnaud Ebalard Jan. 13, 2015, 11:21 p.m. UTC | #2
Hi Rob,

Rob Herring <robherring2@gmail.com> writes:

> On Tue, Dec 16, 2014 at 3:18 PM, Arnaud Ebalard <arno@natisbad.org> wrote:
>>
>> This patch fixes DT vendor-prefixes.txt documentation to reference
>> isil (NASDAQ symbol and the most used prefix inside the kernel) for
>> Intersil.
>>
>> It reverts 7c75c1d5e72b ("dt-bindings: Document deprecated device
>> vendor name to fix related warning").
>>
>> Signed-off-by: Arnaud Ebalard <arno@natisbad.org>
>
> Applied for 3.20. Thanks.
>
> While there are not interdepencies here, it probably would have been
> better to keep this series all together.

This was the main reason my v0 was in a single patch. I think I then got
some comment to split things in different patches so that the various
maintainers have their words to say. But I'll try and follow your
approach next time.


> Other patches have already been picked up, so I'm just taking this
> one.

Thanks.

Cheers,

a+
--
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
Rob Herring Jan. 13, 2015, 11:31 p.m. UTC | #3
On Tue, Jan 13, 2015 at 5:21 PM, Arnaud Ebalard <arno@natisbad.org> wrote:
> Hi Rob,
>
> Rob Herring <robherring2@gmail.com> writes:
>
>> On Tue, Dec 16, 2014 at 3:18 PM, Arnaud Ebalard <arno@natisbad.org> wrote:
>>>
>>> This patch fixes DT vendor-prefixes.txt documentation to reference
>>> isil (NASDAQ symbol and the most used prefix inside the kernel) for
>>> Intersil.
>>>
>>> It reverts 7c75c1d5e72b ("dt-bindings: Document deprecated device
>>> vendor name to fix related warning").
>>>
>>> Signed-off-by: Arnaud Ebalard <arno@natisbad.org>
>>
>> Applied for 3.20. Thanks.
>>
>> While there are not interdepencies here, it probably would have been
>> better to keep this series all together.
>
> This was the main reason my v0 was in a single patch. I think I then got
> some comment to split things in different patches so that the various
> maintainers have their words to say. But I'll try and follow your
> approach next time.

I didn't say a single patch, but merge the series together. A separate
commit per maintainer is generally preferred. You just have to be
clear whether you are looking for just acks or to take individual
patches.

I should have chimed in earlier as well, but I missed this initially.

Rob
--
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
Arnaud Ebalard Jan. 14, 2015, 9:25 a.m. UTC | #4
Rob Herring <robherring2@gmail.com> writes:

>>> While there are not interdepencies here, it probably would have been
>>> better to keep this series all together.
>>
>> This was the main reason my v0 was in a single patch. I think I then got
>> some comment to split things in different patches so that the various
>> maintainers have their words to say. But I'll try and follow your
>> approach next time.
>
> I didn't say a single patch, but merge the series together. A separate
> commit per maintainer is generally preferred. You just have to be
> clear whether you are looking for just acks or to take individual
> patches.

Noted.

--
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 423d47418e72..585e5b5dca27 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -81,8 +81,7 @@  innolux	Innolux Corporation
 intel	Intel Corporation
 intercontrol	Inter Control Group
 isee	ISEE 2007 S.L.
-isil    Intersil (deprecated, use isl)
-isl	Intersil
+isil	Intersil
 karo	Ka-Ro electronics GmbH
 keymile	Keymile GmbH
 lacie	LaCie