PATCH: ECS P6IWP-Fe

[Anders Jenbo]

This patch adds the ECS P6IWP-Fe board to coreboot.

Signed-off-by: Anders Jenbo 
---

Anders Jenbo

[Paul Menzel]

Dear Anders,

Am Mittwoch, den 09.06.2010, 01:57 +0200 schrieb Anders Jenbo:
> This patch adds the ECS P6IWP-Fe board to coreboot.

great! Thank you. Could you provide a link to the product page of this
board board? Is there already a Wiki page for this board on what is
supported and so on?

[…]
> Property changes on: src/mainboard/ecs/Kconfig
> ___________________________________________________________________
> Added: svn:executable

This is done to a lot of files. Is this necessary?

[…]

Thanks again,

Paul

[Stefan Reinauer]

On 6/9/10 1:57 AM, Anders Jenbo wrote:
> This patch adds the ECS P6IWP-Fe board to coreboot.
>
> Signed-off-by: Anders Jenbo

Thanks a lot!

Acked-by: Stefan Reinauer 
and checked in as r5623

But please add a license header to the device tree as well in an extra
patch..

Stefan

[Joseph Smith]

On Wed, 09 Jun 2010 10:09:21 +0200, Stefan Reinauer 
wrote:
> On 6/9/10 1:57 AM, Anders Jenbo wrote:
> > This patch adds the ECS P6IWP-Fe board to coreboot.
> >
> > Signed-off-by: Anders Jenbo
>
> Thanks a lot!
>
> Acked-by: Stefan Reinauer 
> and checked in as r5623
>
> But please add a license header to the device tree as well in an extra
> patch..

Yes thanks Anders :-)
One more suggestion. I think from this point forward there is really no
excuse for any Intel 440bx and up to be using romcc. I feel there should be
a new rule, any new boards 440bx and up need to be running CAR for a Ack.
Other wise that will be just one more board to back paddle to CAR. Anyone
else have a thought on this?

[Stefan Reinauer]

On 09.06.2010, at 12:47, Joseph Smith  wrote:
> On Wed, 09 Jun 2010 10:09:21 +0200, Stefan Reinauer <[e-mail]
> >
>
> wrote:
> > On 6/9/10 1:57 AM, Anders Jenbo wrote:
> > > This patch adds the ECS P6IWP-Fe board to coreboot.
> > >
> > > Signed-off-by: Anders Jenbo
> >
> > Thanks a lot!
> >
> > Acked-by: Stefan Reinauer 
> > and checked in as r5623
> >
> > But please add a license header to the device tree as well in an  
> > extra
> > patch..
>
> Yes thanks Anders :-)
> One more suggestion. I think from this point forward there is really  
> no
> excuse for any Intel 440bx and up to be using romcc. I feel there  
> should be
> a new rule, any new boards 440bx and up need to be running CAR for a  
> Ack.
> Other wise that will be just one more board to back paddle to CAR.  
> Anyone
> else have a thought on this?

Since people always copy from existing boards and keep changing them  
until it works, a patch that moves all existing 440 boards to CAR  
would most likely solve the problem better than a rule. Want to give  
it a try?

Stefan
>

[Patrick Georgi]

Am 09.06.2010 13:00, schrieb Stefan Reinauer:
> Since people always copy from existing boards and keep changing them
> until it works, a patch that moves all existing 440 boards to CAR would
> most likely solve the problem better than a rule. Want to give it a try?

This doesn't catch the boards that were copied months ago (but of course
it helps)

Patrick

[Joseph Smith]

On Wed, 9 Jun 2010 13:00:26 +0200, Stefan Reinauer
 wrote:
> On 09.06.2010, at 12:47, Joseph Smith  wrote:
> > On Wed, 09 Jun 2010 10:09:21 +0200, Stefan Reinauer
>
> <[e-mail]
> > >
> >
> > wrote:
> > > On 6/9/10 1:57 AM, Anders Jenbo wrote:
> > > > This patch adds the ECS P6IWP-Fe board to coreboot.
> > > >
> > > > Signed-off-by: Anders Jenbo
> > >
> > > Thanks a lot!
> > >
> > > Acked-by: Stefan Reinauer 
> > > and checked in as r5623
> > >
> > > But please add a license header to the device tree as well in an
> > > extra
> > > patch..
> >
> > Yes thanks Anders :-)
> > One more suggestion. I think from this point forward there is really
> > no
> > excuse for any Intel 440bx and up to be using romcc. I feel there
> > should be
> > a new rule, any new boards 440bx and up need to be running CAR for a
> > Ack.
> > Other wise that will be just one more board to back paddle to CAR.
> > Anyone
> > else have a thought on this?
>
> Since people always copy from existing boards and keep changing them
> until it works, a patch that moves all existing 440 boards to CAR
> would most likely solve the problem better than a rule. Want to give
> it a try?

I could but not sure how much sense that would make considering I do not
have a 440bx to test....

On the other hand I do have some i810's, That chip I would be glad to
convert :-)

[Joseph Smith]

On Wed, 09 Jun 2010 07:16:48 -0400, Joseph Smith 
wrote:
> On Wed, 9 Jun 2010 13:00:26 +0200, Stefan Reinauer
>  wrote:
> > On 09.06.2010, at 12:47, Joseph Smith  wrote:
> > > On Wed, 09 Jun 2010 10:09:21 +0200, Stefan Reinauer
> >
> > <[e-mail]
> > > >
> > >
> > > wrote:
> > > > On 6/9/10 1:57 AM, Anders Jenbo wrote:
> > > > > This patch adds the ECS P6IWP-Fe board to coreboot.
> > > > >
> > > > > Signed-off-by: Anders Jenbo
> > > >
> > > > Thanks a lot!
> > > >
> > > > Acked-by: Stefan Reinauer 
> > > > and checked in as r5623
> > > >
> > > > But please add a license header to the device tree as well in an
> > > > extra
> > > > patch..
> > >
> > > Yes thanks Anders :-)
> > > One more suggestion. I think from this point forward there is really
> > > no
> > > excuse for any Intel 440bx and up to be using romcc. I feel there
> > > should be
> > > a new rule, any new boards 440bx and up need to be running CAR for a
> > > Ack.
> > > Other wise that will be just one more board to back paddle to CAR.
> > > Anyone
> > > else have a thought on this?
> >
> > Since people always copy from existing boards and keep changing them
> > until it works, a patch that moves all existing 440 boards to CAR
> > would most likely solve the problem better than a rule. Want to give
> > it a try?
>
> I could but not sure how much sense that would make considering I do not
> have a 440bx to test....
>
> On the other hand I do have some i810's, That chip I would be glad to
> convert :-)

Yes, this is going to be a new side project. 
Hey maybe a i815 port will even come out of it :-)