dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Luca Ceresoli" <luca.ceresoli@bootlin.com>
To: "Tomi Valkeinen" <tomi.valkeinen@ideasonboard.com>,
	phasta@kernel.org, "Hemanth Selam" <hemanth.selam@gmail.com>,
	"Sumit Semwal" <sumit.semwal@linaro.org>,
	"Christian König" <christian.koenig@amd.com>,
	"Andrzej Hajda" <andrzej.hajda@intel.com>,
	"Neil Armstrong" <neil.armstrong@linaro.org>,
	"Robert Foss" <rfoss@kernel.org>,
	"Laurent Pinchart" <Laurent.pinchart@ideasonboard.com>,
	"Jonas Karlman" <jonas@kwiboo.se>,
	"Jernej Skrabec" <jernej.skrabec@gmail.com>,
	"Luca Ceresoli" <luca.ceresoli@bootlin.com>,
	"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
	"Maxime Ripard" <mripard@kernel.org>,
	"Thomas Zimmermann" <tzimmermann@suse.de>,
	"David Airlie" <airlied@gmail.com>,
	"Simona Vetter" <simona@ffwll.ch>,
	"Inki Dae" <inki.dae@samsung.com>,
	"Jagan Teki" <jagan@amarulasolutions.com>,
	"Marek Szyprowski" <m.szyprowski@samsung.com>,
	"Patrik Jakobsson" <patrik.r.jakobsson@gmail.com>,
	"Laurentiu Palcu" <laurentiu.palcu@oss.nxp.com>,
	"Lucas Stach" <l.stach@pengutronix.de>,
	"Frank Li" <Frank.Li@nxp.com>,
	"Sascha Hauer" <s.hauer@pengutronix.de>,
	"Pengutronix Kernel Team" <kernel@pengutronix.de>,
	"Fabio Estevam" <festevam@gmail.com>,
	"Jianmin Lv" <lvjianmin@loongson.cn>,
	"Qianhai Wu" <wuqianhai@loongson.cn>,
	"Huacai Chen" <chenhuacai@kernel.org>,
	"Mingcong Bai" <jeffbai@aosc.io>,
	"Xi Ruoyao" <xry111@xry111.site>,
	"Icenowy Zheng" <zhengxingda@iscas.ac.cn>,
	"Dave Airlie" <airlied@redhat.com>,
	"Jocelyn Falempe" <jfalempe@redhat.com>,
	"Lyude Paul" <lyude@redhat.com>,
	"Danilo Krummrich" <dakr@kernel.org>,
	"Jessica Zhang" <jesszhan0024@gmail.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Kieran Bingham" <kieran.bingham+renesas@ideasonboard.com>,
	"Geert Uytterhoeven" <geert+renesas@glider.be>,
	"Magnus Damm" <magnus.damm@gmail.com>,
	"Matthew Brost" <matthew.brost@intel.com>,
	"Chen-Yu Tsai" <wens@kernel.org>,
	"Samuel Holland" <samuel@sholland.org>,
	"Huang Rui" <ray.huang@amd.com>,
	"Matthew Auld" <matthew.auld@intel.com>,
	"Dave Stevenson" <dave.stevenson@raspberrypi.com>,
	"Maíra Canal" <mcanal@igalia.com>,
	"Raspberry Pi Kernel Maintenance" <kernel-list@raspberrypi.com>,
	"Dmitry Osipenko" <dmitry.osipenko@collabora.com>,
	"Gurchetan Singh" <gurchetansingh@chromium.org>,
	"Chia-I Wu" <olvaffe@gmail.com>,
	"Zack Rusin" <zack.rusin@broadcom.com>,
	"Broadcom internal kernel review list"
	<bcm-kernel-feedback-list@broadcom.com>,
	"Michal Simek" <michal.simek@amd.com>,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	"Natalie Vock" <nat@pixelcluster.dev>,
	"Tejun Heo" <tj@kernel.org>,
	"Johannes Weiner" <hannes@cmpxchg.org>,
	"Michal Koutný" <mkoutny@suse.com>
Cc: <linux-kernel@vger.kernel.org>, <linux-media@vger.kernel.org>,
	<dri-devel@lists.freedesktop.org>,
	<linaro-mm-sig@lists.linaro.org>, <imx@lists.linux.dev>,
	<linux-arm-kernel@lists.infradead.org>,
	<nouveau@lists.freedesktop.org>, <virtualization@lists.linux.dev>,
	<spice-devel@lists.freedesktop.org>,
	<linux-renesas-soc@vger.kernel.org>,
	<linux-sunxi@lists.linux.dev>, <cgroups@vger.kernel.org>
