Search Linux Wireless

Re: [PATCH] ssb: fix pcicore build breakage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



W dniu 13 maja 2011 15:33 uÅytkownik John W. Linville
<linville@xxxxxxxxxxxxx> napisaÅ:
> On Fri, May 13, 2011 at 03:33:03PM +0200, RafaÅ MiÅecki wrote:
>> 2011/5/13 John W. Linville <linville@xxxxxxxxxxxxx>:
>> > drivers/ssb/main.c:1336: error: 'SSB_PCICORE_BCAST_ADDR' undeclared (first use in this function)
>> > drivers/ssb/main.c:1337: error: 'SSB_PCICORE_BCAST_DATA' undeclared (first use in this function)
>> > drivers/ssb/main.c:1349: error: 'struct ssb_pcicore' has no member named 'dev'
>> >
>> > Reported-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
>> > Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
>> > ---
>> > Âdrivers/ssb/main.c | Â Â2 ++
>> > Â1 files changed, 2 insertions(+), 0 deletions(-)
>> >
>> > diff --git a/drivers/ssb/main.c b/drivers/ssb/main.c
>> > index ee2937c..a53fc4d 100644
>> > --- a/drivers/ssb/main.c
>> > +++ b/drivers/ssb/main.c
>> > @@ -1332,6 +1332,7 @@ EXPORT_SYMBOL(ssb_bus_powerup);
>> > Âstatic void ssb_broadcast_value(struct ssb_device *dev,
>> > Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âu32 address, u32 data)
>> > Â{
>> > +#ifdef CONFIG_SSB_DRIVER_PCICORE
>> > Â Â Â Â/* This is used for both, PCI and ChipCommon core, so be careful. */
>> > Â Â Â ÂBUILD_BUG_ON(SSB_PCICORE_BCAST_ADDR != SSB_CHIPCO_BCAST_ADDR);
>> > Â Â Â ÂBUILD_BUG_ON(SSB_PCICORE_BCAST_DATA != SSB_CHIPCO_BCAST_DATA);
>> > @@ -1340,6 +1341,7 @@ static void ssb_broadcast_value(struct ssb_device *dev,
>> > Â Â Â Âssb_read32(dev, SSB_PCICORE_BCAST_ADDR); /* flush */
>> > Â Â Â Âssb_write32(dev, SSB_PCICORE_BCAST_DATA, data);
>> > Â Â Â Âssb_read32(dev, SSB_PCICORE_BCAST_DATA); /* flush */
>> > +#endif
>> > Â}
>> >
>> > Âvoid ssb_commit_settings(struct ssb_bus *bus)
>>
>> From *quick* review I think it's wrong. WIth this patch we don't do
>> commiting without PCI driver. This is still possible, we just need to
>> use ChipCommon for commiting.
>
> I'm happy for a better patch, but since the build is failing as-is...

Today late night will submit.

-- 
RafaÅ
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux