On 09/03/2012 09:15 PM, Rafał Miłecki wrote:

2012/8/31 Hauke Mehrtens [off-list ref]:

quoted

+u32 bcma_gpio_in(struct bcma_bus *bus, u32 mask)
+{
+       unsigned long flags;
+       u32 res = 0;
+
+       spin_lock_irqsave(&bus->gpio_lock, flags);
+       res = bcma_chipco_gpio_in(&bus->drv_cc, mask);
+       spin_unlock_irqrestore(&bus->gpio_lock, flags);
+
+       return res;
+}
+EXPORT_SYMBOL(bcma_gpio_in);

Could we put here direct ops on ChipCommon regs and drop GPIO
functions from driver_chipcommon.c?

So you mean that all accesses to the gpio registers are locked, also
when b43 or bcma accesses them? If so, I will change my patch and test
it on my devices. I am currently not at my development machine and do
not have the test devices here, but I will see if I find some time on
Tuesday when I am home again.

So bcma_chipco_gpio_XXX() should not be exported in the way it is done
now any more and all accesses to the gpio registers should be locked.

Hauke