Subject: Re: [PATCH 1/3] drm: fix typos in comments
Date: Fri, 04 Sep 2026 15:19:15 +0200	[thread overview]
Message-ID: <DL6K8VPR37C3.2LQJ8ZM6PIZTS@bootlin.com> (raw)
In-Reply-To: <a64864e0-3f10-4f81-b304-c45ef932817a@ideasonboard.com>

On Fri Sep 4, 2026 at 2:34 PM CEST, Tomi Valkeinen wrote:
> Hi,
>
> On 04/09/2026 15:13, Philipp Stanner wrote:
>> On Fri, 2026-09-04 at 17:34 +0530, Hemanth Selam wrote:
>>> Fix typos in comments, reported by scripts/checkpatch.pl using the
>>> misspelling list in scripts/spelling.txt.  Only touches comments, no code
>>> changes.
>>>
>>> Assisted-by: Cursor:claude-opus-5
>>> Signed-off-by: Hemanth Selam <hemanth.selam@gmail.com>
>>> ---
>>>   drivers/dma-buf/sync_debug.h                           | 2 +-
>>>   drivers/gpu/drm/bridge/parade-ps8622.c                 | 2 +-
>>>   drivers/gpu/drm/bridge/samsung-dsim.c                  | 2 +-
>>>   drivers/gpu/drm/bridge/tc358767.c                      | 6 +++---
>>>   drivers/gpu/drm/display/drm_dp_mst_topology.c          | 2 +-
>>>   drivers/gpu/drm/drm_drv.c                              | 2 +-
>>>   drivers/gpu/drm/gma500/oaktrail_lvds.c                 | 2 +-
>>>   drivers/gpu/drm/gma500/power.c                         | 2 +-
>>>   drivers/gpu/drm/gma500/psb_drv.h                       | 4 ++--
>>>   drivers/gpu/drm/imx/dcss/dcss-ctxld.c                  | 2 +-
>>>   drivers/gpu/drm/loongson/lsdc_drv.c                    | 2 +-
>>>   drivers/gpu/drm/mgag200/mgag200_g200se.c               | 2 +-
>>>   drivers/gpu/drm/nouveau/dispnv50/tile.h                | 2 +-
>>>   drivers/gpu/drm/nouveau/nouveau_bios.c                 | 2 +-
>>>   drivers/gpu/drm/nouveau/nouveau_exec.c                 | 2 +-
>>>   drivers/gpu/drm/nouveau/nouveau_led.c                  | 2 +-
>>>   drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c       | 2 +-
>>>   drivers/gpu/drm/nouveau/nvkm/engine/gr/gf100.c         | 2 +-
>>>   drivers/gpu/drm/omapdrm/dss/dispc.c                    | 4 ++--
>>>   drivers/gpu/drm/omapdrm/omap_gem.c                     | 2 +-
>>>   drivers/gpu/drm/panel/panel-novatek-nt36672a.c         | 2 +-
>>>   drivers/gpu/drm/panel/panel-samsung-s6e63j0x03.c       | 2 +-
>>>   drivers/gpu/drm/qxl/qxl_gem.c                          | 2 +-
>>>   drivers/gpu/drm/renesas/rcar-du/rcar_lvds.c            | 2 +-
>>>   drivers/gpu/drm/scheduler/sched_main.c                 | 4 ++--
>>>   drivers/gpu/drm/sun4i/sun4i_tcon.c                     | 2 +-
>>>   drivers/gpu/drm/tiny/panel-mipi-dbi.c                  | 2 +-
>>>   drivers/gpu/drm/ttm/ttm_pool.c                         | 2 +-
>>>   drivers/gpu/drm/vc4/vc4_hdmi.c                         | 2 +-
>>>   drivers/gpu/drm/vc4/vc4_regs.h                         | 2 +-
>>>   drivers/gpu/drm/virtio/virtgpu_ioctl.c                 | 2 +-
>>>   drivers/gpu/drm/vmwgfx/device_include/vm_basic_types.h | 2 +-
>>>   drivers/gpu/drm/vmwgfx/ttm_object.c                    | 2 +-
>>>   drivers/gpu/drm/vmwgfx/vmwgfx_drv.h                    | 6 +++---
>>>   drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c                | 2 +-
>>>   drivers/gpu/drm/vmwgfx/vmwgfx_kms.h                    | 2 +-
>>>   drivers/gpu/drm/vmwgfx/vmwgfx_so.h                     | 2 +-
>>>   drivers/gpu/drm/vmwgfx/vmwgfx_surface.c                | 2 +-
>>>   drivers/gpu/drm/xlnx/zynqmp_dp.c                       | 2 +-
>>>   drivers/gpu/ipu-v3/ipu-prg.c                           | 2 +-
>>>   include/drm/drm_framebuffer.h                          | 4 ++--
>>>   include/drm/drm_plane.h                                | 2 +-
>>>   include/uapi/drm/nouveau_drm.h                         | 2 +-
>>>   include/uapi/drm/vmwgfx_drm.h                          | 2 +-
>>>   kernel/cgroup/dmem.c                                   | 2 +-
>>>   45 files changed, 53 insertions(+), 53 deletions(-)

