* [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-11 4:42 ianwmorrison 2018-04-11 9:24 ` Jani Nikula ` (2 more replies) 0 siblings, 3 replies; 31+ messages in thread From: ianwmorrison @ 2018-04-11 4:42 UTC (permalink / raw) To: rodrigo.vivi, anusha.srivatsa, michal.wajdeczko, gregkh, airlied, jani.nikula, joonas.lahtinen Cc: linux-kernel, stable, intel-gfx, dri-devel, ianwmorrison From: Ian W MORRISON <ianwmorrison@gmail.com> As the Geminilake firmware is now merged to linux-firmware.git use MODUE_FIRMWARE to load the firmware. This removes the error message in the dmesg log: i915 0000:00:02.0: Direct firmware load for i915/glk_dmc_ver1_04.bin failed with error -2 i915 0000:00:02.0: Failed to load DMC firmware i915/glk_dmc_ver1_04.bin. Disabling runtime power management. i915 0000:00:02.0: DMC firmware homepage: https://01.org/linuxgraphics/downloads/firmware and now shows that the firmware has correctly loaded: [drm] Finished loading DMC firmware i915/glk_dmc_ver1_04.bin (v1.4) Cc: stable@vger.kernel.org Signed-off-by: Ian W MORRISON <ianwmorrison@gmail.com> --- drivers/gpu/drm/i915/intel_csr.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c index 41e6c75a7f3c..f9550ea46c26 100644 --- a/drivers/gpu/drm/i915/intel_csr.c +++ b/drivers/gpu/drm/i915/intel_csr.c @@ -35,6 +35,7 @@ */ #define I915_CSR_GLK "i915/glk_dmc_ver1_04.bin" +MODULE_FIRMWARE(I915_CSR_GLK); #define GLK_CSR_VERSION_REQUIRED CSR_VERSION(1, 4) #define I915_CSR_CNL "i915/cnl_dmc_ver1_07.bin" -- 2.11.0 ^ permalink raw reply related [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-11 4:42 [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake ianwmorrison @ 2018-04-11 9:24 ` Jani Nikula 2018-04-11 10:54 ` ✓ Fi.CI.BAT: success for series starting with [RESEND,1/1] " Patchwork 2018-04-11 12:37 ` ✓ Fi.CI.IGT: " Patchwork 2 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-11 9:24 UTC (permalink / raw) To: ianwmorrison, rodrigo.vivi, anusha.srivatsa, michal.wajdeczko, gregkh, airlied, joonas.lahtinen Cc: linux-kernel, stable, intel-gfx, dri-devel, ianwmorrison On Wed, 11 Apr 2018, ianwmorrison@gmail.com wrote: > From: Ian W MORRISON <ianwmorrison@gmail.com> > > As the Geminilake firmware is now merged to linux-firmware.git > use MODUE_FIRMWARE to load the firmware. > > This removes the error message in the dmesg log: > > i915 0000:00:02.0: Direct firmware load for > i915/glk_dmc_ver1_04.bin failed with error -2 > i915 0000:00:02.0: Failed to load DMC firmware > i915/glk_dmc_ver1_04.bin. Disabling runtime power management. > i915 0000:00:02.0: DMC firmware homepage: > https://01.org/linuxgraphics/downloads/firmware > > and now shows that the firmware has correctly loaded: > > [drm] Finished loading DMC firmware i915/glk_dmc_ver1_04.bin (v1.4) > > Cc: stable@vger.kernel.org NAK on indiscriminate Cc: stable. There are zero guarantees that older kernels will work with whatever firmware you throw at them. Anusha, Rodrigo, are we even ready to do this in our -next? BR, Jani. PS. How is this a "RESEND"? I haven't seen this before. > Signed-off-by: Ian W MORRISON <ianwmorrison@gmail.com> > --- > drivers/gpu/drm/i915/intel_csr.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c > index 41e6c75a7f3c..f9550ea46c26 100644 > --- a/drivers/gpu/drm/i915/intel_csr.c > +++ b/drivers/gpu/drm/i915/intel_csr.c > @@ -35,6 +35,7 @@ > */ > > #define I915_CSR_GLK "i915/glk_dmc_ver1_04.bin" > +MODULE_FIRMWARE(I915_CSR_GLK); > #define GLK_CSR_VERSION_REQUIRED CSR_VERSION(1, 4) > > #define I915_CSR_CNL "i915/cnl_dmc_ver1_07.bin" -- Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-11 9:24 ` Jani Nikula 0 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-11 9:24 UTC (permalink / raw) To: rodrigo.vivi, anusha.srivatsa, michal.wajdeczko, gregkh, airlied, joonas.lahtinen Cc: dri-devel, ianwmorrison, intel-gfx, linux-kernel, stable On Wed, 11 Apr 2018, ianwmorrison@gmail.com wrote: > From: Ian W MORRISON <ianwmorrison@gmail.com> > > As the Geminilake firmware is now merged to linux-firmware.git > use MODUE_FIRMWARE to load the firmware. > > This removes the error message in the dmesg log: > > i915 0000:00:02.0: Direct firmware load for > i915/glk_dmc_ver1_04.bin failed with error -2 > i915 0000:00:02.0: Failed to load DMC firmware > i915/glk_dmc_ver1_04.bin. Disabling runtime power management. > i915 0000:00:02.0: DMC firmware homepage: > https://01.org/linuxgraphics/downloads/firmware > > and now shows that the firmware has correctly loaded: > > [drm] Finished loading DMC firmware i915/glk_dmc_ver1_04.bin (v1.4) > > Cc: stable@vger.kernel.org NAK on indiscriminate Cc: stable. There are zero guarantees that older kernels will work with whatever firmware you throw at them. Anusha, Rodrigo, are we even ready to do this in our -next? BR, Jani. PS. How is this a "RESEND"? I haven't seen this before. > Signed-off-by: Ian W MORRISON <ianwmorrison@gmail.com> > --- > drivers/gpu/drm/i915/intel_csr.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c > index 41e6c75a7f3c..f9550ea46c26 100644 > --- a/drivers/gpu/drm/i915/intel_csr.c > +++ b/drivers/gpu/drm/i915/intel_csr.c > @@ -35,6 +35,7 @@ > */ > > #define I915_CSR_GLK "i915/glk_dmc_ver1_04.bin" > +MODULE_FIRMWARE(I915_CSR_GLK); > #define GLK_CSR_VERSION_REQUIRED CSR_VERSION(1, 4) > > #define I915_CSR_CNL "i915/cnl_dmc_ver1_07.bin" -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-11 9:24 ` Jani Nikula @ 2018-04-11 10:41 ` Ian W MORRISON -1 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-11 10:41 UTC (permalink / raw) To: Jani Nikula Cc: rodrigo.vivi, anusha.srivatsa, michal.wajdeczko, Greg KH, airlied, joonas.lahtinen, linux-kernel, stable, intel-gfx, dri-devel <snip> > > NAK on indiscriminate Cc: stable. There are zero guarantees that older > kernels will work with whatever firmware you throw at them. > I included 'Cc: stable' so the patch would get added to the v4.16 and v4.15 kernels which I have tested with the patch. I found that earlier kernels didn't support the 'linux-firmware' package required to get wifi working on Intel's new Gemini Lake NUC. > > PS. How is this a "RESEND"? I haven't seen this before. > It is a 'RESEND' for that very reason. I initially sent the patch to the same people as a similar patch (https://patchwork.kernel.org/patch/10143637/) however after realising this omitted required addresses I added them and resent the patch. Best regards, Ian ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-11 10:41 ` Ian W MORRISON 0 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-11 10:41 UTC (permalink / raw) To: Jani Nikula Cc: anusha.srivatsa, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, rodrigo.vivi, michal.wajdeczko <snip> > > NAK on indiscriminate Cc: stable. There are zero guarantees that older > kernels will work with whatever firmware you throw at them. > I included 'Cc: stable' so the patch would get added to the v4.16 and v4.15 kernels which I have tested with the patch. I found that earlier kernels didn't support the 'linux-firmware' package required to get wifi working on Intel's new Gemini Lake NUC. > > PS. How is this a "RESEND"? I haven't seen this before. > It is a 'RESEND' for that very reason. I initially sent the patch to the same people as a similar patch (https://patchwork.kernel.org/patch/10143637/) however after realising this omitted required addresses I added them and resent the patch. Best regards, Ian _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-11 10:41 ` Ian W MORRISON (?) @ 2018-04-11 12:27 ` Jani Nikula 2018-04-11 13:04 ` Ian W MORRISON 2018-04-16 22:47 ` Srivatsa, Anusha -1 siblings, 2 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-11 12:27 UTC (permalink / raw) To: Ian W MORRISON Cc: rodrigo.vivi, anusha.srivatsa, michal.wajdeczko, Greg KH, airlied, joonas.lahtinen, linux-kernel, stable, intel-gfx, dri-devel On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > <snip> > >> >> NAK on indiscriminate Cc: stable. There are zero guarantees that older >> kernels will work with whatever firmware you throw at them. >> > > I included 'Cc: stable' so the patch would get added to the v4.16 and > v4.15 kernels which I have tested with the patch. I found that earlier > kernels didn't support the 'linux-firmware' package required to get > wifi working on Intel's new Gemini Lake NUC. You realize that this patch should have nothing to do with wifi? Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate the specific versions of stable it is appropriate for. BR, Jani. > >> >> PS. How is this a "RESEND"? I haven't seen this before. >> > > It is a 'RESEND' for that very reason. I initially sent the patch to > the same people as > a similar patch (https://patchwork.kernel.org/patch/10143637/) however > after realising > this omitted required addresses I added them and resent the patch. > > Best regards, > Ian -- Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-11 12:27 ` Jani Nikula @ 2018-04-11 13:04 ` Ian W MORRISON 2018-04-16 22:47 ` Srivatsa, Anusha 1 sibling, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-11 13:04 UTC (permalink / raw) To: Jani Nikula Cc: rodrigo.vivi, anusha.srivatsa, michal.wajdeczko, Greg KH, airlied, joonas.lahtinen, linux-kernel, stable, intel-gfx, dri-devel On 11 April 2018 at 22:27, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> <snip> >> >>> >>> NAK on indiscriminate Cc: stable. There are zero guarantees that older >>> kernels will work with whatever firmware you throw at them. >>> >> >> I included 'Cc: stable' so the patch would get added to the v4.16 and >> v4.15 kernels which I have tested with the patch. I found that earlier >> kernels didn't support the 'linux-firmware' package required to get >> wifi working on Intel's new Gemini Lake NUC. > > You realize that this patch should have nothing to do with wifi? > Yes. Runtime power management depends on the patch but when distros such as Ubuntu are installed on a Intel Gemini Lake NUC as an example, wifi/bluetooth depends on Intel Wireless-AC 9462 which in turn depends on v4.15 as a minimum so my logic was to add the patch to 4.15 and later. > Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate > the specific versions of stable it is appropriate for. > Thanks you.. Best regards, Ian ^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-11 12:27 ` Jani Nikula 2018-04-11 13:04 ` Ian W MORRISON @ 2018-04-16 22:47 ` Srivatsa, Anusha 2018-04-17 9:02 ` Jani Nikula 1 sibling, 1 reply; 31+ messages in thread From: Srivatsa, Anusha @ 2018-04-16 22:47 UTC (permalink / raw) To: Jani Nikula, Ian W MORRISON Cc: Vivi, Rodrigo, Wajdeczko, Michal, Greg KH, airlied, joonas.lahtinen, linux-kernel, stable, intel-gfx, dri-devel >-----Original Message----- >From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >Sent: Wednesday, April 11, 2018 5:27 AM >To: Ian W MORRISON <ianwmorrison@gmail.com> >Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha ><anusha.srivatsa@intel.com>; Wajdeczko, Michal ><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >airlied@linux.ie; joonas.lahtinen@linux.intel.com; linux-kernel@vger.kernel.org; >stable@vger.kernel.org; intel-gfx@lists.freedesktop.org; dri- >devel@lists.freedesktop.org >Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >Geminilake > >On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> <snip> >> >>> >>> NAK on indiscriminate Cc: stable. There are zero guarantees that >>> older kernels will work with whatever firmware you throw at them. >>> >> >> I included 'Cc: stable' so the patch would get added to the v4.16 and >> v4.15 kernels which I have tested with the patch. I found that earlier >> kernels didn't support the 'linux-firmware' package required to get >> wifi working on Intel's new Gemini Lake NUC. > >You realize that this patch should have nothing to do with wifi? > >Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate the specific >versions of stable it is appropriate for. Hi Jani, The stable kernel version is 4.12 and beyond. It is appropriate to add the CC: stable in my opinion Anusha >BR, >Jani. > >> >>> >>> PS. How is this a "RESEND"? I haven't seen this before. >>> >> >> It is a 'RESEND' for that very reason. I initially sent the patch to >> the same people as a similar patch >> (https://patchwork.kernel.org/patch/10143637/) however after realising >> this omitted required addresses I added them and resent the patch. >> >> Best regards, >> Ian > >-- >Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-16 22:47 ` Srivatsa, Anusha @ 2018-04-17 9:02 ` Jani Nikula 0 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-17 9:02 UTC (permalink / raw) To: Srivatsa, Anusha, Ian W MORRISON Cc: Vivi, Rodrigo, Wajdeczko, Michal, Greg KH, airlied, joonas.lahtinen, linux-kernel, stable, intel-gfx, dri-devel On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: >>-----Original Message----- >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >>Sent: Wednesday, April 11, 2018 5:27 AM >>To: Ian W MORRISON <ianwmorrison@gmail.com> >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha >><anusha.srivatsa@intel.com>; Wajdeczko, Michal >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; linux-kernel@vger.kernel.org; >>stable@vger.kernel.org; intel-gfx@lists.freedesktop.org; dri- >>devel@lists.freedesktop.org >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >>Geminilake >> >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >>> <snip> >>> >>>> >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that >>>> older kernels will work with whatever firmware you throw at them. >>>> >>> >>> I included 'Cc: stable' so the patch would get added to the v4.16 and >>> v4.15 kernels which I have tested with the patch. I found that earlier >>> kernels didn't support the 'linux-firmware' package required to get >>> wifi working on Intel's new Gemini Lake NUC. >> >>You realize that this patch should have nothing to do with wifi? >> >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate the specific >>versions of stable it is appropriate for. > > Hi Jani, > > The stable kernel version is 4.12 and beyond. > It is appropriate to add the CC: stable in my opinion Who tested the firmware with v4.12 and later? We only have the CI results against *current* drm-tip. We don't even know about v4.16. I'm not going to ack and take responsibility for the stable backports unless someone actually comes forward with credible Tested-bys. BR, Jani. > > Anusha >>BR, >>Jani. >> >>> >>>> >>>> PS. How is this a "RESEND"? I haven't seen this before. >>>> >>> >>> It is a 'RESEND' for that very reason. I initially sent the patch to >>> the same people as a similar patch >>> (https://patchwork.kernel.org/patch/10143637/) however after realising >>> this omitted required addresses I added them and resent the patch. >>> >>> Best regards, >>> Ian >> >>-- >>Jani Nikula, Intel Open Source Technology Center -- Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-17 9:02 ` Jani Nikula 0 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-17 9:02 UTC (permalink / raw) To: Srivatsa, Anusha, Ian W MORRISON Cc: airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Vivi, Rodrigo, Wajdeczko, Michal On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: >>-----Original Message----- >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >>Sent: Wednesday, April 11, 2018 5:27 AM >>To: Ian W MORRISON <ianwmorrison@gmail.com> >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha >><anusha.srivatsa@intel.com>; Wajdeczko, Michal >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; linux-kernel@vger.kernel.org; >>stable@vger.kernel.org; intel-gfx@lists.freedesktop.org; dri- >>devel@lists.freedesktop.org >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >>Geminilake >> >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >>> <snip> >>> >>>> >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that >>>> older kernels will work with whatever firmware you throw at them. >>>> >>> >>> I included 'Cc: stable' so the patch would get added to the v4.16 and >>> v4.15 kernels which I have tested with the patch. I found that earlier >>> kernels didn't support the 'linux-firmware' package required to get >>> wifi working on Intel's new Gemini Lake NUC. >> >>You realize that this patch should have nothing to do with wifi? >> >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate the specific >>versions of stable it is appropriate for. > > Hi Jani, > > The stable kernel version is 4.12 and beyond. > It is appropriate to add the CC: stable in my opinion Who tested the firmware with v4.12 and later? We only have the CI results against *current* drm-tip. We don't even know about v4.16. I'm not going to ack and take responsibility for the stable backports unless someone actually comes forward with credible Tested-bys. BR, Jani. > > Anusha >>BR, >>Jani. >> >>> >>>> >>>> PS. How is this a "RESEND"? I haven't seen this before. >>>> >>> >>> It is a 'RESEND' for that very reason. I initially sent the patch to >>> the same people as a similar patch >>> (https://patchwork.kernel.org/patch/10143637/) however after realising >>> this omitted required addresses I added them and resent the patch. >>> >>> Best regards, >>> Ian >> >>-- >>Jani Nikula, Intel Open Source Technology Center -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-17 9:02 ` Jani Nikula (?) @ 2018-04-17 14:14 ` Joonas Lahtinen 2018-04-20 5:47 ` Ian W MORRISON -1 siblings, 1 reply; 31+ messages in thread From: Joonas Lahtinen @ 2018-04-17 14:14 UTC (permalink / raw) To: Srivatsa, Anusha, Ian W MORRISON, Jani Nikula Cc: Vivi, Rodrigo, Wajdeczko, Michal, Greg KH, airlied, linux-kernel, stable, intel-gfx, dri-devel Quoting Jani Nikula (2018-04-17 12:02:52) > On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: > >>-----Original Message----- > >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] > >>Sent: Wednesday, April 11, 2018 5:27 AM > >>To: Ian W MORRISON <ianwmorrison@gmail.com> > >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha > >><anusha.srivatsa@intel.com>; Wajdeczko, Michal > >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; > >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; linux-kernel@vger.kernel.org; > >>stable@vger.kernel.org; intel-gfx@lists.freedesktop.org; dri- > >>devel@lists.freedesktop.org > >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for > >>Geminilake > >> > >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > >>> <snip> > >>> > >>>> > >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that > >>>> older kernels will work with whatever firmware you throw at them. > >>>> > >>> > >>> I included 'Cc: stable' so the patch would get added to the v4.16 and > >>> v4.15 kernels which I have tested with the patch. I found that earlier > >>> kernels didn't support the 'linux-firmware' package required to get > >>> wifi working on Intel's new Gemini Lake NUC. > >> > >>You realize that this patch should have nothing to do with wifi? > >> > >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate the specific > >>versions of stable it is appropriate for. > > > > Hi Jani, > > > > The stable kernel version is 4.12 and beyond. > > It is appropriate to add the CC: stable in my opinion > > Who tested the firmware with v4.12 and later? We only have the CI > results against *current* drm-tip. We don't even know about v4.16. > > I'm not going to ack and take responsibility for the stable backports > unless someone actually comes forward with credible Tested-bys. And even then, some distros will be surprised of the new MODULE_FIRMWARE and will need to update the linux-firmware package, too. Regards, Joonas > > BR, > Jani. > > > > > > Anusha > >>BR, > >>Jani. > >> > >>> > >>>> > >>>> PS. How is this a "RESEND"? I haven't seen this before. > >>>> > >>> > >>> It is a 'RESEND' for that very reason. I initially sent the patch to > >>> the same people as a similar patch > >>> (https://patchwork.kernel.org/patch/10143637/) however after realising > >>> this omitted required addresses I added them and resent the patch. > >>> > >>> Best regards, > >>> Ian > >> > >>-- > >>Jani Nikula, Intel Open Source Technology Center > > -- > Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-17 14:14 ` Joonas Lahtinen @ 2018-04-20 5:47 ` Ian W MORRISON 2018-04-20 7:50 ` Jani Nikula 0 siblings, 1 reply; 31+ messages in thread From: Ian W MORRISON @ 2018-04-20 5:47 UTC (permalink / raw) To: Joonas Lahtinen Cc: Srivatsa, Anusha, Jani Nikula, Vivi, Rodrigo, Wajdeczko, Michal, Greg KH, airlied, linux-kernel, stable, intel-gfx, dri-devel On 18 April 2018 at 00:14, Joonas Lahtinen <joonas.lahtinen@linux.intel.com> wrote: > Quoting Jani Nikula (2018-04-17 12:02:52) >> On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: >> >>-----Original Message----- >> >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >> >>Sent: Wednesday, April 11, 2018 5:27 AM >> >>To: Ian W MORRISON <ianwmorrison@gmail.com> >> >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha >> >><anusha.srivatsa@intel.com>; Wajdeczko, Michal >> >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >> >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; linux-kernel@vger.kernel.org; >> >>stable@vger.kernel.org; intel-gfx@lists.freedesktop.org; dri- >> >>devel@lists.freedesktop.org >> >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >> >>Geminilake <snip> In summary so far: Jani: > NAK on indiscriminate Cc: stable. There are zero guarantees that > older kernels will work with whatever firmware you throw at them. > Who tested the firmware with v4.12 and later? We only have the CI > results against *current* drm-tip. We don't even know about v4.16. > I'm not going to ack and take responsibility for the stable backports > unless someone actually comes forward with credible Tested-bys. Anusha: > The stable kernel version is 4.12 and beyond. > It is appropriate to add the CC: stable in my opinion Joonas: > And even then, some distros will be surprised of the new MODULE_FIRMWARE > and will need to update the linux-firmware package, too. I've performed backport testing and some additional analysis as follows: The DMC firmware for GLK was initially included in 4.11 (commit: dbb28b5c3d3cb945a63030fab8d3894cf335ce19). Then the firmware version was upgraded to 1.03 in 4.12 (commit: f4a791819ed00a749a90387aa139706a507aa690). However MODULE_FIRMWARE for the GLK DMC firmware was also removed in 4.12 (commit: d9321a03efcda867b3a8c6327e01808516f0acd7) together with the firmware version being bumped to 1.04 (commit: aebfd1d37194e00d4c417e7be97efeb736cd9c04). The patch below effectively reverts commit d9321a03 because the GLK firmware is now available in the linux-firmware repository. To test stable backports I've used Ubuntu 18.04 (Beta 2) userspace with both Ubuntu (generic) and self-compiled mainline (patched) kernels. The conclusion was that the patch works across 4.12 to 4.17-rc1 kernels additionally displaying a 'Possible missing firmware' message when installing a kernel with the expected firmware missing. The following are abridged backport test results: Scenario: No DMC (glk_dmc_ver1_04.bin) firmware installed in '/lib/firmware/i915' Test:Kernel installation ('grep -i dmc' output from 'apt install'): 4.12-generic and 4.15-generic: No output # as expected 4.12 to 4.17-rc1-patched: W: Possible missing firmware /lib/firmware/i915/glk_dmc_ver1_04.bin for module i915 Result: The effect of the patch is to add a 'Possible missing firmware' message. Test: Booting ('grep -i dmc' output from 'dmesg'): 4.12-generic: No output # as expected 4.15-generic: i915 0000:00:02.0: Direct firmware load for i915/glk_dmc_ver1_04.bin failed with error -2 i915 0000:00:02.0: Failed to load DMC firmware i915/glk_dmc_ver1_04.bin. Disabling runtime power management. i915 0000:00:02.0: DMC firmware homepage: https://01.org/linuxgraphics/downloads/firmware 4.12-patched: No output # as expected 4.13 to 4.14-patched: i915 0000:00:02.0: Direct firmware load for i915/glk_dmc_ver1_04.bin failed with error -2 i915 0000:00:02.0: Failed to load DMC firmware [https://01.org/linuxgraphics/downloads/firmware], disabling runtime power management. 4.15 to 4.17-rc1-patched: i915 0000:00:02.0: Direct firmware load for i915/glk_dmc_ver1_04.bin failed with error -2 i915 0000:00:02.0: Failed to load DMC firmware i915/glk_dmc_ver1_04.bin. Disabling runtime power management. i915 0000:00:02.0: DMC firmware homepage: https://01.org/linuxgraphics/downloads/firmware Result: The effect of the patch does not change existing (non-patched kernel) messages. Scenario: DMC (glk_dmc_ver1_04.bin) firmware installed in '/lib/firmware/i915' Test:Kernel installation ('grep -i dmc' output from 'apt install') All kernels: No messages # as expected Result: The effect of the patch does not change existing messages. Test" Booting ('grep -i dmc' output from 'dmesg'): 4.12-generic: No output # as expected 4.15-generic: i915 0000:00:02.0: Direct firmware load for i915/glk_dmc_ver1_04.bin failed with error -2 i915 0000:00:02.0: Failed to load DMC firmware i915/glk_dmc_ver1_04.bin. Disabling runtime power management. i915 0000:00:02.0: DMC firmware homepage: https://01.org/linuxgraphics/downloads/firmware 4.12-patched: No output # as expected 4.13 to 4.17-rc1-patched: [drm] Finished loading DMC firmware i915/glk_dmc_ver1_04.bin (v1.4) Result: The effect of the patch is to remove the 'Failed to load' message. Regards, Ian ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-20 5:47 ` Ian W MORRISON @ 2018-04-20 7:50 ` Jani Nikula 0 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-20 7:50 UTC (permalink / raw) To: Ian W MORRISON, Joonas Lahtinen Cc: Srivatsa, Anusha, Vivi, Rodrigo, Wajdeczko, Michal, Greg KH, airlied, linux-kernel, stable, intel-gfx, dri-devel On Fri, 20 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > I've performed backport testing and some additional analysis as follows: What testing did you do beyond booting? Did you run igt? BR, Jani. -- Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-20 7:50 ` Jani Nikula 0 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-20 7:50 UTC (permalink / raw) To: Ian W MORRISON, Joonas Lahtinen Cc: Srivatsa, Anusha, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Vivi, Rodrigo, Wajdeczko, Michal On Fri, 20 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > I've performed backport testing and some additional analysis as follows: What testing did you do beyond booting? Did you run igt? BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-20 7:50 ` Jani Nikula @ 2018-04-20 8:25 ` Ian W MORRISON -1 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-20 8:25 UTC (permalink / raw) To: Jani Nikula Cc: Joonas Lahtinen, Srivatsa, Anusha, Vivi, Rodrigo, Wajdeczko, Michal, Greg KH, airlied, linux-kernel, stable, intel-gfx, dri-devel On 20 April 2018 at 17:50, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Fri, 20 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> I've performed backport testing and some additional analysis as follows: > > What testing did you do beyond booting? Did you run igt? > > BR, > Jani. I did some basic testing including running a 4K video on Chrome whilst running intel_gpu_top. The intel_gpu_top program doesn't display anything on 4.12 either with or without the patch as neither I nor Canonical enabled CONFIG_DRM_I915_ALPHA_SUPPORT. Do you think that the stable tag should exclude 4.12 because of the 'alpha quality support' and if so do you want a V2 of the patch? Best regards, Ian ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-20 8:25 ` Ian W MORRISON 0 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-20 8:25 UTC (permalink / raw) To: Jani Nikula Cc: airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Vivi, Rodrigo On 20 April 2018 at 17:50, Jani Nikula <jani.nikula@linux.intel.com> wrote: > On Fri, 20 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> I've performed backport testing and some additional analysis as follows: > > What testing did you do beyond booting? Did you run igt? > > BR, > Jani. I did some basic testing including running a 4K video on Chrome whilst running intel_gpu_top. The intel_gpu_top program doesn't display anything on 4.12 either with or without the patch as neither I nor Canonical enabled CONFIG_DRM_I915_ALPHA_SUPPORT. Do you think that the stable tag should exclude 4.12 because of the 'alpha quality support' and if so do you want a V2 of the patch? Best regards, Ian _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-17 9:02 ` Jani Nikula @ 2018-04-20 18:03 ` Rodrigo Vivi -1 siblings, 0 replies; 31+ messages in thread From: Rodrigo Vivi @ 2018-04-20 18:03 UTC (permalink / raw) To: Jani Nikula Cc: Srivatsa, Anusha, Ian W MORRISON, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal On Tue, Apr 17, 2018 at 12:02:52PM +0300, Jani Nikula wrote: > On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: > >>-----Original Message----- > >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] > >>Sent: Wednesday, April 11, 2018 5:27 AM > >>To: Ian W MORRISON <ianwmorrison@gmail.com> > >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha > >><anusha.srivatsa@intel.com>; Wajdeczko, Michal > >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; > >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; linux-kernel@vger.kernel.org; > >>stable@vger.kernel.org; intel-gfx@lists.freedesktop.org; dri- > >>devel@lists.freedesktop.org > >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for > >>Geminilake > >> > >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > >>> <snip> > >>> > >>>> > >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that > >>>> older kernels will work with whatever firmware you throw at them. > >>>> > >>> > >>> I included 'Cc: stable' so the patch would get added to the v4.16 and > >>> v4.15 kernels which I have tested with the patch. I found that earlier > >>> kernels didn't support the 'linux-firmware' package required to get > >>> wifi working on Intel's new Gemini Lake NUC. > >> > >>You realize that this patch should have nothing to do with wifi? > >> > >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate the specific > >>versions of stable it is appropriate for. > > > > Hi Jani, > > > > The stable kernel version is 4.12 and beyond. > > It is appropriate to add the CC: stable in my opinion > > Who tested the firmware with v4.12 and later? We only have the CI > results against *current* drm-tip. We don't even know about v4.16. > I understand your concerns, but the problem was that our old process was a bit (lot?) messed and there was the unreliable time until the firmware really lands on linux-firmware.git. So MODULE_FIRMWARE call was only added after firmware was really there on firmware repository but it wasn't about the testing. In other words, the bump version patch was merged after tested, but MODULE_FIRMWARE was left behind because firmware blob took a while to get pulled into linux-firmware.git and we end up forgetting to add it there. In my opinion it should be safe to add the MODULE_FIRMWARE there based on the tests from when the version was bumped. > I'm not going to ack and take responsibility for the stable backports > unless someone actually comes forward with credible Tested-bys. > > BR, > Jani. > > > > > > Anusha > >>BR, > >>Jani. > >> > >>> > >>>> > >>>> PS. How is this a "RESEND"? I haven't seen this before. > >>>> > >>> > >>> It is a 'RESEND' for that very reason. I initially sent the patch to > >>> the same people as a similar patch > >>> (https://patchwork.kernel.org/patch/10143637/) however after realising > >>> this omitted required addresses I added them and resent the patch. > >>> > >>> Best regards, > >>> Ian > >> > >>-- > >>Jani Nikula, Intel Open Source Technology Center > > -- > Jani Nikula, Intel Open Source Technology Center > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-20 18:03 ` Rodrigo Vivi 0 siblings, 0 replies; 31+ messages in thread From: Rodrigo Vivi @ 2018-04-20 18:03 UTC (permalink / raw) To: Jani Nikula Cc: Srivatsa, Anusha, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Ian W MORRISON, Wajdeczko, Michal On Tue, Apr 17, 2018 at 12:02:52PM +0300, Jani Nikula wrote: > On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: > >>-----Original Message----- > >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] > >>Sent: Wednesday, April 11, 2018 5:27 AM > >>To: Ian W MORRISON <ianwmorrison@gmail.com> > >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha > >><anusha.srivatsa@intel.com>; Wajdeczko, Michal > >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; > >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; linux-kernel@vger.kernel.org; > >>stable@vger.kernel.org; intel-gfx@lists.freedesktop.org; dri- > >>devel@lists.freedesktop.org > >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for > >>Geminilake > >> > >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > >>> <snip> > >>> > >>>> > >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that > >>>> older kernels will work with whatever firmware you throw at them. > >>>> > >>> > >>> I included 'Cc: stable' so the patch would get added to the v4.16 and > >>> v4.15 kernels which I have tested with the patch. I found that earlier > >>> kernels didn't support the 'linux-firmware' package required to get > >>> wifi working on Intel's new Gemini Lake NUC. > >> > >>You realize that this patch should have nothing to do with wifi? > >> > >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please indicate the specific > >>versions of stable it is appropriate for. > > > > Hi Jani, > > > > The stable kernel version is 4.12 and beyond. > > It is appropriate to add the CC: stable in my opinion > > Who tested the firmware with v4.12 and later? We only have the CI > results against *current* drm-tip. We don't even know about v4.16. > I understand your concerns, but the problem was that our old process was a bit (lot?) messed and there was the unreliable time until the firmware really lands on linux-firmware.git. So MODULE_FIRMWARE call was only added after firmware was really there on firmware repository but it wasn't about the testing. In other words, the bump version patch was merged after tested, but MODULE_FIRMWARE was left behind because firmware blob took a while to get pulled into linux-firmware.git and we end up forgetting to add it there. In my opinion it should be safe to add the MODULE_FIRMWARE there based on the tests from when the version was bumped. > I'm not going to ack and take responsibility for the stable backports > unless someone actually comes forward with credible Tested-bys. > > BR, > Jani. > > > > > > Anusha > >>BR, > >>Jani. > >> > >>> > >>>> > >>>> PS. How is this a "RESEND"? I haven't seen this before. > >>>> > >>> > >>> It is a 'RESEND' for that very reason. I initially sent the patch to > >>> the same people as a similar patch > >>> (https://patchwork.kernel.org/patch/10143637/) however after realising > >>> this omitted required addresses I added them and resent the patch. > >>> > >>> Best regards, > >>> Ian > >> > >>-- > >>Jani Nikula, Intel Open Source Technology Center > > -- > Jani Nikula, Intel Open Source Technology Center > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-20 18:03 ` Rodrigo Vivi @ 2018-04-20 18:30 ` Srivatsa, Anusha -1 siblings, 0 replies; 31+ messages in thread From: Srivatsa, Anusha @ 2018-04-20 18:30 UTC (permalink / raw) To: Vivi, Rodrigo, Jani Nikula, Botello Ortega, Luis, Martinez Monroy, Elio Cc: Ian W MORRISON, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal >-----Original Message----- >From: Vivi, Rodrigo >Sent: Friday, April 20, 2018 11:04 AM >To: Jani Nikula <jani.nikula@linux.intel.com> >Cc: Srivatsa, Anusha <anusha.srivatsa@intel.com>; Ian W MORRISON ><ianwmorrison@gmail.com>; airlied@linux.ie; Greg KH ><gregkh@linuxfoundation.org>; intel-gfx@lists.freedesktop.org; linux- >kernel@vger.kernel.org; stable@vger.kernel.org; dri- >devel@lists.freedesktop.org; Wajdeczko, Michal <Michal.Wajdeczko@intel.com> >Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >Geminilake > >On Tue, Apr 17, 2018 at 12:02:52PM +0300, Jani Nikula wrote: >> On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: >> >>-----Original Message----- >> >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >> >>Sent: Wednesday, April 11, 2018 5:27 AM >> >>To: Ian W MORRISON <ianwmorrison@gmail.com> >> >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha >> >><anusha.srivatsa@intel.com>; Wajdeczko, Michal >> >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >> >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; >> >>linux-kernel@vger.kernel.org; stable@vger.kernel.org; >> >>intel-gfx@lists.freedesktop.org; dri- devel@lists.freedesktop.org >> >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE >> >>for Geminilake >> >> >> >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> >>> <snip> >> >>> >> >>>> >> >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that >> >>>> older kernels will work with whatever firmware you throw at them. >> >>>> >> >>> >> >>> I included 'Cc: stable' so the patch would get added to the v4.16 >> >>> and >> >>> v4.15 kernels which I have tested with the patch. I found that >> >>> earlier kernels didn't support the 'linux-firmware' package >> >>> required to get wifi working on Intel's new Gemini Lake NUC. >> >> >> >>You realize that this patch should have nothing to do with wifi? >> >> >> >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please >> >>indicate the specific versions of stable it is appropriate for. >> > >> > Hi Jani, >> > >> > The stable kernel version is 4.12 and beyond. >> > It is appropriate to add the CC: stable in my opinion >> >> Who tested the firmware with v4.12 and later? We only have the CI >> results against *current* drm-tip. We don't even know about v4.16. >> > >I understand your concerns, but the problem was that our old process was a bit >(lot?) messed and there was the unreliable time until the firmware really lands on >linux-firmware.git. So MODULE_FIRMWARE call was only added after firmware >was really there on firmware repository but it wasn't about the testing. > >In other words, the bump version patch was merged after tested, but >MODULE_FIRMWARE was left behind because firmware blob took a while to get >pulled into linux-firmware.git and we end up forgetting to add it there. > >In my opinion it should be safe to add the MODULE_FIRMWARE there based on >the tests from when the version was bumped. Luis, Elio, can you guys confirm that this firmware is tested and healthy? And also, give a tested-by to this patch please? Thanks, Anusha >> I'm not going to ack and take responsibility for the stable backports >> unless someone actually comes forward with credible Tested-bys. >> >> BR, >> Jani. >> >> >> > >> > Anusha >> >>BR, >> >>Jani. >> >> >> >>> >> >>>> >> >>>> PS. How is this a "RESEND"? I haven't seen this before. >> >>>> >> >>> >> >>> It is a 'RESEND' for that very reason. I initially sent the patch >> >>> to the same people as a similar patch >> >>> (https://patchwork.kernel.org/patch/10143637/) however after >> >>> realising this omitted required addresses I added them and resent the >patch. >> >>> >> >>> Best regards, >> >>> Ian >> >> >> >>-- >> >>Jani Nikula, Intel Open Source Technology Center >> >> -- >> Jani Nikula, Intel Open Source Technology Center >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-20 18:30 ` Srivatsa, Anusha 0 siblings, 0 replies; 31+ messages in thread From: Srivatsa, Anusha @ 2018-04-20 18:30 UTC (permalink / raw) To: Vivi, Rodrigo, Jani Nikula, Botello Ortega, Luis, Martinez Monroy, Elio Cc: airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Ian W MORRISON, Wajdeczko, Michal >-----Original Message----- >From: Vivi, Rodrigo >Sent: Friday, April 20, 2018 11:04 AM >To: Jani Nikula <jani.nikula@linux.intel.com> >Cc: Srivatsa, Anusha <anusha.srivatsa@intel.com>; Ian W MORRISON ><ianwmorrison@gmail.com>; airlied@linux.ie; Greg KH ><gregkh@linuxfoundation.org>; intel-gfx@lists.freedesktop.org; linux- >kernel@vger.kernel.org; stable@vger.kernel.org; dri- >devel@lists.freedesktop.org; Wajdeczko, Michal <Michal.Wajdeczko@intel.com> >Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >Geminilake > >On Tue, Apr 17, 2018 at 12:02:52PM +0300, Jani Nikula wrote: >> On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: >> >>-----Original Message----- >> >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >> >>Sent: Wednesday, April 11, 2018 5:27 AM >> >>To: Ian W MORRISON <ianwmorrison@gmail.com> >> >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha >> >><anusha.srivatsa@intel.com>; Wajdeczko, Michal >> >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >> >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; >> >>linux-kernel@vger.kernel.org; stable@vger.kernel.org; >> >>intel-gfx@lists.freedesktop.org; dri- devel@lists.freedesktop.org >> >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE >> >>for Geminilake >> >> >> >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> >>> <snip> >> >>> >> >>>> >> >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that >> >>>> older kernels will work with whatever firmware you throw at them. >> >>>> >> >>> >> >>> I included 'Cc: stable' so the patch would get added to the v4.16 >> >>> and >> >>> v4.15 kernels which I have tested with the patch. I found that >> >>> earlier kernels didn't support the 'linux-firmware' package >> >>> required to get wifi working on Intel's new Gemini Lake NUC. >> >> >> >>You realize that this patch should have nothing to do with wifi? >> >> >> >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please >> >>indicate the specific versions of stable it is appropriate for. >> > >> > Hi Jani, >> > >> > The stable kernel version is 4.12 and beyond. >> > It is appropriate to add the CC: stable in my opinion >> >> Who tested the firmware with v4.12 and later? We only have the CI >> results against *current* drm-tip. We don't even know about v4.16. >> > >I understand your concerns, but the problem was that our old process was a bit >(lot?) messed and there was the unreliable time until the firmware really lands on >linux-firmware.git. So MODULE_FIRMWARE call was only added after firmware >was really there on firmware repository but it wasn't about the testing. > >In other words, the bump version patch was merged after tested, but >MODULE_FIRMWARE was left behind because firmware blob took a while to get >pulled into linux-firmware.git and we end up forgetting to add it there. > >In my opinion it should be safe to add the MODULE_FIRMWARE there based on >the tests from when the version was bumped. Luis, Elio, can you guys confirm that this firmware is tested and healthy? And also, give a tested-by to this patch please? Thanks, Anusha >> I'm not going to ack and take responsibility for the stable backports >> unless someone actually comes forward with credible Tested-bys. >> >> BR, >> Jani. >> >> >> > >> > Anusha >> >>BR, >> >>Jani. >> >> >> >>> >> >>>> >> >>>> PS. How is this a "RESEND"? I haven't seen this before. >> >>>> >> >>> >> >>> It is a 'RESEND' for that very reason. I initially sent the patch >> >>> to the same people as a similar patch >> >>> (https://patchwork.kernel.org/patch/10143637/) however after >> >>> realising this omitted required addresses I added them and resent the >patch. >> >>> >> >>> Best regards, >> >>> Ian >> >> >> >>-- >> >>Jani Nikula, Intel Open Source Technology Center >> >> -- >> Jani Nikula, Intel Open Source Technology Center >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-20 18:30 ` Srivatsa, Anusha @ 2018-04-21 1:22 ` Botello Ortega, Luis -1 siblings, 0 replies; 31+ messages in thread From: Botello Ortega, Luis @ 2018-04-21 1:22 UTC (permalink / raw) To: Srivatsa, Anusha, Vivi, Rodrigo, Jani Nikula, Martinez Monroy, Elio, Vega, Ricardo Cc: Ian W MORRISON, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal Hi all: We tested GLK DMC 1.04 FW in last week of September 2017, using the latest drm-tip version for that time (4.14.0-rc2) and according to our results we could declare this FW as acceptable and healthy to be used with kernel version 4.14 . However, we cannot guarantee quality and healthy of this FW if it is used in top of current drm-tip kernel (4.17-rc0). Best Regards Luis Botello -----Original Message----- From: Srivatsa, Anusha Sent: Friday, April 20, 2018 1:30 PM To: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Jani Nikula <jani.nikula@linux.intel.com>; Botello Ortega, Luis <luis.botello.ortega@intel.com>; Martinez Monroy, Elio <elio.martinez.monroy@intel.com> Cc: Ian W MORRISON <ianwmorrison@gmail.com>; airlied@linux.ie; Greg KH <gregkh@linuxfoundation.org>; intel-gfx@lists.freedesktop.org; linux-kernel@vger.kernel.org; stable@vger.kernel.org; dri-devel@lists.freedesktop.org; Wajdeczko, Michal <Michal.Wajdeczko@intel.com> Subject: RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake >-----Original Message----- >From: Vivi, Rodrigo >Sent: Friday, April 20, 2018 11:04 AM >To: Jani Nikula <jani.nikula@linux.intel.com> >Cc: Srivatsa, Anusha <anusha.srivatsa@intel.com>; Ian W MORRISON ><ianwmorrison@gmail.com>; airlied@linux.ie; Greg KH ><gregkh@linuxfoundation.org>; intel-gfx@lists.freedesktop.org; linux- >kernel@vger.kernel.org; stable@vger.kernel.org; dri- >devel@lists.freedesktop.org; Wajdeczko, Michal ><Michal.Wajdeczko@intel.com> >Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >Geminilake > >On Tue, Apr 17, 2018 at 12:02:52PM +0300, Jani Nikula wrote: >> On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: >> >>-----Original Message----- >> >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >> >>Sent: Wednesday, April 11, 2018 5:27 AM >> >>To: Ian W MORRISON <ianwmorrison@gmail.com> >> >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha >> >><anusha.srivatsa@intel.com>; Wajdeczko, Michal >> >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >> >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; >> >>linux-kernel@vger.kernel.org; stable@vger.kernel.org; >> >>intel-gfx@lists.freedesktop.org; dri- devel@lists.freedesktop.org >> >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE >> >>for Geminilake >> >> >> >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> >>> <snip> >> >>> >> >>>> >> >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that >> >>>> older kernels will work with whatever firmware you throw at them. >> >>>> >> >>> >> >>> I included 'Cc: stable' so the patch would get added to the v4.16 >> >>> and >> >>> v4.15 kernels which I have tested with the patch. I found that >> >>> earlier kernels didn't support the 'linux-firmware' package >> >>> required to get wifi working on Intel's new Gemini Lake NUC. >> >> >> >>You realize that this patch should have nothing to do with wifi? >> >> >> >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please >> >>indicate the specific versions of stable it is appropriate for. >> > >> > Hi Jani, >> > >> > The stable kernel version is 4.12 and beyond. >> > It is appropriate to add the CC: stable in my opinion >> >> Who tested the firmware with v4.12 and later? We only have the CI >> results against *current* drm-tip. We don't even know about v4.16. >> > >I understand your concerns, but the problem was that our old process >was a bit >(lot?) messed and there was the unreliable time until the firmware >really lands on linux-firmware.git. So MODULE_FIRMWARE call was only >added after firmware was really there on firmware repository but it wasn't about the testing. > >In other words, the bump version patch was merged after tested, but >MODULE_FIRMWARE was left behind because firmware blob took a while to >get pulled into linux-firmware.git and we end up forgetting to add it there. > >In my opinion it should be safe to add the MODULE_FIRMWARE there based >on the tests from when the version was bumped. Luis, Elio, can you guys confirm that this firmware is tested and healthy? And also, give a tested-by to this patch please? Thanks, Anusha >> I'm not going to ack and take responsibility for the stable backports >> unless someone actually comes forward with credible Tested-bys. >> >> BR, >> Jani. >> >> >> > >> > Anusha >> >>BR, >> >>Jani. >> >> >> >>> >> >>>> >> >>>> PS. How is this a "RESEND"? I haven't seen this before. >> >>>> >> >>> >> >>> It is a 'RESEND' for that very reason. I initially sent the patch >> >>> to the same people as a similar patch >> >>> (https://patchwork.kernel.org/patch/10143637/) however after >> >>> realising this omitted required addresses I added them and resent >> >>> the >patch. >> >>> >> >>> Best regards, >> >>> Ian >> >> >> >>-- >> >>Jani Nikula, Intel Open Source Technology Center >> >> -- >> Jani Nikula, Intel Open Source Technology Center >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-21 1:22 ` Botello Ortega, Luis 0 siblings, 0 replies; 31+ messages in thread From: Botello Ortega, Luis @ 2018-04-21 1:22 UTC (permalink / raw) To: Srivatsa, Anusha, Vivi, Rodrigo, Jani Nikula, Martinez Monroy, Elio, Vega, Ricardo Cc: airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Ian W MORRISON Hi all: We tested GLK DMC 1.04 FW in last week of September 2017, using the latest drm-tip version for that time (4.14.0-rc2) and according to our results we could declare this FW as acceptable and healthy to be used with kernel version 4.14 . However, we cannot guarantee quality and healthy of this FW if it is used in top of current drm-tip kernel (4.17-rc0). Best Regards Luis Botello -----Original Message----- From: Srivatsa, Anusha Sent: Friday, April 20, 2018 1:30 PM To: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Jani Nikula <jani.nikula@linux.intel.com>; Botello Ortega, Luis <luis.botello.ortega@intel.com>; Martinez Monroy, Elio <elio.martinez.monroy@intel.com> Cc: Ian W MORRISON <ianwmorrison@gmail.com>; airlied@linux.ie; Greg KH <gregkh@linuxfoundation.org>; intel-gfx@lists.freedesktop.org; linux-kernel@vger.kernel.org; stable@vger.kernel.org; dri-devel@lists.freedesktop.org; Wajdeczko, Michal <Michal.Wajdeczko@intel.com> Subject: RE: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake >-----Original Message----- >From: Vivi, Rodrigo >Sent: Friday, April 20, 2018 11:04 AM >To: Jani Nikula <jani.nikula@linux.intel.com> >Cc: Srivatsa, Anusha <anusha.srivatsa@intel.com>; Ian W MORRISON ><ianwmorrison@gmail.com>; airlied@linux.ie; Greg KH ><gregkh@linuxfoundation.org>; intel-gfx@lists.freedesktop.org; linux- >kernel@vger.kernel.org; stable@vger.kernel.org; dri- >devel@lists.freedesktop.org; Wajdeczko, Michal ><Michal.Wajdeczko@intel.com> >Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for >Geminilake > >On Tue, Apr 17, 2018 at 12:02:52PM +0300, Jani Nikula wrote: >> On Mon, 16 Apr 2018, "Srivatsa, Anusha" <anusha.srivatsa@intel.com> wrote: >> >>-----Original Message----- >> >>From: Jani Nikula [mailto:jani.nikula@linux.intel.com] >> >>Sent: Wednesday, April 11, 2018 5:27 AM >> >>To: Ian W MORRISON <ianwmorrison@gmail.com> >> >>Cc: Vivi, Rodrigo <rodrigo.vivi@intel.com>; Srivatsa, Anusha >> >><anusha.srivatsa@intel.com>; Wajdeczko, Michal >> >><Michal.Wajdeczko@intel.com>; Greg KH <gregkh@linuxfoundation.org>; >> >>airlied@linux.ie; joonas.lahtinen@linux.intel.com; >> >>linux-kernel@vger.kernel.org; stable@vger.kernel.org; >> >>intel-gfx@lists.freedesktop.org; dri- devel@lists.freedesktop.org >> >>Subject: Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE >> >>for Geminilake >> >> >> >>On Wed, 11 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: >> >>> <snip> >> >>> >> >>>> >> >>>> NAK on indiscriminate Cc: stable. There are zero guarantees that >> >>>> older kernels will work with whatever firmware you throw at them. >> >>>> >> >>> >> >>> I included 'Cc: stable' so the patch would get added to the v4.16 >> >>> and >> >>> v4.15 kernels which I have tested with the patch. I found that >> >>> earlier kernels didn't support the 'linux-firmware' package >> >>> required to get wifi working on Intel's new Gemini Lake NUC. >> >> >> >>You realize that this patch should have nothing to do with wifi? >> >> >> >>Rodrigo, Anusha, if you think Cc: stable is appropriate, please >> >>indicate the specific versions of stable it is appropriate for. >> > >> > Hi Jani, >> > >> > The stable kernel version is 4.12 and beyond. >> > It is appropriate to add the CC: stable in my opinion >> >> Who tested the firmware with v4.12 and later? We only have the CI >> results against *current* drm-tip. We don't even know about v4.16. >> > >I understand your concerns, but the problem was that our old process >was a bit >(lot?) messed and there was the unreliable time until the firmware >really lands on linux-firmware.git. So MODULE_FIRMWARE call was only >added after firmware was really there on firmware repository but it wasn't about the testing. > >In other words, the bump version patch was merged after tested, but >MODULE_FIRMWARE was left behind because firmware blob took a while to >get pulled into linux-firmware.git and we end up forgetting to add it there. > >In my opinion it should be safe to add the MODULE_FIRMWARE there based >on the tests from when the version was bumped. Luis, Elio, can you guys confirm that this firmware is tested and healthy? And also, give a tested-by to this patch please? Thanks, Anusha >> I'm not going to ack and take responsibility for the stable backports >> unless someone actually comes forward with credible Tested-bys. >> >> BR, >> Jani. >> >> >> > >> > Anusha >> >>BR, >> >>Jani. >> >> >> >>> >> >>>> >> >>>> PS. How is this a "RESEND"? I haven't seen this before. >> >>>> >> >>> >> >>> It is a 'RESEND' for that very reason. I initially sent the patch >> >>> to the same people as a similar patch >> >>> (https://patchwork.kernel.org/patch/10143637/) however after >> >>> realising this omitted required addresses I added them and resent >> >>> the >patch. >> >>> >> >>> Best regards, >> >>> Ian >> >> >> >>-- >> >>Jani Nikula, Intel Open Source Technology Center >> >> -- >> Jani Nikula, Intel Open Source Technology Center >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-21 1:22 ` Botello Ortega, Luis @ 2018-04-21 22:46 ` Ian W MORRISON -1 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-21 22:46 UTC (permalink / raw) To: Botello Ortega, Luis Cc: Srivatsa, Anusha, Vivi, Rodrigo, Jani Nikula, Martinez Monroy, Elio, Vega, Ricardo, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal On 21 April 2018 at 11:22, Botello Ortega, Luis <luis.botello.ortega@intel.com> wrote: > Hi all: > > We tested GLK DMC 1.04 FW in last week of September 2017, using the latest drm-tip version for that time (4.14.0-rc2) and according to our results we could declare this FW as acceptable and healthy to be used with kernel version 4.14 . > However, we cannot guarantee quality and healthy of this FW if it is used in top of current drm-tip kernel (4.17-rc0). > > Best Regards > Luis Botello > > Your measured response is appreciated and raises at least four possible alternatives for this patch to proceed: 1. It is normal that firmware is *only* tested at a certain point in time with the drm-tip version of the mainline kernel so that the statement 'firmware X is acceptable and healthy to be used with kernel version Y but is not guaranteed with Y+1 or Y+n' always holds true for any microarchitecture codename's DMC firmware. Therefore it is appropriate for this patch to have a restricted 'Cc: stable@vger.kernel.org # 4.14' tag together with an explicit 'Tested-by: (in 4.14)' tag. 2. As firmware testing was not performed on 4.15 and subsequent versions, testing is still required on versions 4.15 through to the current RC of version 4.17 for the patch to receive a 'Tested-by:' tag together with a 'Cc: stable@vger.kernel.org # 4.14' tag. 3. Further firmware testing will only be undertaken for the current RC of version 4.17 and on successful completion the patch will only receive a 'Tested-by:' tag. A 'Cc: stable@vger.kernel.org' tag is not applicable. 4. The firmware was not tested on 4.15 and subsequent versions as it's functionality was provided by alternative means and therefore the patch is only required in version 4.14 and should have a 'Cc: stable@vger.kernel.org # 4.14 only' tag together with a restricted 'Tested-by: (only in 4.14)' tag. Could you indicate which alternative is appropriate? If further firmware testing is required (as in points 2 or 3) then can an expected completion date be provided? Best regards, Ian ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-21 22:46 ` Ian W MORRISON 0 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-21 22:46 UTC (permalink / raw) To: Botello Ortega, Luis Cc: dri-devel, Vega, Ricardo, airlied, Greg KH, intel-gfx, linux-kernel, stable, Vivi, Rodrigo On 21 April 2018 at 11:22, Botello Ortega, Luis <luis.botello.ortega@intel.com> wrote: > Hi all: > > We tested GLK DMC 1.04 FW in last week of September 2017, using the latest drm-tip version for that time (4.14.0-rc2) and according to our results we could declare this FW as acceptable and healthy to be used with kernel version 4.14 . > However, we cannot guarantee quality and healthy of this FW if it is used in top of current drm-tip kernel (4.17-rc0). > > Best Regards > Luis Botello > > Your measured response is appreciated and raises at least four possible alternatives for this patch to proceed: 1. It is normal that firmware is *only* tested at a certain point in time with the drm-tip version of the mainline kernel so that the statement 'firmware X is acceptable and healthy to be used with kernel version Y but is not guaranteed with Y+1 or Y+n' always holds true for any microarchitecture codename's DMC firmware. Therefore it is appropriate for this patch to have a restricted 'Cc: stable@vger.kernel.org # 4.14' tag together with an explicit 'Tested-by: (in 4.14)' tag. 2. As firmware testing was not performed on 4.15 and subsequent versions, testing is still required on versions 4.15 through to the current RC of version 4.17 for the patch to receive a 'Tested-by:' tag together with a 'Cc: stable@vger.kernel.org # 4.14' tag. 3. Further firmware testing will only be undertaken for the current RC of version 4.17 and on successful completion the patch will only receive a 'Tested-by:' tag. A 'Cc: stable@vger.kernel.org' tag is not applicable. 4. The firmware was not tested on 4.15 and subsequent versions as it's functionality was provided by alternative means and therefore the patch is only required in version 4.14 and should have a 'Cc: stable@vger.kernel.org # 4.14 only' tag together with a restricted 'Tested-by: (only in 4.14)' tag. Could you indicate which alternative is appropriate? If further firmware testing is required (as in points 2 or 3) then can an expected completion date be provided? Best regards, Ian _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-21 22:46 ` Ian W MORRISON @ 2018-04-25 23:15 ` Ian W MORRISON -1 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-25 23:15 UTC (permalink / raw) To: Botello Ortega, Luis Cc: Srivatsa, Anusha, Vivi, Rodrigo, Jani Nikula, Martinez Monroy, Elio, Vega, Ricardo, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal Hi Anusha, Can I ask if this is on anyone's radar as I'm concerned this patch will stall otherwise? I see that the significance of testing with the 4.14 kernel enabled the firmware to be included in the latest Chrome OS kernel ( https://groups.google.com/a/chromium.org/forum/#!topic/chromium-os-reviews/jtpHvZOfZ-Q). It is important to similarly include in the mainline and stable kernels to facilitate various distros that are now raising bug reports (for example: https://bugs.launchpad.net/intel/+bug/1760545). Many thanks, Ian On Sun, 22 Apr 2018 at 08:46, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > On 21 April 2018 at 11:22, Botello Ortega, Luis > <luis.botello.ortega@intel.com> wrote: > > Hi all: > > > > We tested GLK DMC 1.04 FW in last week of September 2017, using the latest drm-tip version for that time (4.14.0-rc2) and according to our results we could declare this FW as acceptable and healthy to be used with kernel version 4.14 . > > However, we cannot guarantee quality and healthy of this FW if it is used in top of current drm-tip kernel (4.17-rc0). > > > > Best Regards > > Luis Botello > > > > > Your measured response is appreciated and raises at least four > possible alternatives for this patch to proceed: > 1. It is normal that firmware is *only* tested at a certain point in > time with the drm-tip version of the mainline kernel so that the > statement 'firmware X is acceptable and healthy to be used with kernel > version Y but is not guaranteed with Y+1 or Y+n' always holds true for > any microarchitecture codename's DMC firmware. Therefore it is > appropriate for this patch to have a restricted 'Cc: > stable@vger.kernel.org # 4.14' tag together with an explicit > 'Tested-by: (in 4.14)' tag. > 2. As firmware testing was not performed on 4.15 and subsequent > versions, testing is still required on versions 4.15 through to the > current RC of version 4.17 for the patch to receive a 'Tested-by:' tag > together with a 'Cc: stable@vger.kernel.org # 4.14' tag. > 3. Further firmware testing will only be undertaken for the current RC > of version 4.17 and on successful completion the patch will only > receive a 'Tested-by:' tag. A 'Cc: stable@vger.kernel.org' tag is not > applicable. > 4. The firmware was not tested on 4.15 and subsequent versions as it's > functionality was provided by alternative means and therefore the > patch is only required in version 4.14 and should have a 'Cc: > stable@vger.kernel.org # 4.14 only' tag together with a restricted > 'Tested-by: (only in 4.14)' tag. > Could you indicate which alternative is appropriate? If further > firmware testing is required (as in points 2 or 3) then can an > expected completion date be provided? > Best regards, > Ian ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-25 23:15 ` Ian W MORRISON 0 siblings, 0 replies; 31+ messages in thread From: Ian W MORRISON @ 2018-04-25 23:15 UTC (permalink / raw) To: Botello Ortega, Luis Cc: dri-devel, Vega, Ricardo, airlied, Greg KH, intel-gfx, linux-kernel, stable, Vivi, Rodrigo Hi Anusha, Can I ask if this is on anyone's radar as I'm concerned this patch will stall otherwise? I see that the significance of testing with the 4.14 kernel enabled the firmware to be included in the latest Chrome OS kernel ( https://groups.google.com/a/chromium.org/forum/#!topic/chromium-os-reviews/jtpHvZOfZ-Q). It is important to similarly include in the mainline and stable kernels to facilitate various distros that are now raising bug reports (for example: https://bugs.launchpad.net/intel/+bug/1760545). Many thanks, Ian On Sun, 22 Apr 2018 at 08:46, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > On 21 April 2018 at 11:22, Botello Ortega, Luis > <luis.botello.ortega@intel.com> wrote: > > Hi all: > > > > We tested GLK DMC 1.04 FW in last week of September 2017, using the latest drm-tip version for that time (4.14.0-rc2) and according to our results we could declare this FW as acceptable and healthy to be used with kernel version 4.14 . > > However, we cannot guarantee quality and healthy of this FW if it is used in top of current drm-tip kernel (4.17-rc0). > > > > Best Regards > > Luis Botello > > > > > Your measured response is appreciated and raises at least four > possible alternatives for this patch to proceed: > 1. It is normal that firmware is *only* tested at a certain point in > time with the drm-tip version of the mainline kernel so that the > statement 'firmware X is acceptable and healthy to be used with kernel > version Y but is not guaranteed with Y+1 or Y+n' always holds true for > any microarchitecture codename's DMC firmware. Therefore it is > appropriate for this patch to have a restricted 'Cc: > stable@vger.kernel.org # 4.14' tag together with an explicit > 'Tested-by: (in 4.14)' tag. > 2. As firmware testing was not performed on 4.15 and subsequent > versions, testing is still required on versions 4.15 through to the > current RC of version 4.17 for the patch to receive a 'Tested-by:' tag > together with a 'Cc: stable@vger.kernel.org # 4.14' tag. > 3. Further firmware testing will only be undertaken for the current RC > of version 4.17 and on successful completion the patch will only > receive a 'Tested-by:' tag. A 'Cc: stable@vger.kernel.org' tag is not > applicable. > 4. The firmware was not tested on 4.15 and subsequent versions as it's > functionality was provided by alternative means and therefore the > patch is only required in version 4.14 and should have a 'Cc: > stable@vger.kernel.org # 4.14 only' tag together with a restricted > 'Tested-by: (only in 4.14)' tag. > Could you indicate which alternative is appropriate? If further > firmware testing is required (as in points 2 or 3) then can an > expected completion date be provided? > Best regards, > Ian _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-25 23:15 ` Ian W MORRISON @ 2018-04-27 9:20 ` Jani Nikula -1 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-27 9:20 UTC (permalink / raw) To: Ian W MORRISON, Botello Ortega, Luis, Joonas Lahtinen Cc: Srivatsa, Anusha, Vivi, Rodrigo, Martinez Monroy, Elio, Vega, Ricardo, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal On Wed, 25 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > Can I ask if this is on anyone's radar as I'm concerned this patch will > stall otherwise? Pushed to drm-intel-next-queued, thanks for the patch. I opted to drop the Cc: stable for now. This doesn't mean it can't be backported later on, I'm just punting on that call right now to make some forward progress here. Joonas, please do pick f6d3e06f0747 ("drm/i915/glk: Add MODULE_FIRMWARE for Geminilake") to drm-intel-fixes to queue it to v4.17. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake @ 2018-04-27 9:20 ` Jani Nikula 0 siblings, 0 replies; 31+ messages in thread From: Jani Nikula @ 2018-04-27 9:20 UTC (permalink / raw) To: Ian W MORRISON, Botello Ortega, Luis, Joonas Lahtinen Cc: Srivatsa, Anusha, Vivi, Rodrigo, Martinez Monroy, Elio, Vega, Ricardo, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal On Wed, 25 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > Can I ask if this is on anyone's radar as I'm concerned this patch will > stall otherwise? Pushed to drm-intel-next-queued, thanks for the patch. I opted to drop the Cc: stable for now. This doesn't mean it can't be backported later on, I'm just punting on that call right now to make some forward progress here. Joonas, please do pick f6d3e06f0747 ("drm/i915/glk: Add MODULE_FIRMWARE for Geminilake") to drm-intel-fixes to queue it to v4.17. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* Re: [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-27 9:20 ` Jani Nikula (?) @ 2018-04-30 7:58 ` Joonas Lahtinen -1 siblings, 0 replies; 31+ messages in thread From: Joonas Lahtinen @ 2018-04-30 7:58 UTC (permalink / raw) To: Botello Ortega, Luis, Ian W MORRISON, Jani Nikula Cc: Srivatsa, Anusha, Vivi, Rodrigo, Martinez Monroy, Elio, Vega, Ricardo, airlied, Greg KH, intel-gfx, linux-kernel, stable, dri-devel, Wajdeczko, Michal Quoting Jani Nikula (2018-04-27 12:20:55) > On Wed, 25 Apr 2018, Ian W MORRISON <ianwmorrison@gmail.com> wrote: > > Can I ask if this is on anyone's radar as I'm concerned this patch will > > stall otherwise? > > Pushed to drm-intel-next-queued, thanks for the patch. > > I opted to drop the Cc: stable for now. This doesn't mean it can't be > backported later on, I'm just punting on that call right now to make > some forward progress here. > > Joonas, please do pick f6d3e06f0747 ("drm/i915/glk: Add MODULE_FIRMWARE > for Geminilake") to drm-intel-fixes to queue it to v4.17. Done. That's the only drm-intel-fixes patch there seems to be for -rc4. Regards, Joonas > > BR, > Jani. > > -- > Jani Nikula, Intel Open Source Technology Center ^ permalink raw reply [flat|nested] 31+ messages in thread
* ✓ Fi.CI.BAT: success for series starting with [RESEND,1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-11 4:42 [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake ianwmorrison 2018-04-11 9:24 ` Jani Nikula @ 2018-04-11 10:54 ` Patchwork 2018-04-11 12:37 ` ✓ Fi.CI.IGT: " Patchwork 2 siblings, 0 replies; 31+ messages in thread From: Patchwork @ 2018-04-11 10:54 UTC (permalink / raw) To: Ian W MORRISON; +Cc: intel-gfx == Series Details == Series: series starting with [RESEND,1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake URL : https://patchwork.freedesktop.org/series/41533/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4043 -> Patchwork_8664 = == Summary - SUCCESS == No regressions found. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8664/ == Known issues == Here are the changes found in Patchwork_8664 that come from known issues: === IGT changes === ==== Possible fixes ==== igt@debugfs_test@read_all_entries: fi-snb-2520m: INCOMPLETE (fdo#103713) -> PASS igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c: fi-bxt-dsi: INCOMPLETE (fdo#103927) -> PASS fdo#103713 https://bugs.freedesktop.org/show_bug.cgi?id=103713 fdo#103927 https://bugs.freedesktop.org/show_bug.cgi?id=103927 == Participating hosts (35 -> 32) == Missing (3): fi-ctg-p8600 fi-ilk-m540 fi-skl-6700hq == Build changes == * Linux: CI_DRM_4043 -> Patchwork_8664 CI_DRM_4043: 5fca09304f690a61408129c7ed166d7f1ddfa66c @ git://anongit.freedesktop.org/gfx-ci/linux IGT_4421: 27cd3793a3a183e4ec70c392f293e840642f2799 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_8664: de9215d6a97175391aa3ef8a7f6b10bd5dda9a17 @ git://anongit.freedesktop.org/gfx-ci/linux piglit_4421: 45e115f293fd6acc0c9647cf2d3b76be78819ba5 @ git://anongit.freedesktop.org/piglit == Patchwork commits == de9215d6a971 drm/i915/glk: Add MODULE_FIRMWARE for Geminilake == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8664/issues.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 31+ messages in thread
* ✓ Fi.CI.IGT: success for series starting with [RESEND,1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake 2018-04-11 4:42 [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake ianwmorrison 2018-04-11 9:24 ` Jani Nikula 2018-04-11 10:54 ` ✓ Fi.CI.BAT: success for series starting with [RESEND,1/1] " Patchwork @ 2018-04-11 12:37 ` Patchwork 2 siblings, 0 replies; 31+ messages in thread From: Patchwork @ 2018-04-11 12:37 UTC (permalink / raw) To: Ian W MORRISON; +Cc: intel-gfx == Series Details == Series: series starting with [RESEND,1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake URL : https://patchwork.freedesktop.org/series/41533/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4043_full -> Patchwork_8664_full = == Summary - WARNING == Minor unknown changes coming with Patchwork_8664_full need to be verified manually. If you think the reported changes have nothing to do with the changes introduced in Patchwork_8664_full, please notify your bug team to allow them to document this new failure mode, which will reduce false positives in CI. External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8664/ == Possible new issues == Here are the unknown changes that may have been introduced in Patchwork_8664_full: === IGT changes === ==== Warnings ==== igt@gem_mocs_settings@mocs-rc6-blt: shard-kbl: PASS -> SKIP igt@kms_vblank@pipe-a-ts-continuation-suspend: shard-snb: PASS -> SKIP igt@pm_rc6_residency@rc6-accuracy: shard-kbl: SKIP -> PASS == Known issues == Here are the changes found in Patchwork_8664_full that come from known issues: === IGT changes === ==== Issues hit ==== igt@perf_pmu@busy-accuracy-50-bcs0: shard-kbl: PASS -> FAIL (fdo#105157) ==== Possible fixes ==== igt@kms_atomic_transition@plane-all-modeset-transition: shard-hsw: DMESG-WARN (fdo#102614) -> PASS igt@kms_flip@2x-plain-flip-fb-recreate-interruptible: shard-hsw: FAIL (fdo#100368) -> PASS +1 igt@kms_flip@flip-vs-blocking-wf-vblank: shard-hsw: FAIL (fdo#103928) -> PASS fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368 fdo#102614 https://bugs.freedesktop.org/show_bug.cgi?id=102614 fdo#103928 https://bugs.freedesktop.org/show_bug.cgi?id=103928 fdo#105157 https://bugs.freedesktop.org/show_bug.cgi?id=105157 == Participating hosts (6 -> 4) == Missing (2): shard-glk shard-glkb == Build changes == * Linux: CI_DRM_4043 -> Patchwork_8664 CI_DRM_4043: 5fca09304f690a61408129c7ed166d7f1ddfa66c @ git://anongit.freedesktop.org/gfx-ci/linux IGT_4421: 27cd3793a3a183e4ec70c392f293e840642f2799 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools Patchwork_8664: de9215d6a97175391aa3ef8a7f6b10bd5dda9a17 @ git://anongit.freedesktop.org/gfx-ci/linux piglit_4421: 45e115f293fd6acc0c9647cf2d3b76be78819ba5 @ git://anongit.freedesktop.org/piglit == Logs == For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_8664/shards.html _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 31+ messages in thread
end of thread, other threads:[~2018-04-30 7:58 UTC | newest] Thread overview: 31+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-04-11 4:42 [RESEND PATCH 1/1] drm/i915/glk: Add MODULE_FIRMWARE for Geminilake ianwmorrison 2018-04-11 9:24 ` Jani Nikula 2018-04-11 9:24 ` Jani Nikula 2018-04-11 10:41 ` Ian W MORRISON 2018-04-11 10:41 ` Ian W MORRISON 2018-04-11 12:27 ` Jani Nikula 2018-04-11 13:04 ` Ian W MORRISON 2018-04-16 22:47 ` Srivatsa, Anusha 2018-04-17 9:02 ` Jani Nikula 2018-04-17 9:02 ` Jani Nikula 2018-04-17 14:14 ` Joonas Lahtinen 2018-04-20 5:47 ` Ian W MORRISON 2018-04-20 7:50 ` Jani Nikula 2018-04-20 7:50 ` Jani Nikula 2018-04-20 8:25 ` Ian W MORRISON 2018-04-20 8:25 ` Ian W MORRISON 2018-04-20 18:03 ` Rodrigo Vivi 2018-04-20 18:03 ` Rodrigo Vivi 2018-04-20 18:30 ` Srivatsa, Anusha 2018-04-20 18:30 ` Srivatsa, Anusha 2018-04-21 1:22 ` Botello Ortega, Luis 2018-04-21 1:22 ` Botello Ortega, Luis 2018-04-21 22:46 ` Ian W MORRISON 2018-04-21 22:46 ` Ian W MORRISON 2018-04-25 23:15 ` Ian W MORRISON 2018-04-25 23:15 ` Ian W MORRISON 2018-04-27 9:20 ` Jani Nikula 2018-04-27 9:20 ` Jani Nikula 2018-04-30 7:58 ` Joonas Lahtinen 2018-04-11 10:54 ` ✓ Fi.CI.BAT: success for series starting with [RESEND,1/1] " Patchwork 2018-04-11 12:37 ` ✓ Fi.CI.IGT: " Patchwork
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.