[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <dd1616d5-4808-46be-9167-6fbfe76e0c73@redhat.com>
Date: Tue, 25 Mar 2025 10:05:13 +0100
From: Jocelyn Falempe <jfalempe@...hat.com>
To: Miguel Ojeda <ojeda@...nel.org>, Alex Gaynor <alex.gaynor@...il.com>
Cc: Boqun Feng <boqun.feng@...il.com>, Gary Guo <gary@...yguo.net>,
Björn Roy Baron <bjorn3_gh@...tonmail.com>,
Benno Lossin <benno.lossin@...ton.me>,
Andreas Hindborg <a.hindborg@...nel.org>, Alice Ryhl <aliceryhl@...gle.com>,
Trevor Gross <tmgross@...ch.edu>, Danilo Krummrich <dakr@...nel.org>,
rust-for-linux@...r.kernel.org, linux-kernel@...r.kernel.org,
patches@...ts.linux.dev,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH 02/10] drm/panic: add missing Markdown code span
On 24/03/2025 22:03, Miguel Ojeda wrote:
> Add missing Markdown code span.
>
> This was found using the Clippy `doc_markdown` lint, which we may want
> to enable.
Thanks, it looks good to me.
Reviewed-by: Jocelyn Falempe <jfalempe@...hat.com>
You want to take the whole series in the rust tree?
Otherwise I can push the patch 1-2 to drm-misc-next if needed.
--
Jocelyn
>
> Fixes: cb5164ac43d0 ("drm/panic: Add a QR code panic screen")
> Signed-off-by: Miguel Ojeda <ojeda@...nel.org>
> ---
> drivers/gpu/drm/drm_panic_qr.rs | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_panic_qr.rs b/drivers/gpu/drm/drm_panic_qr.rs
> index 9bd4d131f033..a8f832598c70 100644
> --- a/drivers/gpu/drm/drm_panic_qr.rs
> +++ b/drivers/gpu/drm/drm_panic_qr.rs
> @@ -914,7 +914,7 @@ fn draw_all(&mut self, data: impl Iterator<Item = u8>) {
> /// will be encoded as binary segment, otherwise it will be encoded
> /// efficiently as a numeric segment, and appended to the URL.
> /// * `data_len`: Length of the data, that needs to be encoded, must be less
> -/// than data_size.
> +/// than `data_size`.
> /// * `data_size`: Size of data buffer, it should be at least 4071 bytes to hold
> /// a V40 QR code. It will then be overwritten with the QR code image.
> /// * `tmp`: A temporary buffer that the QR code encoder will use, to write the
Powered by blists - more mailing lists