Vladimir Kondratiev <QCA_vkondrat@xxxxxxxxxxxxxxxx> writes: > On Monday, March 30, 2015 12:38:08 PM Kalle Valo wrote: >> Vladimir Kondratiev <QCA_vkondrat@xxxxxxxxxxxxxxxx> writes: >> >> > Firmware "board" file name has changed from wil6210.board >> > to wil6210.brd by the FW generation tools. >> > >> > Reflect this in the driver. >> > > >> > -#define WIL_FW2_NAME "wil6210.board" /* board & radio parameters */ >> > +#define WIL_FW2_NAME "wil6210.brd" /* board & radio parameters */ >> >> I consider the firmware file names as part of stable user space >> interface (older user space need to work with newer kernel etc) and this >> change breaks that. > > Hardware was not released to the public yet; no one besides developers > in the lab should be affected by this. Actually that doesn't matter as it's common to run few years old kernels, especially in the embedded world. So no matter what, this breaks compatibility with user space and without any real benefit whatsoever. Now will anyone hit this in practise I don't know. I guess this change is ok for now as wil6210 hw is not that common, but in the future please be careful with this. -- Kalle Valo -- 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
