From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Jani Nikula <jani.nikula@linux.intel.com>
Cc: "Murthy, Arun R" <arun.r.murthy@intel.com>,
"intel-xe@lists.freedesktop.org" <intel-xe@lists.freedesktop.org>
Subject: Re: [Intel-xe] [PATCH 1/2] drm/xe: Add module parameter 'fastboot'
Date: Mon, 25 Sep 2023 14:47:17 +0300 [thread overview]
Message-ID: <ZRFzRdldsC7BXITb@intel.com> (raw)
In-Reply-To: <874jjio5ik.fsf@intel.com>
On Mon, Sep 25, 2023 at 02:29:23PM +0300, Jani Nikula wrote:
> On Mon, 25 Sep 2023, "Shankar, Uma" <uma.shankar@intel.com> wrote:
> >> -----Original Message-----
> >> From: Intel-xe <intel-xe-bounces@lists.freedesktop.org> On Behalf Of Arun R
> >> Murthy
> >> Sent: Sunday, September 24, 2023 8:18 PM
> >> To: intel-xe@lists.freedesktop.org
> >> Cc: Murthy, Arun R <arun.r.murthy@intel.com>
> >> Subject: [Intel-xe] [PATCH 1/2] drm/xe: Add module parameter 'fastboot'
> >>
> >> Fastboot is by default enabled on all Display 9+ platforms.
> >> Module param is added in xe for controlling the same.
> >
> > On Xe, we don't have the module param yet. So, there is no
> > way to change the default what we set here in this patch as of
> > now. We should modify the statement here.
> >
> > It should be ok to get rid of the #define and have fastboot disabled
> > by default for now. But we should work on the task to get the
> > module param or any other mechanism acceptable by maintainers
> > to control fastboot feature.
>
> The question is, do we *want* that module parameter in xe?
We don't want it in i915 either. Just nuke it IMO.
--
Ville Syrjälä
Intel
next prev parent reply other threads:[~2023-09-25 11:47 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-24 14:47 [Intel-xe] [PATCH 1/2] drm/xe: Add module parameter 'fastboot' Arun R Murthy
2023-09-24 14:47 ` [Intel-xe] [PATCH 2/2] Revert "FIXME: drm/i915: xe doesn't have fastboot module parameter" Arun R Murthy
2023-09-25 9:17 ` Shankar, Uma
2023-09-25 11:27 ` Jani Nikula
2023-09-24 14:59 ` [Intel-xe] ✓ CI.Patch_applied: success for series starting with [1/2] drm/xe: Add module parameter 'fastboot' Patchwork
2023-09-24 14:59 ` [Intel-xe] ✗ CI.checkpatch: warning " Patchwork
2023-09-24 15:00 ` [Intel-xe] ✓ CI.KUnit: success " Patchwork
2023-09-24 15:07 ` [Intel-xe] ✓ CI.Build: " Patchwork
2023-09-24 15:08 ` [Intel-xe] ✓ CI.Hooks: " Patchwork
2023-09-24 15:09 ` [Intel-xe] ✗ CI.checksparse: warning " Patchwork
2023-09-24 15:43 ` [Intel-xe] ✓ CI.BAT: success " Patchwork
2023-09-25 9:15 ` [Intel-xe] [PATCH 1/2] " Shankar, Uma
2023-09-25 11:29 ` Jani Nikula
2023-09-25 11:47 ` Ville Syrjälä [this message]
2023-09-26 4:28 ` Murthy, Arun R
2023-09-26 5:11 ` [Intel-xe] [PATCHv2 1/2] drm/i915: Remove the " Arun R Murthy
2023-09-26 5:11 ` [Intel-xe] [PATCHv2 2/2] Revert "FIXME: drm/i915: xe doesn't have fastboot module parameter" Arun R Murthy
2023-09-26 7:40 ` [Intel-xe] [PATCHv2 1/2] drm/i915: Remove the module parameter 'fastboot' Jani Nikula
2023-09-26 5:54 ` [Intel-xe] ✗ CI.Patch_applied: failure for series starting with [PATCHv2,1/2] drm/i915: Remove the module parameter 'fastboot' (rev2) Patchwork
2023-10-12 8:47 ` [Intel-xe] [PATCH 1/2] drm/i915: Remove the module parameter 'fastboot' Arun R Murthy
2023-10-12 8:47 ` [Intel-xe] [PATCH 2/2] Revert "FIXME: drm/i915: xe doesn't have fastboot module parameter" Arun R Murthy
2023-10-12 9:06 ` [Intel-xe] [PATCH 1/2] drm/i915: Remove the module parameter 'fastboot' Jani Nikula
2023-10-12 9:10 ` Murthy, Arun R
2023-10-12 9:23 ` Jani Nikula
2023-10-12 9:14 ` [Intel-xe] ✗ CI.Patch_applied: failure for series starting with [2/2] Revert "FIXME: drm/i915: xe doesn't have fastboot module parameter" (rev4) Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZRFzRdldsC7BXITb@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=arun.r.murthy@intel.com \
--cc=intel-xe@lists.freedesktop.org \
--cc=jani.nikula@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.