[...]

>> Not sure if the cost-benefit-ratio would then pay off?
> I would skip these. Fixing a typo in a key-word (say, a register name)
> would be useful, but I'm not sure if any of these count.

I agree, and a patch touching so many files will be annoying to cherry-pick
due to conflicts.

I'd save LLM water and energy consumption for more valuable changes.

Luca

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2026-09-04 13:19 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20260904120430.17224-1-hemanth.selam@gmail.com>
2026-09-04 12:04 ` [PATCH 1/3] drm: fix typos in comments Hemanth Selam
2026-09-04 12:13   ` Philipp Stanner
2026-09-04 12:34     ` Tomi Valkeinen
2026-09-04 13:19       ` Luca Ceresoli [this message]
2026-09-04 12:32   ` Natalie Vock
2026-09-04 12:04 ` [PATCH 2/3] drm: fix repeated words " Hemanth Selam
2026-09-04 12:09   ` sashiko-bot
2026-09-04 12:04 ` [PATCH 3/3] drm/vmwgfx: fix repeated word in log message Hemanth Selam
     [not found] <20260904120044.14303-1-hemanth.selam@gmail.com>
2026-09-04 12:00 ` [PATCH 1/3] drm: fix typos in comments Hemanth Selam

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=DL6K8VPR37C3.2LQJ8ZM6PIZTS@bootlin.com \
    --to=luca.ceresoli@bootlin.com \
    --cc=Frank.Li@nxp.com \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=airlied@gmail.com \
    --cc=airlied@redhat.com \
    --cc=andrzej.hajda@intel.com \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=cgroups@vger.kernel.org \
    --cc=chenhuacai@kernel.org \
    --cc=christian.koenig@amd.com \
    --cc=dakr@kernel.org \
    --cc=dave.stevenson@raspberrypi.com \
    --cc=dmitry.osipenko@collabora.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=festevam@gmail.com \
    --cc=geert+renesas@glider.be \
    --cc=gurchetansingh@chromium.org \
    --cc=hannes@cmpxchg.org \
    --cc=hemanth.selam@gmail.com \
    --cc=imx@lists.linux.dev \
    --cc=inki.dae@samsung.com \
    --cc=jagan@amarulasolutions.com \
    --cc=jeffbai@aosc.io \
    --cc=jernej.skrabec@gmail.com \
    --cc=jesszhan0024@gmail.com \
    --cc=jfalempe@redhat.com \
    --cc=jonas@kwiboo.se \
    --cc=kernel-list@raspberrypi.com \
    --cc=kernel@pengutronix.de \
    --cc=kieran.bingham+renesas@ideasonboard.com \
    --cc=kraxel@redhat.com \
    --cc=l.stach@pengutronix.de \
    --cc=laurentiu.palcu@oss.nxp.com \
    --cc=linaro-mm-sig@lists.linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=linux-sunxi@lists.linux.dev \
    --cc=lvjianmin@loongson.cn \
    --cc=lyude@redhat.com \
    --cc=m.szyprowski@samsung.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=magnus.damm@gmail.com \
    --cc=matthew.auld@intel.com \
    --cc=matthew.brost@intel.com \
    --cc=mcanal@igalia.com \
    --cc=michal.simek@amd.com \
    --cc=mkoutny@suse.com \
    --cc=mripard@kernel.org \
    --cc=nat@pixelcluster.dev \
    --cc=neil.armstrong@linaro.org \
    --cc=nouveau@lists.freedesktop.org \
    --cc=olvaffe@gmail.com \
    --cc=p.zabel@pengutronix.de \
    --cc=patrik.r.jakobsson@gmail.com \
    --cc=phasta@kernel.org \
    --cc=ray.huang@amd.com \
    --cc=rfoss@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=samuel@sholland.org \
    --cc=simona@ffwll.ch \
    --cc=spice-devel@lists.freedesktop.org \
    --cc=sumit.semwal@linaro.org \
    --cc=tj@kernel.org \
    --cc=tomi.valkeinen@ideasonboard.com \
    --cc=tzimmermann@suse.de \
    --cc=virtualization@lists.linux.dev \
    --cc=wens@kernel.org \
    --cc=wuqianhai@loongson.cn \
    --cc=xry111@xry111.site \
    --cc=zack.rusin@broadcom.com \
    --cc=zhengxingda@iscas.ac.cn \
    /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