sfx.c driver build failure

2008-07-24 Thread Benjamin Herrenschmidt
Hi Michael !

The sfx driver (which happens to be part of some of our test configs)
fails to build in current Linus tree on powerpc with this error:

/home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c: In function 
‘efx_probe_interrupts’:
/home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c:845: error: lvalue 
required as unary ‘&’ oper
and

Cheers,
Ben.


___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: sfx.c driver build failure

2008-07-25 Thread Stephen Rothwell
On Fri, 25 Jul 2008 16:15:59 +1000 Benjamin Herrenschmidt <[EMAIL PROTECTED]> 
wrote:
>
> The sfx driver (which happens to be part of some of our test configs)
> fails to build in current Linus tree on powerpc with this error:
> 
> /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c: In function 
> ‘efx_probe_interrupts’:
> /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c:845: error: lvalue 
> required as unary ‘&’ operand

Commit 695a6b456307455a10059512208e8ed0d376ecd3 "topology: work around
topology_core_siblings() breakage" in linux-next (from the cpus4096 tree)
comments out the offending lines ...  With the comment "the topology API
is a mess right now".

-- 
Cheers,
Stephen Rothwell[EMAIL PROTECTED]
http://www.canb.auug.org.au/~sfr/


pgpmfbW5J1PAU.pgp
Description: PGP signature
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: sfx.c driver build failure

2008-07-25 Thread Ingo Molnar

* Stephen Rothwell <[EMAIL PROTECTED]> wrote:

> On Fri, 25 Jul 2008 16:15:59 +1000 Benjamin Herrenschmidt <[EMAIL PROTECTED]> 
> wrote:
> >
> > The sfx driver (which happens to be part of some of our test configs)
> > fails to build in current Linus tree on powerpc with this error:
> > 
> > /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c: In function 
> > ‘efx_probe_interrupts’:
> > /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c:845: error: lvalue 
> > required as unary ‘&’ operand
> 
> Commit 695a6b456307455a10059512208e8ed0d376ecd3 "topology: work around 
> topology_core_siblings() breakage" in linux-next (from the cpus4096 
> tree) comments out the offending lines ...  With the comment "the 
> topology API is a mess right now".

there's a patch from Mike Travis that solves it all from grounds up and 
which approach has the ack of Rusty as well, but it's not fully tested 
through yet - i will get to it later today.

Ingo
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: sfx.c driver build failure

2008-07-25 Thread Benjamin Herrenschmidt
On Fri, 2008-07-25 at 08:35 +0100, Michael Brown wrote:
> On Fri, 25 Jul 2008, Benjamin Herrenschmidt wrote:
> > The sfx driver (which happens to be part of some of our test configs) 
> > fails to build in current Linus tree on powerpc with this error:
> > 
> > /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c: In function 
> > ÿÿefx_probe_interruptsÿÿ:
> > /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c:845: error: lvalue 
> > required as unary ÿÿ&ÿÿ oper
> > and
> 
> I'm not the maintainer of that code; I wrote the original version, but 
> it's now maintained by Solarflare.
> 
> I don't see anything on that line that ought to be architecture-specific.  
> Could you send me the preprocessed efx.c from your powerpc build?

Apparently, it's a know generic breakage of the topology stuff,
it's beeing sorted out.

Cheers,
Ben.


___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: sfx.c driver build failure

2008-07-25 Thread Michael Brown
On Fri, 25 Jul 2008, Benjamin Herrenschmidt wrote:
> The sfx driver (which happens to be part of some of our test configs) 
> fails to build in current Linus tree on powerpc with this error:
> 
> /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c: In function 
> ÿÿefx_probe_interruptsÿÿ:
> /home/benh/kernels/linux-powerpc/drivers/net/sfc/efx.c:845: error: lvalue 
> required as unary ÿÿ&ÿÿ oper
> and

I'm not the maintainer of that code; I wrote the original version, but 
it's now maintained by Solarflare.

I don't see anything on that line that ought to be architecture-specific.  
Could you send me the preprocessed efx.c from your powerpc build?

Michael___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev