[PATCH v2 0/8] platform/x86: Fix typos in comments

Hemanth Selam posted 8 patches 2 weeks, 1 day ago
drivers/platform/x86/asus-laptop.c          | 2 +-
drivers/platform/x86/asus-nb-wmi.c          | 2 +-
drivers/platform/x86/hp/hp_accel.c          | 2 +-
drivers/platform/x86/intel/vsec_tpmi.c      | 2 +-
drivers/platform/x86/msi-laptop.c           | 2 +-
drivers/platform/x86/panasonic-laptop.c     | 2 +-
drivers/platform/x86/samsung-laptop.c       | 2 +-
drivers/platform/x86/uniwill/uniwill-acpi.c | 2 +-
8 files changed, 8 insertions(+), 8 deletions(-)
[PATCH v2 0/8] platform/x86: Fix typos in comments
Posted by Hemanth Selam 2 weeks, 1 day ago
v1 was one patch touching eight drivers.  Denis Benato asked for it to be
split one patch per file or driver, so here it is as eight patches, one
per driver.  The corrections themselves are unchanged from v1.

The words all come from scripts/spelling.txt, which is the list
checkpatch.pl uses.  codespell finds nothing further in these eight files
except "ALS" in asus-laptop.c, which is the Ambient Light Sensor and must
stay as it is.

Two of these comments have a grammar problem in addition to the
misspelling:

  asus-laptop.c     "The HWRS method return information about the
                    hardware" wants "returns"
  samsung-laptop.c  "This allow to call arbitrary sabi commands" wants
                    "allows calling"

I have left both alone so that this series stays a spelling fix and so
Mark's review still covers exactly the text he read.  I am happy to send
those two as a separate patch if you would like them corrected.

Mark Pearson reviewed v1 in full, so his tag is carried on every patch.
Denis offered his tag on the condition that Ilpo agrees to the split, and
it is included on that basis; please drop it if that is not right.

v1: https://lore.kernel.org/all/20260907085426.3661-1-hemanth.selam@gmail.com/

Hemanth Selam (8):
  platform/x86: asus-laptop: Fix typo in infos_show() comment
  platform/x86: asus-nb-wmi: Fix typo in asus_nb_wmi_quirks() comment
  platform/x86: hp_accel: Fix typo in lis3lv02d_acpi_write() comment
  platform/x86/intel/tpmi: Fix typo in TPMI_FEATURE_NAME_LEN comment
  platform/x86: msi-laptop: Fix typo in struct quirk_entry comment
  platform/x86: panasonic-laptop: Fix typo in the file history comment
  platform/x86: samsung-laptop: Fix typo in sabi_configs comment
  platform/x86: uniwill-laptop: Fix typo in the kbd LED brightness
    comment

 drivers/platform/x86/asus-laptop.c          | 2 +-
 drivers/platform/x86/asus-nb-wmi.c          | 2 +-
 drivers/platform/x86/hp/hp_accel.c          | 2 +-
 drivers/platform/x86/intel/vsec_tpmi.c      | 2 +-
 drivers/platform/x86/msi-laptop.c           | 2 +-
 drivers/platform/x86/panasonic-laptop.c     | 2 +-
 drivers/platform/x86/samsung-laptop.c       | 2 +-
 drivers/platform/x86/uniwill/uniwill-acpi.c | 2 +-
 8 files changed, 8 insertions(+), 8 deletions(-)

-- 
2.48.1
Re: [PATCH v2 0/8] platform/x86: Fix typos in comments
Posted by Ilpo Järvinen 1 week, 2 days ago
On Thu, 10 Sep 2026 10:14:35 +0530, Hemanth Selam wrote:

> v1 was one patch touching eight drivers.  Denis Benato asked for it to be
> split one patch per file or driver, so here it is as eight patches, one
> per driver.  The corrections themselves are unchanged from v1.
> 
> The words all come from scripts/spelling.txt, which is the list
> checkpatch.pl uses.  codespell finds nothing further in these eight files
> except "ALS" in asus-laptop.c, which is the Ambient Light Sensor and must
> stay as it is.
> 
> [...]

Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.

FYI [if applicable to your patch], as per Linus' policy change, also
fixes are mostly routed through for-next unless the fix is for a
commit introduced in the most recent cycle or is clearly a regression
fix.

The list of commits applied:
[1/8] platform/x86: asus-laptop: Fix typo in infos_show() comment
      commit: 3bf79c45d88b54c9a76f8bfb6733249acfc3c8b7
[2/8] platform/x86: asus-nb-wmi: Fix typo in asus_nb_wmi_quirks() comment
      commit: b6cf8c621b85648b82a6704b1a68106eb193616f
[3/8] platform/x86: hp_accel: Fix typo in lis3lv02d_acpi_write() comment
      commit: ad483da2140339400bc50c69b017f6fd0cc9774b
[4/8] platform/x86/intel/tpmi: Fix typo in TPMI_FEATURE_NAME_LEN comment
      commit: 65d3f988c5fae277a3ff732115697d252d5b3913
[5/8] platform/x86: msi-laptop: Fix typo in struct quirk_entry comment
      commit: d11503cd82f48242ccaf7b3d481a103d1a75105d
[6/8] platform/x86: panasonic-laptop: Fix typo in the file history comment
      commit: d766ef2dab2d718ca760e4f1771e883736ccb861
[7/8] platform/x86: samsung-laptop: Fix typo in sabi_configs comment
      commit: 0934b4b77a462240d591f90472b690788b3988b8
[8/8] platform/x86: uniwill-laptop: Fix typo in the kbd LED brightness comment
      commit: 974014bb4087fea3b481c0ac358873587f628663

--
 i.