mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>
To: Mingyou Chen <qby140326@gmail.com>
Cc: "Hans de Goede" <hansg@kernel.org>,
	"Ilya Gladyshev" <ilya.gladyshev@linux.dev>,
	KentoNion <kento@kekto.ru>,
	platform-driver-x86@vger.kernel.org,
	"ted eric" <ericted8810@gmail.com>, "Armin Wolf" <W_Armin@gmx.de>,
	LKML <linux-kernel@vger.kernel.org>,
	"Chris Taraszka" <chris@miget.com>,
	"Matias Civadda" <matias.civadda2342001@gmail.com>,
	"高 俊文" <btx342@gmail.com>, powerfullz <wleizc7319@gmail.com>,
	"孙 誉铭" <wolf109909@outlook.com>,
	"Miloš Vlku" <vlku.milos.fun@gmail.com>, Rsplwe <i@rsplwe.com>,
	"Bozhen Peng" <bozhenpeng93@gmail.com>,
	"Anton Karasev" <uselessfire@gmail.com>,
	"Jame Jao" <jaojame6@gmail.com>,
	"Serhiy Shliapuhin" <shlyapugin@gmail.com>
Subject: Re: [PATCH v14 4/6] platform/x86: redmi-wmi: Drop redmi-wmi driver
Date: Tue, 27 Oct 2026 12:23:58 +0200 (EET)	[thread overview]
Message-ID: <43db32ea-e8db-fcc3-fbe1-545f8c5fa33a@linux.intel.com> (raw)
In-Reply-To: <20261026102101.224568-5-qby140326@gmail.com>

On Mon, 26 Oct 2026, Mingyou Chen wrote:

> The redmi-wmi driver is no longer needed because its functionality
> has been superseded by the new generic bitland platform driver.
> All supported Redmi laptop models are now handled properly via the
> updated WMI interfaces in bitland-mifs-wmi.
> 
> Remove the obsolete code, Kconfig entries, and Makefile references
> to clean up the platform directory.
> 
> Signed-off-by: Mingyou Chen <qby140326@gmail.com>
> ---
>  MAINTAINERS                      |   6 --
>  drivers/platform/x86/Kconfig     |  15 +--
>  drivers/platform/x86/Makefile    |   1 -
>  drivers/platform/x86/redmi-wmi.c | 154 -------------------------------
>  4 files changed, 8 insertions(+), 168 deletions(-)
>  delete mode 100644 drivers/platform/x86/redmi-wmi.c
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 250c471e674d..8ca065eb448f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -23038,12 +23038,6 @@ S:	Maintained
>  T:	git https://github.com/pkshih/rtw.git
>  F:	drivers/net/wireless/realtek/rtw89/
>  
> -REDMIBOOK WMI DRIVERS
> -M:	Gladyshev Ilya <foxido@foxido.dev>
> -L:	platform-driver-x86@vger.kernel.org
> -S:	Maintained
> -F:	drivers/platform/x86/redmi-wmi.c
> -
>  BITLAND LAPTOP DRIVER
>  M:	Mingyou Chen <qby140326@gmail.com>
>  M:	Ilya Gladyshev <ilya.gladyshev@linux.dev>
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index 7ee9ff3879c9..6e561359a91b 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -79,16 +79,17 @@ config XIAOMI_WMI
>  	  be called xiaomi-wmi.
>  
>  config REDMI_WMI
> -	tristate "Redmibook WMI key driver"
> +	tristate "Redmibook WMI key driver (DEPRECATED)"
>  	depends on ACPI_WMI
> +	depends on HWMON
>  	depends on INPUT
> -	select INPUT_SPARSEKMAP
> +	depends on LEDS_CLASS
> +	depends on POWER_SUPPLY
> +	select BITLAND_MIFS_WMI
> +	default BITLAND_MIFS_WMI
>  	help
> -	  Say Y here if you want support for WMI-based hotkey events on
> -	  Xiaomi Redmibook devices.
> -
> -	  To compile this driver as a module, choose M here: the module will
> -	  be called redmi-wmi.
> +	  This option is deprecated and will be removed in a future release.
> +	  Please enable CONFIG_BITLAND_MIFS_WMI directly.

Hi,

I think what you should do instead is use default REDMI_WMI if REDMI_WMI 
on BITLAND side. Though I'm not sure if that's any different from plain 
default REDMI_WMI.

(I'm not always sure of things when it comes to Kconfig, so I don't know 
if it matters to avoid what is effectively default n when REDMI_WMI=n 
with the more complex clause or not, so played safe.)

Since this is only a transitional config symbol, you should drop the help 
text so it isn't user selectable anymore.

-- 
 i.

  parent reply	other threads:[~2026-10-27 10:24 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-26 10:20 [PATCH v14 RESEND 0/6] Merge redmi-wmi into bitland-mifs-wmi Mingyou Chen
2026-10-26 10:20 ` [PATCH v14 1/6] MAINTAINERS: Add maintainer entry of bitland-mifs-wmi driver Mingyou Chen
2026-10-26 10:20 ` [PATCH v14 2/6] platform/x86: bitland-mifs-wmi: Merge the function of redmi-wmi into the bitland driver Mingyou Chen
2026-10-26 10:20 ` [PATCH v14 3/6] platform/x86: bitland-mifs-wmi: Add Redmi mic-mute key entries Mingyou Chen
2026-10-26 10:20 ` [PATCH v14 4/6] platform/x86: redmi-wmi: Drop redmi-wmi driver Mingyou Chen
2026-10-26 22:21   ` kernel test robot
2026-10-27 10:23   ` Ilpo Järvinen [this message]
2026-10-26 10:21 ` [PATCH v14 5/6] platform/x86: bitland-mifs-wmi: Add quirks table Mingyou Chen
2026-10-26 10:21 ` [PATCH v14 6/6] platform/x86: bitland-mifs-wmi: Fallback to balanced platform profile if last performance profile is not available Mingyou Chen
2026-10-26 11:45 ` [PATCH v14 RESEND 0/6] Merge redmi-wmi into bitland-mifs-wmi Hans de Goede
2026-10-26 19:56   ` Chris Taraszka
2026-10-26 20:02     ` Hans de Goede
2026-10-26 20:20       ` Chris Taraszka
  -- strict thread matches above, loose matches on Subject: below --
2026-10-26 10:09 [PATCH v14 " Mingyou Chen
2026-10-26 10:09 ` [PATCH v14 4/6] platform/x86: redmi-wmi: Drop redmi-wmi driver Mingyou Chen

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=43db32ea-e8db-fcc3-fbe1-545f8c5fa33a@linux.intel.com \
    --to=ilpo.jarvinen@linux.intel.com \
    --cc=W_Armin@gmx.de \
    --cc=bozhenpeng93@gmail.com \
    --cc=btx342@gmail.com \
    --cc=chris@miget.com \
    --cc=ericted8810@gmail.com \
    --cc=hansg@kernel.org \
    --cc=i@rsplwe.com \
    --cc=ilya.gladyshev@linux.dev \
    --cc=jaojame6@gmail.com \
    --cc=kento@kekto.ru \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matias.civadda2342001@gmail.com \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=qby140326@gmail.com \
    --cc=shlyapugin@gmail.com \
    --cc=uselessfire@gmail.com \
    --cc=vlku.milos.fun@gmail.com \
    --cc=wleizc7319@gmail.com \
    --cc=wolf109909@outlook.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

all inboxes | Powered by JetHome®