Re: [PATCH] ARM: imx_v6_v7_defconfig: Add CONFIG_FW_LOADER_USER_HELPER_FALLBACK

[Gary Bisson]

Hi Fabio,

On Thu, Aug 17, 2017 at 5:11 PM, Fabio Estevam <festevam@xxxxxxxxx> wrote:
> Hi Gary,
>
> On Thu, Aug 17, 2017 at 11:04 AM, Gary Bisson
> <gary.bisson@xxxxxxxxxxxxxxxxxxx> wrote:
>> Allows to load firmware files which aren't built inside the kernel.
>>
>> Especially useful for CODA firmware (vpu_fw_imx6q.bin) which is usually
>> located in the rootfs.
>>
>> Signed-off-by: Gary Bisson <gary.bisson@xxxxxxxxxxxxxxxxxxx>
>> ---
>> Hi Shawn,
>>
>> I'm not sure if this patch is acceptable since I've noticed that no
>> configuration currently has that option enabled (why?).
>>
>> As mentioned in the commit, I had to enable that in order to have CODA
>> load the VPU firmware properly.
>>
>> Another option would be to make the CODA driver a module...
>
> Yes, that's what we have currently:
>
> commit a5a56f07c2726cc801efccd23f8df29af04d6d19
> Author: Fabio Estevam <fabio.estevam@xxxxxxx>
> Date:   Fri Jul 7 12:06:45 2017 -0300
>
>     ARM: imx_v6_v7_defconfig: Select the coda driver as module

Ok good, I tested on v4.13-rc5, that's why I had the problem.

I'll make sure to check Shawn's tree next time.

Regards,
Gary