From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B693B1E2023 for ; Wed, 4 Sep 2024 18:22:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.133.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725474131; cv=none; b=mhF7h3FUzQeW0vm/DZdgkgm70kMjkHrcE/ZN/S1h/9c4iV56MCBLOtyIPar3abyOjiEKns1lBgqz1gpVWPUXfBaaNe0/N4/7c/RsO7dxi/WkuFYadmdpTGLSwsCwvgh8Ji6bayon/fiyCoCMwNSgnsoGosKDhJujIu7CRtmm7b4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725474131; c=relaxed/simple; bh=tWOx+kJUNjhWQxU5i1TgfPGae9CYkDJkGDzWYJi6pak=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=suYVEoQGu0aJ06RVDERQznY8bd/rkKtPFI4V51lAMAkjrTdBDxU3fMyVC5Qbk6F/97UdkAD8aHD1qt90Y5fYz+sMw95lzDs5wHzl2EKOzMtuH24h0tvVXI2pwsO9rwFs5cUymfb6hWVOgCzj2XQYoEjoN0LwbVc0u1VT8khgSE4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=bIwZCgfr; arc=none smtp.client-ip=170.10.133.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="bIwZCgfr" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1725474127; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=BgEVCBZc6OJujUCg9MZDAnWZMkIdyqhdTdlNtGcy9IM=; b=bIwZCgfrG7tPaGtk7fDcopYdKxsQC2xZkR2/z9CVUsfuv4ao5ZKbzXHAwNRG7JWl2sN+A2 u+1FbKoFtv5FlGJBukgFes3fXow5WijeuzJcN01GDPmPsYbCjmgQXkebUp7gvAOXo2lDlu ztY09xj9baOrtKD57mi9oqYcL4kPOHY= Received: from mail-ed1-f71.google.com (mail-ed1-f71.google.com [209.85.208.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-614-wxZuUp7fOyGjP2ZSGC42YA-1; Wed, 04 Sep 2024 14:22:06 -0400 X-MC-Unique: wxZuUp7fOyGjP2ZSGC42YA-1 Received: by mail-ed1-f71.google.com with SMTP id 4fb4d7f45d1cf-5c245525d37so3485631a12.3 for ; Wed, 04 Sep 2024 11:22:06 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1725474125; x=1726078925; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=BgEVCBZc6OJujUCg9MZDAnWZMkIdyqhdTdlNtGcy9IM=; b=qIOCeNX1OknpIL8sNrwwABlebyEXq+h6uNeN8I/SuXlk/LKm4QKG80dgXYx9DIESgK fJ0Hg4tFoFHEOfZz3RPoIf5H4taWlXQ1UA/A6XxmqH8LIUCIOoMD116z69+R2nnIW76U rn/ld9RYIjnndOQQVgQF4Aa2N+iUtH+4wNqtIXIRSiIIN7opEdDixAA4vrKRMuaYy75l VhEcmu7nU2FIYrT7XqWPcnPgVIRPNmGyfTjckk72TMk9R9coQNuesjFfzAMSQpr173p1 63aG2GUKVmUEMb6+j3xBNQcIsi4EiJ0gtZw015Beb94g8+2m9EXOo8JOcUogYn6oLf6G ArZg== X-Forwarded-Encrypted: i=1; AJvYcCXj0CuTHd5fvTHV4qc5+P9AefSlAIhpcPhl68xnSHimpf4A7i1pYqlDAMRRQs1J27VpxcQsKj783nhSI6Q=@vger.kernel.org X-Gm-Message-State: AOJu0YwDlgDb3S60b3AiioKGXb4Bx/59By08ML4kWMgw1tZEQ7U8w8b/ BhUXx245VqlpPK432UOm+EeA0erE31R6tHJlttaZf2h5oiD0Uks8eVZz8AkNPfzqiOqfsU9XbLZ UJo74MvIbzEVrDFsrwfVSZjYDjtr9g6pldyeT54sX1itsMTXQdkXyEONNhw/gNw== X-Received: by 2002:a17:907:2d07:b0:a80:7193:bd93 with SMTP id a640c23a62f3a-a897f84d734mr1658547766b.25.1725474124765; Wed, 04 Sep 2024 11:22:04 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHXei4yR90yXA7ay3rYSeWasBYKFoPrJq296h7UUZ2xoHcRGV/eMLJFnkLA17wbys/c+/poUg== X-Received: by 2002:a17:907:2d07:b0:a80:7193:bd93 with SMTP id a640c23a62f3a-a897f84d734mr1658545266b.25.1725474124020; Wed, 04 Sep 2024 11:22:04 -0700 (PDT) Received: from ?IPV6:2001:1c00:c32:7800:5bfa:a036:83f0:f9ec? (2001-1c00-0c32-7800-5bfa-a036-83f0-f9ec.cable.dynamic.v6.ziggo.nl. [2001:1c00:c32:7800:5bfa:a036:83f0:f9ec]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a8a623e33f6sm20727966b.212.2024.09.04.11.22.02 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 04 Sep 2024 11:22:02 -0700 (PDT) Message-ID: Date: Wed, 4 Sep 2024 20:22:02 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v1 1/1] platform/x86: x86-android-tablets: Fix spelling in the comments To: Andy Shevchenko , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org Cc: =?UTF-8?Q?Ilpo_J=C3=A4rvinen?= References: <20240902150625.2722187-1-andriy.shevchenko@linux.intel.com> Content-Language: en-US, nl From: Hans de Goede In-Reply-To: <20240902150625.2722187-1-andriy.shevchenko@linux.intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Hi, On 9/2/24 5:06 PM, Andy Shevchenko wrote: > Fix spelling across comments (besides obvious grammar issues): > - spell words in full, e.g., 'img' --> 'image' > - refer to 'gpio-keys' consistently > - refer to acpi_power_off() clearly as to function > - make sure that the first line is only for the affected model(s) > - miscellaneous improvements > > Signed-off-by: Andy Shevchenko Thank you for your patch, I've applied this patch to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Note it will show up in my review-hans branch once I've pushed my local branch there, which might take a while. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > .../platform/x86/x86-android-tablets/Kconfig | 2 +- > .../platform/x86/x86-android-tablets/asus.c | 8 ++--- > .../platform/x86/x86-android-tablets/core.c | 20 ++++++------- > .../platform/x86/x86-android-tablets/dmi.c | 11 +++---- > .../platform/x86/x86-android-tablets/lenovo.c | 22 +++++++------- > .../platform/x86/x86-android-tablets/other.c | 30 +++++++++---------- > .../x86/x86-android-tablets/shared-psy-info.c | 4 +-- > .../x86-android-tablets/x86-android-tablets.h | 2 +- > 8 files changed, 50 insertions(+), 49 deletions(-) > > diff --git a/drivers/platform/x86/x86-android-tablets/Kconfig b/drivers/platform/x86/x86-android-tablets/Kconfig > index b591419de80c..88d9e8f2ff24 100644 > --- a/drivers/platform/x86/x86-android-tablets/Kconfig > +++ b/drivers/platform/x86/x86-android-tablets/Kconfig > @@ -20,4 +20,4 @@ config X86_ANDROID_TABLETS > are missing from the DSDT. > > If you have a x86 Android tablet say Y or M here, for a generic x86 > - distro config say M here. > + distro configuration say M here. > diff --git a/drivers/platform/x86/x86-android-tablets/asus.c b/drivers/platform/x86/x86-android-tablets/asus.c > index 227afbb51078..07fbeab2319a 100644 > --- a/drivers/platform/x86/x86-android-tablets/asus.c > +++ b/drivers/platform/x86/x86-android-tablets/asus.c > @@ -37,7 +37,7 @@ static const struct x86_gpio_button asus_me176c_tf103c_lid __initconst = { > .pin = 12, > }; > > -/* Asus ME176C tablets have an Android factory img with everything hardcoded */ > +/* Asus ME176C tablets have an Android factory image with everything hardcoded */ > static const char * const asus_me176c_accel_mount_matrix[] = { > "-1", "0", "0", > "0", "1", "0", > @@ -112,7 +112,7 @@ static const struct x86_i2c_client_info asus_me176c_i2c_clients[] __initconst = > }, > .adapter_path = "\\_SB_.I2C5", > }, { > - /* kxtj21009 accel */ > + /* kxtj21009 accelerometer */ > .board_info = { > .type = "kxtj21009", > .addr = 0x0f, > @@ -181,7 +181,7 @@ const struct x86_dev_info asus_me176c_info __initconst = { > .modules = bq24190_modules, > }; > > -/* Asus TF103C tablets have an Android factory img with everything hardcoded */ > +/* Asus TF103C tablets have an Android factory image with everything hardcoded */ > static const char * const asus_tf103c_accel_mount_matrix[] = { > "0", "-1", "0", > "-1", "0", "0", > @@ -280,7 +280,7 @@ static const struct x86_i2c_client_info asus_tf103c_i2c_clients[] __initconst = > }, > .adapter_path = "\\_SB_.I2C5", > }, { > - /* kxtj21009 accel */ > + /* kxtj21009 accelerometer */ > .board_info = { > .type = "kxtj21009", > .addr = 0x0f, > diff --git a/drivers/platform/x86/x86-android-tablets/core.c b/drivers/platform/x86/x86-android-tablets/core.c > index 919ef4471229..1427a9a39008 100644 > --- a/drivers/platform/x86/x86-android-tablets/core.c > +++ b/drivers/platform/x86/x86-android-tablets/core.c > @@ -26,19 +26,19 @@ > static struct platform_device *x86_android_tablet_device; > > /* > - * This helper allows getting a gpio_desc *before* the actual device consuming > - * the GPIO has been instantiated. This function _must_ only be used to handle > - * this special case such as e.g. : > + * This helper allows getting a GPIO descriptor *before* the actual device > + * consuming it has been instantiated. This function MUST only be used to > + * handle this special case such as, e.g.: > * > * 1. Getting an IRQ from a GPIO for i2c_board_info.irq which is passed to > * i2c_client_new() to instantiate i2c_client-s; or > - * 2. Calling desc_to_gpio() to get an old style GPIO number for gpio_keys > + * 2. Calling desc_to_gpio() to get an old style GPIO number for gpio-keys > * platform_data which still uses old style GPIO numbers. > * > - * Since the consuming device has not been instatiated yet a dynamic lookup > - * is generated using the special x86_android_tablet dev for dev_id. > + * Since the consuming device has not been instantiated yet a dynamic lookup > + * is generated using the special x86_android_tablet device for dev_id. > * > - * For normal GPIO lookups a standard static gpiod_lookup_table _must_ be used. > + * For normal GPIO lookups a standard static struct gpiod_lookup_table MUST be used. > */ > int x86_android_tablet_get_gpiod(const char *chip, int pin, const char *con_id, > bool active_low, enum gpiod_flags dflags, > @@ -87,7 +87,7 @@ int x86_acpi_irq_helper_get(const struct x86_acpi_irq_data *data) > /* > * The DSDT may already reference the GSI in a device skipped by > * acpi_quirk_skip_i2c_client_enumeration(). Unregister the GSI > - * to avoid EBUSY errors in this case. > + * to avoid -EBUSY errors in this case. > */ > acpi_unregister_gsi(data->index); > irq = acpi_register_gsi(NULL, data->index, data->trigger, data->polarity); > @@ -379,7 +379,7 @@ static __init int x86_android_tablet_probe(struct platform_device *pdev) > } > } > > - /* + 1 to make space for (optional) gpio_keys_button pdev */ > + /* + 1 to make space for the (optional) gpio_keys_button platform device */ > pdevs = kcalloc(dev_info->pdev_count + 1, sizeof(*pdevs), GFP_KERNEL); > if (!pdevs) { > x86_android_tablet_remove(pdev); > @@ -432,7 +432,7 @@ static __init int x86_android_tablet_probe(struct platform_device *pdev) > > buttons[i] = dev_info->gpio_button[i].button; > buttons[i].gpio = desc_to_gpio(gpiod); > - /* Release gpiod so that gpio-keys can request it */ > + /* Release GPIO descriptor so that gpio-keys can request it */ > devm_gpiod_put(&x86_android_tablet_device->dev, gpiod); > } > > diff --git a/drivers/platform/x86/x86-android-tablets/dmi.c b/drivers/platform/x86/x86-android-tablets/dmi.c > index 387dd092c4dd..034f97a8ff27 100644 > --- a/drivers/platform/x86/x86-android-tablets/dmi.c > +++ b/drivers/platform/x86/x86-android-tablets/dmi.c > @@ -99,14 +99,15 @@ const struct dmi_system_id x86_android_tablet_ids[] __initconst = { > { > /* Lenovo Yoga Book X91F / X91L */ > .matches = { > - /* Non exact match to match F + L versions */ > + /* Inexact match to match F + L versions */ > DMI_MATCH(DMI_PRODUCT_NAME, "Lenovo YB1-X91"), > }, > .driver_data = (void *)&lenovo_yogabook_x91_info, > }, > { > /* > - * Lenovo Yoga Tablet 2 Pro 1380F/L (13") This has more or less > + * Lenovo Yoga Tablet 2 Pro 1380F/L (13") > + * This has more or less > * the same BIOS as the 830F/L or 1050F/L (8" and 10") below, > * but unlike the 8" / 10" models which share the same mainboard > * this model has a different mainboard. > @@ -124,8 +125,8 @@ const struct dmi_system_id x86_android_tablet_ids[] __initconst = { > }, > { > /* > - * Lenovo Yoga Tablet 2 830F/L or 1050F/L (The 8" and 10" > - * Lenovo Yoga Tablet 2 use the same mainboard) > + * Lenovo Yoga Tablet 2 830F/L or 1050F/L > + * The 8" and 10" * Lenovo Yoga Tablet 2 use the same mainboard. > */ > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "Intel Corp."), > @@ -163,7 +164,7 @@ const struct dmi_system_id x86_android_tablet_ids[] __initconst = { > .driver_data = (void *)&nextbook_ares8_info, > }, > { > - /* Nextbook Ares 8A (CHT version)*/ > + /* Nextbook Ares 8A (CHT version) */ > .matches = { > DMI_MATCH(DMI_SYS_VENDOR, "Insyde"), > DMI_MATCH(DMI_PRODUCT_NAME, "CherryTrail"), > diff --git a/drivers/platform/x86/x86-android-tablets/lenovo.c b/drivers/platform/x86/x86-android-tablets/lenovo.c > index 74f39b658d2c..ae087f1471c1 100644 > --- a/drivers/platform/x86/x86-android-tablets/lenovo.c > +++ b/drivers/platform/x86/x86-android-tablets/lenovo.c > @@ -59,7 +59,7 @@ static struct lp855x_platform_data lenovo_lp8557_reg_only_pdata = { > .initial_brightness = 128, > }; > > -/* Lenovo Yoga Book X90F / X90L's Android factory img has everything hardcoded */ > +/* Lenovo Yoga Book X90F / X90L's Android factory image has everything hardcoded */ > > static const struct property_entry lenovo_yb1_x90_wacom_props[] = { > PROPERTY_ENTRY_U32("hid-descr-addr", 0x0001), > @@ -262,7 +262,7 @@ const struct x86_dev_info lenovo_yogabook_x90_info __initconst = { > .init = lenovo_yb1_x90_init, > }; > > -/* Lenovo Yoga Book X91F/L Windows tablet needs manual instantiation of the fg client */ > +/* Lenovo Yoga Book X91F/L Windows tablet needs manual instantiation of the fuel-gauge client */ > static const struct x86_i2c_client_info lenovo_yogabook_x91_i2c_clients[] __initconst = { > { > /* BQ27542 fuel-gauge */ > @@ -281,7 +281,7 @@ const struct x86_dev_info lenovo_yogabook_x91_info __initconst = { > .i2c_client_count = ARRAY_SIZE(lenovo_yogabook_x91_i2c_clients), > }; > > -/* Lenovo Yoga Tablet 2 1050F/L's Android factory img has everything hardcoded */ > +/* Lenovo Yoga Tablet 2 1050F/L's Android factory image has everything hardcoded */ > static const struct property_entry lenovo_yoga_tab2_830_1050_bq24190_props[] = { > PROPERTY_ENTRY_STRING_ARRAY_LEN("supplied-from", tusb1211_chg_det_psy, 1), > PROPERTY_ENTRY_REF("monitored-battery", &generic_lipo_hv_4v35_battery_node), > @@ -521,9 +521,9 @@ static int __init lenovo_yoga_tab2_830_1050_init_codec(void) > } > > /* > - * These tablet's DSDT does not set acpi_gbl_reduced_hardware, so acpi_power_off > + * These tablet's DSDT does not set acpi_gbl_reduced_hardware, so acpi_power_off() > * gets used as pm_power_off handler. This causes "poweroff" on these tablets > - * to hang hard. Requiring pressing the powerbutton for 30 seconds *twice* > + * to hang hard. Requiring pressing the power button for 30 seconds *twice* > * followed by a normal 3 second press to recover. Avoid this by doing an EFI > * poweroff instead. > */ > @@ -546,7 +546,7 @@ static int __init lenovo_yoga_tab2_830_1050_init(struct device *dev) > if (ret) > return ret; > > - /* SYS_OFF_PRIO_FIRMWARE + 1 so that it runs before acpi_power_off */ > + /* SYS_OFF_PRIO_FIRMWARE + 1 so that it runs before acpi_power_off() */ > lenovo_yoga_tab2_830_1050_sys_off_handler = > register_sys_off_handler(SYS_OFF_MODE_POWER_OFF, SYS_OFF_PRIO_FIRMWARE + 1, > lenovo_yoga_tab2_830_1050_power_off, NULL); > @@ -742,7 +742,7 @@ static int __init lenovo_yoga_tab2_1380_init(struct device *dev) > if (ret) > return ret; > > - /* SYS_OFF_PRIO_FIRMWARE + 1 so that it runs before acpi_power_off */ > + /* SYS_OFF_PRIO_FIRMWARE + 1 so that it runs before acpi_power_off() */ > lenovo_yoga_tab2_830_1050_sys_off_handler = > register_sys_off_handler(SYS_OFF_MODE_POWER_OFF, SYS_OFF_PRIO_FIRMWARE + 1, > lenovo_yoga_tab2_830_1050_power_off, NULL); > @@ -799,7 +799,7 @@ static const struct software_node fg_bq25890_1_supply_node = { > .properties = fg_bq25890_1_supply_props, > }; > > -/* bq25892 charger settings for the flat lipo battery behind the screen */ > +/* bq25892 charger settings for the flat LiPo battery behind the screen */ > static const struct property_entry lenovo_yt3_bq25892_0_props[] = { > PROPERTY_ENTRY_STRING_ARRAY("supplied-from", lenovo_yt3_bq25892_0_suppliers), > PROPERTY_ENTRY_U32("linux,iinlim-percentage", 40), > @@ -833,7 +833,7 @@ static const struct software_node lenovo_yt3_hideep_ts_node = { > > static const struct x86_i2c_client_info lenovo_yt3_i2c_clients[] __initconst = { > { > - /* bq27500 fuel-gauge for the flat lipo battery behind the screen */ > + /* bq27500 fuel-gauge for the flat LiPo battery behind the screen */ > .board_info = { > .type = "bq27500", > .addr = 0x55, > @@ -842,7 +842,7 @@ static const struct x86_i2c_client_info lenovo_yt3_i2c_clients[] __initconst = { > }, > .adapter_path = "\\_SB_.PCI0.I2C1", > }, { > - /* bq25892 charger for the flat lipo battery behind the screen */ > + /* bq25892 charger for the flat LiPo battery behind the screen */ > .board_info = { > .type = "bq25892", > .addr = 0x6b, > @@ -859,7 +859,7 @@ static const struct x86_i2c_client_info lenovo_yt3_i2c_clients[] __initconst = { > .con_id = "bq25892_0_irq", > }, > }, { > - /* bq27500 fuel-gauge for the round li-ion cells in the hinge */ > + /* bq27500 fuel-gauge for the round Li-ion cells in the hinge */ > .board_info = { > .type = "bq27500", > .addr = 0x55, > diff --git a/drivers/platform/x86/x86-android-tablets/other.c b/drivers/platform/x86/x86-android-tablets/other.c > index eb0e55c69dfe..aad7114c43c3 100644 > --- a/drivers/platform/x86/x86-android-tablets/other.c > +++ b/drivers/platform/x86/x86-android-tablets/other.c > @@ -20,7 +20,7 @@ > #include "shared-psy-info.h" > #include "x86-android-tablets.h" > > -/* Acer Iconia One 7 B1-750 has an Android factory img with everything hardcoded */ > +/* Acer Iconia One 7 B1-750 has an Android factory image with everything hardcoded */ > static const char * const acer_b1_750_mount_matrix[] = { > "-1", "0", "0", > "0", "1", "0", > @@ -98,7 +98,7 @@ const struct x86_dev_info acer_b1_750_info __initconst = { > * Advantech MICA-071 > * This is a standard Windows tablet, but it has an extra "quick launch" button > * which is not described in the ACPI tables in anyway. > - * Use the x86-android-tablets infra to create a gpio-button device for this. > + * Use the x86-android-tablets infra to create a gpio-keys device for this. > */ > static const struct x86_gpio_button advantech_mica_071_button __initconst = { > .button = { > @@ -209,7 +209,7 @@ const struct x86_dev_info chuwi_hi8_info __initconst = { > * This comes in both Windows and Android versions and even on Android > * the DSDT is mostly sane. This tablet has 2 extra general purpose buttons > * in the button row with the power + volume-buttons labeled P and F. > - * Use the x86-android-tablets infra to create a gpio-button device for these. > + * Use the x86-android-tablets infra to create a gpio-keys device for these. > */ > static const struct x86_gpio_button cyberbook_t116_buttons[] __initconst = { > { > @@ -276,7 +276,7 @@ const struct x86_dev_info czc_p10t __initconst = { > .init = czc_p10t_init, > }; > > -/* Medion Lifetab S10346 tablets have an Android factory img with everything hardcoded */ > +/* Medion Lifetab S10346 tablets have an Android factory image with everything hardcoded */ > static const char * const medion_lifetab_s10346_accel_mount_matrix[] = { > "0", "1", "0", > "1", "0", "0", > @@ -305,7 +305,7 @@ static const struct software_node medion_lifetab_s10346_touchscreen_node = { > > static const struct x86_i2c_client_info medion_lifetab_s10346_i2c_clients[] __initconst = { > { > - /* kxtj21009 accel */ > + /* kxtj21009 accelerometer */ > .board_info = { > .type = "kxtj21009", > .addr = 0x0f, > @@ -359,7 +359,7 @@ const struct x86_dev_info medion_lifetab_s10346_info __initconst = { > .gpiod_lookup_tables = medion_lifetab_s10346_gpios, > }; > > -/* Nextbook Ares 8 (BYT) tablets have an Android factory img with everything hardcoded */ > +/* Nextbook Ares 8 (BYT) tablets have an Android factory image with everything hardcoded */ > static const char * const nextbook_ares8_accel_mount_matrix[] = { > "0", "-1", "0", > "-1", "0", "0", > @@ -387,7 +387,7 @@ static const struct software_node nextbook_ares8_touchscreen_node = { > > static const struct x86_i2c_client_info nextbook_ares8_i2c_clients[] __initconst = { > { > - /* Freescale MMA8653FC accel */ > + /* Freescale MMA8653FC accelerometer */ > .board_info = { > .type = "mma8653", > .addr = 0x1d, > @@ -428,7 +428,7 @@ const struct x86_dev_info nextbook_ares8_info __initconst = { > .gpiod_lookup_tables = nextbook_ares8_gpios, > }; > > -/* Nextbook Ares 8A (CHT) tablets have an Android factory img with everything hardcoded */ > +/* Nextbook Ares 8A (CHT) tablets have an Android factory image with everything hardcoded */ > static const char * const nextbook_ares8a_accel_mount_matrix[] = { > "1", "0", "0", > "0", "-1", "0", > @@ -446,7 +446,7 @@ static const struct software_node nextbook_ares8a_accel_node = { > > static const struct x86_i2c_client_info nextbook_ares8a_i2c_clients[] __initconst = { > { > - /* Freescale MMA8653FC accel */ > + /* Freescale MMA8653FC accelerometer */ > .board_info = { > .type = "mma8653", > .addr = 0x1d, > @@ -497,7 +497,7 @@ const struct x86_dev_info nextbook_ares8a_info __initconst = { > * Peaq C1010 > * This is a standard Windows tablet, but it has a special Dolby button. > * This button has a WMI interface, but that is broken. Instead of trying to > - * use the broken WMI interface, instantiate a gpio_keys device for this. > + * use the broken WMI interface, instantiate a gpio-keys device for this. > */ > static const struct x86_gpio_button peaq_c1010_button __initconst = { > .button = { > @@ -521,7 +521,7 @@ const struct x86_dev_info peaq_c1010_info __initconst = { > * Whitelabel (sold as various brands) TM800A550L tablets. > * These tablet's DSDT contains a whole bunch of bogus ACPI I2C devices > * (removed through acpi_quirk_skip_i2c_client_enumeration()) and > - * the touchscreen fwnode has the wrong GPIOs. > + * the touchscreen firmware node has the wrong GPIOs. > */ > static const char * const whitelabel_tm800a550l_accel_mount_matrix[] = { > "-1", "0", "0", > @@ -566,7 +566,7 @@ static const struct x86_i2c_client_info whitelabel_tm800a550l_i2c_clients[] __in > .polarity = ACPI_ACTIVE_HIGH, > }, > }, { > - /* kxcj91008 accel */ > + /* kxcj91008 accelerometer */ > .board_info = { > .type = "kxcj91008", > .addr = 0x0f, > @@ -598,12 +598,12 @@ const struct x86_dev_info whitelabel_tm800a550l_info __initconst = { > }; > > /* > - * The fwnode for ktd2026 on Xaomi pad2. It composed of a RGB LED node > + * The firmware node for ktd2026 on Xaomi pad2. It composed of a RGB LED node > * with three subnodes for each color (B/G/R). The RGB LED node is named > * "multi-led" to align with the name in the device tree. > */ > > -/* main fwnode for ktd2026 */ > +/* Main firmware node for ktd2026 */ > static const struct software_node ktd2026_node = { > .name = "ktd2026", > }; > @@ -665,7 +665,7 @@ static const struct software_node *ktd2026_node_group[] = { > }; > > /* > - * For the LEDs which backlight the menu / home / back capacitive buttons on > + * For the LEDs which backlight the Menu / Home / Back capacitive buttons on > * the bottom bezel. These are attached to a TPS61158 LED controller which > * is controlled by the "pwm_soc_lpss_2" PWM output. > */ > diff --git a/drivers/platform/x86/x86-android-tablets/shared-psy-info.c b/drivers/platform/x86/x86-android-tablets/shared-psy-info.c > index d2d0aa51bc3f..a46fa15acfb1 100644 > --- a/drivers/platform/x86/x86-android-tablets/shared-psy-info.c > +++ b/drivers/platform/x86/x86-android-tablets/shared-psy-info.c > @@ -39,7 +39,7 @@ const struct software_node fg_bq25890_supply_node = { > .properties = fg_bq25890_supply_props, > }; > > -/* LiPo HighVoltage (max 4.35V) settings used by most devs with a HV bat. */ > +/* LiPo HighVoltage (max 4.35V) settings used by most devs with a HV battery */ > static const struct property_entry generic_lipo_hv_4v35_battery_props[] = { > PROPERTY_ENTRY_STRING("compatible", "simple-battery"), > PROPERTY_ENTRY_STRING("device-chemistry", "lithium-ion"), > @@ -80,7 +80,7 @@ const char * const bq24190_modules[] __initconst = { > NULL > }; > > -/* Generic pdevs array and gpio-lookups for micro USB ID pin handling */ > +/* Generic platform device array and GPIO lookup table for micro USB ID pin handling */ > const struct platform_device_info int3496_pdevs[] __initconst = { > { > /* For micro USB ID pin handling */ > diff --git a/drivers/platform/x86/x86-android-tablets/x86-android-tablets.h b/drivers/platform/x86/x86-android-tablets/x86-android-tablets.h > index 86402b9b46a3..5517e438c7b6 100644 > --- a/drivers/platform/x86/x86-android-tablets/x86-android-tablets.h > +++ b/drivers/platform/x86/x86-android-tablets/x86-android-tablets.h > @@ -61,7 +61,7 @@ struct x86_serdev_info { > const char *ctrl_uid; > const char *ctrl_devname; > /* > - * ATM the serdev core only supports of or ACPI matching; and sofar all > + * ATM the serdev core only supports of or ACPI matching; and so far all > * Android x86 tablets DSDTs have usable serdev nodes, but sometimes > * under the wrong controller. So we just tie the existing serdev ACPI > * node to the right controller.