From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752892AbbKBJKs (ORCPT ); Mon, 2 Nov 2015 04:10:48 -0500 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:43916 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752490AbbKBJKo (ORCPT ); Mon, 2 Nov 2015 04:10:44 -0500 From: Vincent ABRIOU To: Takashi Iwai , Emil Velikov CC: "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , Benjamin Gaignard Date: Mon, 2 Nov 2015 10:10:27 +0100 Subject: Re: [PATCH] drm/sti: Remove select of CONFIG_FW_LOADER_USER_HELPER_FALLBACK Thread-Topic: [PATCH] drm/sti: Remove select of CONFIG_FW_LOADER_USER_HELPER_FALLBACK Thread-Index: AdEVTlKqHcfG/r5MSnCOA7J2Z8SPTg== Message-ID: <56372883.2020801@st.com> References: <1445235370-6717-1-git-send-email-tiwai@suse.de> <56322B6F.5080101@st.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 acceptlanguage: en-US Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.15.21,1.0.33,0.0.0000 definitions=2015-11-02_05:2015-10-29,2015-11-02,1970-01-01 signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id tA29At3J030243 Hi Takashi, I made a rework to load the firmware at a different time, when file system is available. I am now able to remove FW_LOADER_USER_HELPER_FALLBACK. I will submit a new patch for this. Vincent On 10/29/2015 03:53 PM, Takashi Iwai wrote: > On Thu, 29 Oct 2015 15:37:51 +0100, > Emil Velikov wrote: >> >> On 29 October 2015 at 14:21, Vincent ABRIOU wrote: >>> Hi Takashi, >>> >>> Removing FW_LOADER_USER_HELPER_FALLBACK leads to a failure in our HQVDP >>> firmware execution. >>> Indeed, our firmware is not built-in. It is a proprietary firmware >>> uploaded into the file system that's why we need the >>> USER_HELPER_FALLBACK to be able to load it once file system is available. >>> >> Hmm most other DRM drivers also require firmware. Whist some allow the >> firmware to be picked in initrd it's not a strict requirement. >> So I'm wondering how come there hasn't been (m)any reports, >> considering that neither one sets USER_HELPER_FALLBACK. >> >> Perhaps they also need it, or something in the sti module is done >> differently ? Just some food for thought. > > It's the option each user decides to set or not, depending on the > deployed system. Most of PCs don't need them, and actually enabling > this option causes troubles for them. On other embedded systems, this > might be still needed. So, it's the system setup issue, and not the > thing a driver needs to care. > > Imagine that your driver has "select EXT3_FS" because your system > requires it; without that option, it won't boot, OMG! > Is it the right thing? Obviously no. The same logic is applied to > this case, too. > > > Takashi > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel >{.n++%ݶw{.n+{G{ayʇڙ,jfhz_(階ݢj"mG?&~iOzv^m ?I