On Fri, May 15, 2015 at 4:38 PM, Julian Calaby <julian.calaby@xxxxxxxxx> wrote: > Hi Oleksij, > > On Fri, May 15, 2015 at 10:35 PM, Oleksij Rempel <linux@xxxxxxxxxxxxxxxx> wrote: >> ... and move dup code from ar5008_phy.c and ar9002_phy.c to phy.c > > A better subject might be: > > ath9k: consolidate common phy functions > > "phy.c" doesn't really mean anything to people outside the developers > of this particular driver. > Jupp. Is a consolidation also possible/wise/doable also for phy.h? ( I see this snippet in your new phy.c file... ) ... +#include "phy.h" +#include "ar9002_phy.h" ... IOW, can ar9002_phy.h move into phy.h? - sed@ - -- 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