From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexandre Courbot Subject: Re: [PATCH v2] nouveau: arm: Add MODULE_FIRMWARE for gk20a Date: Thu, 12 Nov 2015 21:53:50 +0900 Message-ID: References: <1443452893-18246-1-git-send-email-kwizart@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Nicolas Chauvet Cc: Ben Skeggs , "nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: linux-tegra@vger.kernel.org On Wed, Nov 11, 2015 at 6:29 PM, Nicolas Chauvet wrote: > 2015-09-30 6:57 GMT+02:00 Alexandre Courbot : >> >> On Tue, Sep 29, 2015 at 12:08 AM, Nicolas Chauvet >> wrote: >> > This patch is needed by initramfs tools to detect >> > the required firmware files for the module. >> > >> > This patch tests for either TEGRA_124_SOC or TEGRA_132_SOC >> > for the firmwares related to the Tegra K1 generation. >> > >> > v2: move the MODULE_FIRMWARE to the nvidia_platform.c file. >> > This will avoid to test for NOUVEAU_PLATFORM_DRIVER >> >> Nice, thanks for doing this change! >> >> Reviewed-by: Alexandre Courbot > > > Thx for the review. > FYI I've tested the patch on top of 4.3+ kernels, and the initramfs > generated with dracut works as expected. > > Is there any other concern with this patch ? > > It's certainly late for 4.4,I would expect it to hit kernel 4.3 at some > point, but I don't think everything is wired from userspace(Xorg/Wayland) > wrt dGPU support. > So It could also wait for 4.5. > What do you think ? I don't see a particular reason to hurry, however this patch has been sitting around for one month now - Ben, is there any particular reason for not merging it? From mboxrd@z Thu Jan 1 00:00:00 1970 From: gnurou@gmail.com (Alexandre Courbot) Date: Thu, 12 Nov 2015 21:53:50 +0900 Subject: [PATCH v2] nouveau: arm: Add MODULE_FIRMWARE for gk20a In-Reply-To: References: <1443452893-18246-1-git-send-email-kwizart@gmail.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Nov 11, 2015 at 6:29 PM, Nicolas Chauvet wrote: > 2015-09-30 6:57 GMT+02:00 Alexandre Courbot : >> >> On Tue, Sep 29, 2015 at 12:08 AM, Nicolas Chauvet >> wrote: >> > This patch is needed by initramfs tools to detect >> > the required firmware files for the module. >> > >> > This patch tests for either TEGRA_124_SOC or TEGRA_132_SOC >> > for the firmwares related to the Tegra K1 generation. >> > >> > v2: move the MODULE_FIRMWARE to the nvidia_platform.c file. >> > This will avoid to test for NOUVEAU_PLATFORM_DRIVER >> >> Nice, thanks for doing this change! >> >> Reviewed-by: Alexandre Courbot > > > Thx for the review. > FYI I've tested the patch on top of 4.3+ kernels, and the initramfs > generated with dracut works as expected. > > Is there any other concern with this patch ? > > It's certainly late for 4.4,I would expect it to hit kernel 4.3 at some > point, but I don't think everything is wired from userspace(Xorg/Wayland) > wrt dGPU support. > So It could also wait for 4.5. > What do you think ? I don't see a particular reason to hurry, however this patch has been sitting around for one month now - Ben, is there any particular reason for not merging it?