From: Jani Nikula <jani.nikula@intel.com>
To: Ville Syrjala <ville.syrjala@linux.intel.com>,
linux-kernel@vger.kernel.org
Cc: "Lucas De Marchi" <lucas.demarchi@intel.com>,
"Dibin Moolakadan Subrahmanian"
<dibin.moolakadan.subrahmanian@intel.com>,
"Imre Deak" <imre.deak@intel.com>,
"David Laight" <david.laight.linux@gmail.com>,
"Geert Uytterhoeven" <geert+renesas@glider.be>,
"Matt Wagantall" <mattw@codeaurora.org>,
"Dejin Zheng" <zhengdejin5@gmail.com>,
intel-gfx@lists.freedesktop.org, intel-xe@lists.freedesktop.org,
"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Simona Vetter" <simona.vetter@ffwll.ch>
Subject: Re: [PATCH v2 1/3] iopoll: Generalize read_poll_timeout() into poll_timeout_us()
Date: Thu, 28 Aug 2025 12:51:35 +0300 [thread overview]
Message-ID: <c34ce332183d24ed29ed23852238fd5ca948d4f1@intel.com> (raw)
In-Reply-To: <20250826121859.15497-1-ville.syrjala@linux.intel.com>
On Tue, 26 Aug 2025, Ville Syrjala <ville.syrjala@linux.intel.com> wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
>
> While read_poll_timeout() & co. were originally introduced just
> for simple I/O usage scenarios they have since been generalized to
> be useful in more cases.
>
> However the interface is very cumbersome to use in the general case.
> Attempt to make it more flexible by combining the 'op', 'var' and
> 'args' parameter into just a single 'op' that the caller can fully
> specify.
Thanks for the patches!
Since there was no opposition now or the last time these were posted
[1], iopoll.h is not in MAINTAINERS, and the previous changes to the
file have gone in through various trees, I've gone ahead and merged the
lot to drm-intel-next with Simona's IRC ack.
I'll be rebasing and posting my follow-up i915 changes on top of this
shortly.
BR,
Jani.
[1] https://lore.kernel.org/r/20250702223439.19752-1-ville.syrjala@linux.intel.com
--
Jani Nikula, Intel
prev parent reply other threads:[~2025-08-28 9:51 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-26 12:18 [PATCH v2 1/3] iopoll: Generalize read_poll_timeout() into poll_timeout_us() Ville Syrjala
2025-08-26 12:18 ` [PATCH v2 2/3] iopoll: Avoid evaluating 'cond' twice in poll_timeout_us() Ville Syrjala
2025-08-26 12:18 ` [PATCH v2 3/3] iopoll: Reorder the timeout handling " Ville Syrjala
2025-08-26 12:55 ` ✓ i915.CI.BAT: success for series starting with [v2,1/3] iopoll: Generalize read_poll_timeout() into poll_timeout_us() Patchwork
2025-08-26 19:15 ` ✗ i915.CI.Full: failure " Patchwork
2025-08-28 9:51 ` Jani Nikula [this message]
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=c34ce332183d24ed29ed23852238fd5ca948d4f1@intel.com \
--to=jani.nikula@intel.com \
--cc=david.laight.linux@gmail.com \
--cc=dibin.moolakadan.subrahmanian@intel.com \
--cc=geert+renesas@glider.be \
--cc=imre.deak@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lucas.demarchi@intel.com \
--cc=mattw@codeaurora.org \
--cc=simona.vetter@ffwll.ch \
--cc=ville.syrjala@linux.intel.com \
--cc=zhengdejin5@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).