Re: [RFC 3/7] bcm47xx-sprom: add Broadcom sprom parser driver
From: Rafał Miłecki <hidden>
Date: 2014-09-01 09:24:40
Also in:
linux-arm-kernel, linux-mips, linux-wireless
On 26 August 2014 23:32, Hauke Mehrtens [off-list ref] wrote:
On 08/25/2014 09:52 AM, Arnd Bergmann wrote:quoted
On Sunday 24 August 2014 23:24:41 Hauke Mehrtens wrote:quoted
drivers/misc/Kconfig | 14 + drivers/misc/Makefile | 1 + drivers/misc/bcm47xx-sprom.c | 690 +++++++++++++++++++++On a similar note, putting the driver into drivers/misc seems suboptimal: misc drivers should by definition be something that is for some odd hardware with no external dependencies on it, whereas your driver seems to be used by multiple other drivers. Would it make sense to put it into drivers/bcma when that is the only bus it is used on?As Jonas already said this code should be used for the bcm53xx ARM code and the bcm47xx MIPS code and it is needed for drivers/bcma/ and drivers/ssb/ (ssb only for old mips devices). Do you have any better idea than putting this to drivers/misc/ ? For the mips SoC we need the code very early and will not use the driver interface but probably directly call the function name.
Ping? Does anyone have any better idea? Both: nvram and sprom drivers will be used by bcm47xx (mips) and bcm53xx (arm). They can't be put in drivers/bcma, as they are used by drivers/ssb as well. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html