diff mbox

[U-Boot,v2] MAINTAINERS: integrator+versatile boards

Message ID 1311149857-6725-1-git-send-email-linus.walleij@linaro.org
State Superseded
Headers show

Commit Message

Linus Walleij July 20, 2011, 8:17 a.m. UTC
- Take maintainership of the unlisted integratorap, and the
  integratorcp boards
- Orphan the versatile maintained by Peter Pearse, as he has retired
  from ARM

Cc: Philippe Robin <philippe.robin@arm.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
Changes v1->v2: take care also of the integratorcp board(s).
---
 MAINTAINERS |   11 ++++++-----
 1 files changed, 6 insertions(+), 5 deletions(-)

Comments

Wolfgang Denk July 28, 2011, 7:22 p.m. UTC | #1
Dear Linus Walleij,

In message <1311149857-6725-1-git-send-email-linus.walleij@linaro.org> you wrote:
> - Take maintainership of the unlisted integratorap, and the
>   integratorcp boards
> - Orphan the versatile maintained by Peter Pearse, as he has retired
>   from ARM
> 
> Cc: Philippe Robin <philippe.robin@arm.com>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> Changes v1->v2: take care also of the integratorcp board(s).
> ---
>  MAINTAINERS |   11 ++++++-----
>  1 files changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 143f31b..4ba93ae 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -550,6 +550,9 @@ Unknown / orphaned boards:
>  
>  	EVB64260	MPC7xx_74xx
>  
> +	versatile	ARM926EJ-S
> +	versatile	ARM926EJ-S

I checked in v1 of this patche before - didn't see v2, because you
broke the mail thread.

Anyway - wha't this duplicated entry intended for?


Please check, and provide an incremental patch against current top of
tree.

Best regards,

Wolfgang Denk
Linus Walleij July 28, 2011, 10:42 p.m. UTC | #2
On Thu, Jul 28, 2011 at 9:22 PM, Wolfgang Denk <wd@denx.de> wrote:

>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 143f31b..4ba93ae 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -550,6 +550,9 @@ Unknown / orphaned boards:
>>
>>       EVB64260        MPC7xx_74xx
>>
>> +     versatile       ARM926EJ-S
>> +     versatile       ARM926EJ-S
>
> I checked in v1 of this patche before - didn't see v2, because you
> broke the mail thread.

Sorry I try not to, I will have to figure out how it happens.
gmail threads the patches fine, so I might be poisoned...

> Anyway - wha't this duplicated entry intended for?

Only Peter knows and he has retired. I think it *may* be that there
are two Versatile boards out there: Versatile AB and Versatile PB.
These variants are handled by the versatile split_by_variant.sh
script.

But in that case they should be named differently, like
versatileap and versatilepb and they aren't, and thus it's just
looking weird. If there should be one entry for just "versatile"
or two is a matter of taste in that case, which would you prefer?

I can rewrite the patch any way...

> Please check, and provide an incremental patch against current top of
> tree.

The v4 version of the patch is for the top of the tree and provides
one entry.

Thanks,
Linus Walleij
Linus Walleij July 28, 2011, 10:46 p.m. UTC | #3
On Fri, Jul 29, 2011 at 12:42 AM, Linus Walleij
<linus.walleij@linaro.org> wrote:
>
> On Thu, Jul 28, 2011 at 9:22 PM, Wolfgang Denk <wd@denx.de> wrote:
>> Please check, and provide an incremental patch against current top of
>> tree.
>
> The v4 version of the patch is for the top of the tree and provides
> one entry.

No bah, I was too quick, I see now that the V1 version of the patch was
applied. I'll send out a new one fixing this up.

Thanks,
Linus Walleij
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 143f31b..4ba93ae 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -550,6 +550,9 @@  Unknown / orphaned boards:
 
 	EVB64260	MPC7xx_74xx
 
+	versatile	ARM926EJ-S
+	versatile	ARM926EJ-S
+
 
 #########################################################################
 # ARM Systems:								#
@@ -777,11 +780,9 @@  Sandeep Paulraj <s-paulraj@ti.com>
 	davinci_dm365evm	ARM926EJS
 	davinci_dm6467evm	ARM926EJS
 
-Peter Pearse <peter.pearse@arm.com>
-	integratorcp	All current ARM supplied & supported core modules
-			-see http://www.arm.com/products/DevTools/Hardware_Platforms.html
-	versatile	ARM926EJ-S
-	versatile	ARM926EJ-S
+Linus Walleij <linus.walleij@linaro.org>
+	integratorap	various
+	integratorcp	various
 
 Dave Peverley <dpeverley@mpc-data.co.uk>