On 12/01/2014 07:58 AM, Rafał Miłecki wrote: > There are two reasons for having this header in the common place: > 1) Simplifying drivers that read NVRAM entries. We will be able to > safely call bcm47xx_nvram_* functions without #ifdef-s. > 2) Getting NVRAM driver out of MIPS arch code. This is needed to support > BCM5301X arch which also requires this NVRAM driver. Patch for that > will follow once we get is reviewed. > > Signed-off-by: Rafał Miłecki <zajec5@xxxxxxxxx> > --- > Hi guys, > > As recently discussed (thanks Paul!), we will try yet another (3rd) path for our > NVRAM driver, this time it will be drivers/firmware/. > However this will require some time as I want to give ppl chance to review it > and probably we will need to discuss it too. > > So meanwhile I'd like to move just a header file to the include/linux/. I > believe it won't raise any/too many objections and it should strip down > further discussions about bcm47xx_nvram.c. > --- > arch/mips/bcm47xx/board.c | 2 +- > arch/mips/bcm47xx/nvram.c | 2 +- > arch/mips/bcm47xx/setup.c | 1 - > arch/mips/bcm47xx/sprom.c | 1 - > arch/mips/bcm47xx/time.c | 1 - > arch/mips/include/asm/mach-bcm47xx/bcm47xx.h | 1 + > drivers/bcma/driver_mips.c | 2 +- > drivers/net/ethernet/broadcom/b44.c | 2 +- > drivers/net/ethernet/broadcom/bgmac.c | 2 +- > drivers/ssb/driver_chipcommon_pmu.c | 2 +- > drivers/ssb/driver_mipscore.c | 2 +- > .../mach-bcm47xx => include/linux}/bcm47xx_nvram.h | 19 ++++++++++++++++--- > 12 files changed, 24 insertions(+), 13 deletions(-) > rename {arch/mips/include/asm/mach-bcm47xx => include/linux}/bcm47xx_nvram.h (63%) Acked-by: Hauke Mehrtens <hauke@xxxxxxxxxx>