2011/7/23 Hauke Mehrtens <hauke@xxxxxxxxxx>: > This adds support for serial console to bcma, when operating on an SoC. > > Signed-off-by: Hauke Mehrtens <hauke@xxxxxxxxxx> > --- > drivers/bcma/bcma_private.h | 8 ++++ > drivers/bcma/driver_chipcommon.c | 48 +++++++++++++++++++++++++++ > drivers/bcma/driver_chipcommon_pmu.c | 26 ++++++++++++++ > drivers/bcma/driver_mips.c | 1 + > include/linux/bcma/bcma_driver_chipcommon.h | 14 ++++++++ > 5 files changed, 97 insertions(+), 0 deletions(-) > > diff --git a/drivers/bcma/bcma_private.h b/drivers/bcma/bcma_private.h > index b97633d..22d3052 100644 > --- a/drivers/bcma/bcma_private.h > +++ b/drivers/bcma/bcma_private.h > @@ -29,6 +29,14 @@ void bcma_init_bus(struct bcma_bus *bus); > /* sprom.c */ > int bcma_sprom_get(struct bcma_bus *bus); > > +/* driver_chipcommon.c */ > +#ifdef CONFIG_BCMA_DRIVER_MIPS > +void bcma_chipco_serial_init(struct bcma_drv_cc *cc); > +#endif /* CONFIG_BCMA_DRIVER_MIPS */ > + > +/* driver_chipcommon_pmu.c */ > +u32 bcma_pmu_alp_clock(struct bcma_drv_cc *cc); > + > #ifdef CONFIG_BCMA_HOST_PCI > /* host_pci.c */ > extern int __init bcma_host_pci_init(void); Not sure, what do you think about this, feel free to comment. My idea was to use bcma_private.h for bcma-internal functions. For example, support for PCI host or SoC host, is something "internal" (as I call it) for bcma. Drivers in theory could be separated modules and I use include/linux/bcma/driver_*.h for them. If following this schema, declarations of bcma_pmu_alp_clock bcma_host_pci_init should be in include/linux/bcma/bcma_driver_chipcommon.h -- Rafał