On Wed, May 6, 2015 at 10:32 AM, Eliad Peller <eliad@xxxxxxxxxx> wrote: > hi Yegor, > > On Mon, May 4, 2015 at 12:16 PM, Yegor Yefremov > <yegorslists@xxxxxxxxxxxxxx> wrote: >> On Wed, Mar 25, 2015 at 2:45 PM, Eliad Peller <eliad@xxxxxxxxxx> wrote: >>> On Wed, Mar 25, 2015 at 3:24 PM, Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> wrote: >>>> On 03/25/2015 01:58 PM, Eliad Peller wrote: >>>>> hi Marc, >>>>> >>>>> On Wed, Mar 25, 2015 at 2:27 PM, Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> wrote: >>>>>> On 12/29/2014 07:24 AM, Eliad Peller wrote: >>>>>>> From: Kobi L <kobi.lev100@xxxxxxxxx> >>>>>>> >>>>>>> Enable ELP authorization in AP mode and enable the use >>>>>>> of the wakeup bit in the ELP register. >>>>>>> >>>>>>> Introduce AP role sleep configuration which is disabled >>>>>>> by default. When configured, it allows the AP to sleep >>>>>>> when ELP is authorized for it. >>>>>>> >>>>>>> Signed-off-by: Kobi Leibovitch <kobi.lev100@xxxxxxxxx> >>>>>>> Signed-off-by: Arik Nemtsov <arik@xxxxxxxxxx> >>>>>>> Signed-off-by: Eliad Peller <eliad@xxxxxxxxxx> >>>>>> >>>>>> [...] >>>>>> >>>>>> >>>>>> This patch requires a new firmware, loading of "wl18xx-conf.bin" from >>>>>> [1] fails with: >>>>>> >>>>>>> wlcore: ERROR configuration binary file size is wrong, expected 1226 got 1221 >>>>>> >>>>> thanks. seems you're right. >>>>> >>>>> anyway, note that this file is not actual fw, but only the default >>>>> configuration params. >>>>> you can just delete it, and the driver will load just fine. >>>>> you can then recreate it with something like: >>>>> cat /sys/kernel/debug/ieee80211/phy0/wlcore/wl18xx/conf > >>>>> /lib/firmware/ti-connectivity/wl18xx-conf.bin >>>>> >>>>> actually, i don't think it makes much sense to have this file in the >>>>> linux-firmware git at all. >>>>> i'll try handling it. >>>> >>>> Thanks, there are more file with size of ~1k, are they optional, too? >>>> What's the "error" message if these optional files are not present? >>>> >>> AFAIK, the other nvs files are basically required for proper operation. >>> >>>> From working with $CUSTOMERS I gained the impression, messages like >>>> "error failed to load...." for optional file don't shine the best light >>>> on Linux, a neutral "using default values" is far better. >>> i agree :) >> >> Added Yaniv Machani in CC. >> >> Any update on the wl18xx-conf.bin issue? This file is still in >> linux-firmware repo. >> > thanks for the reminder :) > > just sent a patch to fallback to default conf in case of invalid/outdated conf. Great. @Yaniv: but how to create a new conf with 1226 bytes for existing ini file? Yegor -- 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
