lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 20 Dec 2016 13:47:23 +0200 From: Kalle Valo <kvalo@...eaurora.org> To: Arend Van Spriel <arend.vanspriel@...adcom.com> Cc: Pali Rohár <pali.rohar@...il.com>, Daniel Wagner <wagi@...om.org>, "Luis R. Rodriguez" <mcgrof@...nel.org>, Tom Gundersen <teg@...m.no>, Johannes Berg <johannes@...solutions.net>, Ming Lei <ming.lei@...onical.com>, Mimi Zohar <zohar@...ux.vnet.ibm.com>, Bjorn Andersson <bjorn.andersson@...aro.org>, Rafał Miłecki <zajec5@...il.com>, Sebastian Reichel <sre@...nel.org>, Pavel Machek <pavel@....cz>, Michal Kazior <michal.kazior@...to.com>, Ivaylo Dimitrov <ivo.g.dimitrov.75@...il.com>, Aaro Koskinen <aaro.koskinen@....fi>, Tony Lindgren <tony@...mide.com>, linux-wireless <linux-wireless@...r.kernel.org>, Network Development <netdev@...r.kernel.org>, "linux-kernel\@vger.kernel.org" <linux-kernel@...r.kernel.org>, David Woodhouse <dwmw2@...radead.org>, Takashi Iwai <tiwai@...e.de>, Josh Boyer <jwboyer@...oraproject.org>, Dmitry Torokhov <dmitry.torokhov@...il.com> Subject: Re: wl1251 & mac address & calibration data Arend Van Spriel <arend.vanspriel@...adcom.com> writes: > On 18-12-2016 13:09, Pali Rohár wrote: > >> File wl1251-nvs.bin is provided by linux-firmware package and contains >> default data which should be overriden by model specific calibrated >> data. > > Ah. Someone thought it was a good idea to provide the "one ring to rule > them all". Nice. Yes, that was a bad idea. wl1251-nvs.bin in linux-firmware.git should be renamed to wl1251-nvs.bin.example, or something like that, as it should be only installed to a real system only if there's no real calibration data available (only for developers to use, not real users). >> But overwriting that one file is not possible as it next update of >> linux-firmware package will overwrite it back. It break any normal usage >> of package management. >> >> Also it is ridiculously broken by design if some "boot" files needs to >> be overwritten to initialize hardware properly. To not break booting you >> need to overwrite that file before first boot. But without booting >> device you cannot read calibration data. So some hack with autoreboot >> after boot is needed. Providing the calibration data via Device Tree is the proper way to solve this. Yes yes, I know N900 doesn't support it but that's a deficiency in N900, not Linux. >> And how to detect that we have real overwritten calibration data and >> not default one from linux-firmware? Any heuristic or checks will be >> broken here. And no, nothing like you need to reboot your device now >> (and similar concept) from windows world is not accepted. > > Well. After reading and creating calibration data you could just rebind > the driver to the device to have it probed again. Or load wl1251 as a module and make sure calibration data is installed before the module is loaded. LEDE does that with ath10k: https://git.lede-project.org/?p=source.git;a=blob;f=target/linux/ar71xx/base-files/etc/hotplug.d/firmware/11-ath10k-caldata;h=97875bd79a579a0010da3f60324b6ec966fe9c6a;hb=HEAD > But yeah, the default one from linux-firmware should never have been > there in the first place. Agreed. -- Kalle Valo
Powered by blists - more mailing lists