Can't build drivers/infiniband/hw/ipath/ipath_keys.c on arch/powerpc

Hi,

Just a heads-up, drivers/infiniband/hw/ipath/ipath_keys.c doesn't build
on powerpc because we have don't have bus_to_virt().

(Actually you can't select CONFIG_INFINIBAND_IPATH on mainline because
powerpc doesn't enable PCI_MSI, but we will real soon now, and when we
do this will start breaking people's all modconfig builds).

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person

On Mon, 2006-11-06 at 18:33 +1100, Michael Ellerman wrote:

Hi,

Just a heads-up, drivers/infiniband/hw/ipath/ipath_keys.c doesn't build
on powerpc because we have don't have bus_to_virt().

(Actually you can't select CONFIG_INFINIBAND_IPATH on mainline because
powerpc doesn't enable PCI_MSI, but we will real soon now, and when we
do this will start breaking people's all modconfig builds).

I'm surprised that something as recent as infiniband requires a
long-deprecated function bus_to_virt().

What is it trying to do there that needs that call ?

Ben.

Michael Ellerman wrote:

Hi,

Just a heads-up, drivers/infiniband/hw/ipath/ipath_keys.c doesn't build
on powerpc because we have don't have bus_to_virt().

(Actually you can't select CONFIG_INFINIBAND_IPATH on mainline because
powerpc doesn't enable PCI_MSI, but we will real soon now, and when we
do this will start breaking people's all modconfig builds).

Thanks for the heads-up.  We're working to remove this requirement as we speak.

Regards,
 Robert.

On Sun, 2006-11-05 at 23:42 -0800, Robert Walsh wrote:

Michael Ellerman wrote:

quoted

Hi,

Just a heads-up, drivers/infiniband/hw/ipath/ipath_keys.c doesn't build
on powerpc because we have don't have bus_to_virt().

(Actually you can't select CONFIG_INFINIBAND_IPATH on mainline because
powerpc doesn't enable PCI_MSI, but we will real soon now, and when we
do this will start breaking people's all modconfig builds).

Thanks for the heads-up.  We're working to remove this requirement as we 
speak.

Sweet.

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person

 > I'm surprised that something as recent as infiniband requires a
 > long-deprecated function bus_to_virt().
 > 
 > What is it trying to do there that needs that call ?

Don't ask -- just enjoy the fact that you don't know about this...

On Mon, 2006-11-06 at 08:52 -0800, Roland Dreier wrote:

 > I'm surprised that something as recent as infiniband requires a
 > long-deprecated function bus_to_virt().
 > 
 > What is it trying to do there that needs that call ?

Don't ask -- just enjoy the fact that you don't know about this...

Deal :)

Ben.