From nobody Mon Dec 15 22:08:17 2025 Received: from vs81.iboxed.net (vs10.datenmanufaktur-hosting.net [213.160.73.65]) (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 5DB1A3164DE; Fri, 31 Oct 2025 19:52:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=213.160.73.65 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761940322; cv=none; b=QoftR+XiR0koC4TZPdQTyNVSep+SS3oSzNXfYwLmZX5hu2p5MimZkg7drkNkllQijrV9E8l2Kvaj2VVSx06gtkX9Fg7jG0ztXgV1ARP3g30ralUKkm5ePH1nwfk5oBn7RqWjOSkdEtJVWuDGTdCxD+HbxmdahAiA/MJqvzR2hl8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761940322; c=relaxed/simple; bh=r+b3tjTeJV3KjwgBwxOoZF/b85zoSUZhDlvxxf/bruY=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=uIqTUXM1Ho1N8Y9WsnRluCCFnGhWs6KNcCWCXIYaCLnxV3VT2zE0VnldUPpXo+rXGF3zmzkXptjN0r5VbZBKVoewRVzgmIunyS4MvduoKETRtPg95U34cmpOH8hJxdUTYjFNSfpzpPsgkDnYZBybFmfrB9LRgL9o3lNDnMClPIc= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=blala.de; spf=pass smtp.mailfrom=blala.de; arc=none smtp.client-ip=213.160.73.65 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=blala.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=blala.de Received: from blala.de (localhost [127.0.0.1]) by vs81.iboxed.net (8.15.2/8.15.2/Debian-14~deb10u1) with ESMTP id 59VJvKRc001670; Fri, 31 Oct 2025 19:57:20 GMT Received: (from akurz@localhost) by blala.de (8.15.2/8.15.2/Submit) id 59VJvKTr001666; Fri, 31 Oct 2025 19:57:20 GMT From: Alexander Kurz To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Dmitry Torokhov , Lee Jones , Dzmitry Sankouski , Griffin Kroah-Hartman , Mathieu Dubois-Briand , Heiko Stuebner , "Dr . David Alan Gilbert" , Job Sava , Alexandre Belloni , =?UTF-8?q?Uwe=20Kleine-K=C3=B6nig?= Cc: devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-input@vger.kernel.org, Alexander Kurz Subject: [PATCH v7 5/6] Input: mc13783-pwrbutton: add OF support and drop platform_data Date: Fri, 31 Oct 2025 19:57:17 +0000 Message-Id: <20251031195718.1586-6-akurz@blala.de> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20251031195718.1586-1-akurz@blala.de> References: <20251031195718.1586-1-akurz@blala.de> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add OF support for the mc13783-pwrbutton so that it can be used with modern DT based systems, dropping support for platform_data. Signed-off-by: Alexander Kurz Acked-by: Lee Jones --- drivers/input/misc/mc13783-pwrbutton.c | 104 +++++++++++++++++++++---- drivers/mfd/mc13xxx-core.c | 4 - include/linux/mfd/mc13xxx.h | 14 ---- 3 files changed, 88 insertions(+), 34 deletions(-) diff --git a/drivers/input/misc/mc13783-pwrbutton.c b/drivers/input/misc/mc= 13783-pwrbutton.c index 08618c59197f..0fa630adff19 100644 --- a/drivers/input/misc/mc13783-pwrbutton.c +++ b/drivers/input/misc/mc13783-pwrbutton.c @@ -27,6 +27,7 @@ #include #include #include +#include #include #include =20 @@ -41,10 +42,20 @@ struct mc13783_pwrb { struct mc13xxx *mc13783; const struct mc13xxx_button_devtype *devtype; int flags; + int b_on_flags[3]; + unsigned int b_on_key[3]; unsigned short keymap[3]; int irq[3]; }; =20 +#define MC13783_BUTTON_DBNC_0MS 0 +#define MC13783_BUTTON_DBNC_30MS 1 +#define MC13783_BUTTON_DBNC_150MS 2 +#define MC13783_BUTTON_DBNC_750MS 3 +#define MC13783_BUTTON_ENABLE (1 << 2) +#define MC13783_BUTTON_POL_INVERT (1 << 3) +#define MC13783_BUTTON_RESET_EN (1 << 4) + #define MC13783_PWRB_B1_POL_INVERT (1 << 0) #define MC13783_PWRB_B2_POL_INVERT (1 << 1) #define MC13783_PWRB_B3_POL_INVERT (1 << 2) @@ -88,9 +99,69 @@ static irqreturn_t button_irq(int irq, void *_priv) return IRQ_HANDLED; } =20 +static int mc13xxx_pwrbutton_parse_properties(struct platform_device *pdev, + struct mc13783_pwrb *priv) +{ + struct fwnode_handle *child; + struct device *dev =3D &pdev->dev; + struct mc13xxx_button_devtype *devtype =3D + (struct mc13xxx_button_devtype *)platform_get_device_id(pdev)->driver_da= ta; + + struct fwnode_handle *parent __free(fwnode_handle) =3D + device_get_named_child_node(dev->parent, "buttons"); + if (!parent) + return -ENODATA; + + fwnode_for_each_named_child_node(parent, child, "onkey") { + u32 idx; + u8 dbnc =3D MC13783_BUTTON_DBNC_30MS; + u16 dbnc_ms; + + if (fwnode_property_read_u32(child, "reg", &idx)) + continue; + + if (idx > devtype->button_id_max) { + dev_warn(dev, "reg out of range\n"); + continue; + } + + fwnode_property_read_u16(child, "debounce-delay-ms", &dbnc_ms); + switch (dbnc_ms) { + case 0: + dbnc =3D MC13783_BUTTON_DBNC_0MS; + break; + case 30: + dbnc =3D MC13783_BUTTON_DBNC_30MS; + break; + case 150: + dbnc =3D MC13783_BUTTON_DBNC_150MS; + break; + case 750: + dbnc =3D MC13783_BUTTON_DBNC_750MS; + break; + default: + dev_warn(dev, "invalid debounce-delay-ms value\n"); + continue; + } + + if (fwnode_property_read_u32(child, "linux,code", &priv->b_on_key[idx])) + continue; + + if (fwnode_property_read_bool(child, "active-low")) + priv->b_on_flags[idx] |=3D MC13783_BUTTON_POL_INVERT; + + if (fwnode_property_read_bool(child, "fsl,enable-reset")) + priv->b_on_flags[idx] |=3D MC13783_BUTTON_RESET_EN; + + priv->b_on_flags[idx] |=3D MC13783_BUTTON_ENABLE | dbnc; + } + + return 0; +} + static int mc13783_pwrbutton_probe(struct platform_device *pdev) { - const struct mc13xxx_buttons_platform_data *pdata; + struct device *dev =3D &pdev->dev; struct mc13xxx *mc13783 =3D dev_get_drvdata(pdev->dev.parent); struct mc13xxx_button_devtype *devtype =3D (struct mc13xxx_button_devtype *)pdev->id_entry->driver_data; @@ -100,11 +171,8 @@ static int mc13783_pwrbutton_probe(struct platform_dev= ice *pdev) int reg =3D 0; int irq =3D 0; =20 - pdata =3D dev_get_platdata(&pdev->dev); - if (!pdata) { - dev_err(&pdev->dev, "missing platform data\n"); - return -ENODEV; - } + if (!dev->parent->of_node) + return -ENODATA; =20 pwr =3D devm_input_allocate_device(&pdev->dev); if (!pwr) @@ -114,14 +182,18 @@ static int mc13783_pwrbutton_probe(struct platform_de= vice *pdev) if (!priv) return -ENOMEM; =20 - if (devtype->button_id_max < 2 && pdata->b_on_flags[2] & 0x3) { + err =3D mc13xxx_pwrbutton_parse_properties(pdev, priv); + if (err) + return err; + + if (devtype->button_id_max < 2 && priv->b_on_flags[2] & 0x3) { dev_err(&pdev->dev, "button not supported\n"); return -ENODEV; } =20 - reg |=3D (pdata->b_on_flags[0] & 0x3) << MC13783_POWER_CONTROL_2_ON1BDBNC; - reg |=3D (pdata->b_on_flags[1] & 0x3) << MC13783_POWER_CONTROL_2_ON2BDBNC; - reg |=3D (pdata->b_on_flags[2] & 0x3) << MC13783_POWER_CONTROL_2_ON3BDBNC; + reg |=3D (priv->b_on_flags[0] & 0x3) << MC13783_POWER_CONTROL_2_ON1BDBNC; + reg |=3D (priv->b_on_flags[1] & 0x3) << MC13783_POWER_CONTROL_2_ON2BDBNC; + reg |=3D (priv->b_on_flags[2] & 0x3) << MC13783_POWER_CONTROL_2_ON3BDBNC; =20 priv->pwr =3D pwr; priv->mc13783 =3D mc13783; @@ -130,17 +202,17 @@ static int mc13783_pwrbutton_probe(struct platform_de= vice *pdev) mc13xxx_lock(mc13783); =20 for (int i =3D 0; i < devtype->button_id_max; i++) { - if ((pdata->b_on_flags[i] & MC13783_BUTTON_ENABLE) =3D=3D 0) + if ((priv->b_on_flags[i] & MC13783_BUTTON_ENABLE) =3D=3D 0) continue; =20 - priv->keymap[i] =3D pdata->b_on_key[i]; - if (pdata->b_on_key[i] !=3D KEY_RESERVED) - __set_bit(pdata->b_on_key[i], pwr->keybit); + priv->keymap[i] =3D priv->b_on_key[i]; + if (priv->b_on_key[i] !=3D KEY_RESERVED) + __set_bit(priv->b_on_key[i], pwr->keybit); =20 - if (pdata->b_on_flags[i] & MC13783_BUTTON_POL_INVERT) + if (priv->b_on_flags[i] & MC13783_BUTTON_POL_INVERT) priv->flags |=3D (MC13783_PWRB_B1_POL_INVERT << i); =20 - if (pdata->b_on_flags[i] & MC13783_BUTTON_RESET_EN) + if (priv->b_on_flags[i] & MC13783_BUTTON_RESET_EN) reg |=3D (MC13783_POWER_CONTROL_2_ON1BRSTEN << i); =20 irq =3D platform_get_irq_byname(pdev, devtype->irq_name[i]); diff --git a/drivers/mfd/mc13xxx-core.c b/drivers/mfd/mc13xxx-core.c index c29974722704..9512136e821b 100644 --- a/drivers/mfd/mc13xxx-core.c +++ b/drivers/mfd/mc13xxx-core.c @@ -504,10 +504,6 @@ int mc13xxx_common_init(struct device *dev) &pdata->regulators, sizeof(pdata->regulators)); mc13xxx_add_subdevice_pdata(mc13xxx, "%s-led", pdata->leds, sizeof(*pdata->leds)); - mc13xxx_add_subdevice_pdata_res(mc13xxx, "%s-pwrbutton", - pdata->buttons, sizeof(*pdata->buttons), - mc13xxx->variant->button_resources, - mc13xxx->variant->button_resources_size); if (mc13xxx->flags & MC13XXX_USE_CODEC) mc13xxx_add_subdevice_pdata(mc13xxx, "%s-codec", pdata->codec, sizeof(*pdata->codec)); diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h index 71c7d3614d4c..ac3765df341d 100644 --- a/include/linux/mfd/mc13xxx.h +++ b/include/linux/mfd/mc13xxx.h @@ -174,19 +174,6 @@ struct mc13xxx_leds_platform_data { u32 led_control[MAX_LED_CONTROL_REGS]; }; =20 -#define MC13783_BUTTON_DBNC_0MS 0 -#define MC13783_BUTTON_DBNC_30MS 1 -#define MC13783_BUTTON_DBNC_150MS 2 -#define MC13783_BUTTON_DBNC_750MS 3 -#define MC13783_BUTTON_ENABLE (1 << 2) -#define MC13783_BUTTON_POL_INVERT (1 << 3) -#define MC13783_BUTTON_RESET_EN (1 << 4) - -struct mc13xxx_buttons_platform_data { - int b_on_flags[3]; - unsigned int b_on_key[3]; -}; - #define MC13783_TS_ATO_FIRST false #define MC13783_TS_ATO_EACH true =20 @@ -219,7 +206,6 @@ struct mc13xxx_platform_data { =20 struct mc13xxx_regulator_platform_data regulators; struct mc13xxx_leds_platform_data *leds; - struct mc13xxx_buttons_platform_data *buttons; struct mc13xxx_ts_platform_data touch; struct mc13xxx_codec_platform_data *codec; }; --=20 2.39.5