From patchwork Wed Dec 11 11:34:38 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nick Chan X-Patchwork-Id: 849559 Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 18B13230274; Wed, 11 Dec 2024 11:35:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.174 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733916942; cv=none; b=OiFe36LgMD1ivRzfZue7sNc4E2f0sKpfEFo0C2xYyI9pFZ2gsPBV9f0YY/IxCunCL1J2/fumZIFmvaviQVHGRXTNilIv2TbxLqy69lxfQC3emb2Sm2LTu1vb/TaIZ5/R8B4BjMgJ8EXq4mcQvkgvnqRSX4u3baYGJjRCBywraS0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1733916942; c=relaxed/simple; bh=/NGZcEXzKfllPc1kHyrV4Jpt9+1VCGGg+ROxJ06pXvI=; h=From:To:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=B1Sn297BOJIfBC3azeogO649f7EJzsSmhDt3/wDQ0c5bZlR0SxqJ/O8eUKS3HbSWypDTy3FZlbheixWa5R+Oj7D68prHF2EQSxoN/DJguJBzk/agQJw57kwLIrSL9fqom2GiWV2EpZQf9wUQR6FPa1DVItJhrosP9aWLWEXmBnI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=DpOHwFzM; arc=none smtp.client-ip=209.85.214.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="DpOHwFzM" Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-21644aca3a0so37618465ad.3; Wed, 11 Dec 2024 03:35:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1733916940; x=1734521740; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:from:to:cc:subject:date:message-id :reply-to; bh=jkHtq9idtKzQ6fNQxVQI03Qs+Bo7JPqfFg778nP11bo=; b=DpOHwFzMKPq1gTkAJTcy+/phZTZ4EEVXEBz/LwDPsZaCNY4ADcl5QMO8gCVrLk30Yr itmlELMKElWfvZ8gLIx2ugHj6IQcv+0VjbSkH4iZqaCOHhbI0WwOVeko0ZNHh7TnCoYH GTiZsC6PE/exF2K8ijh6AQknrDDzzPlZ+VnqnXzCb14BDRZPKwQUDCJW2OSS7ggjlq9Z /gyv6agxLmUTn0nzS0tKokgm80mNicq7r8WCoBc7i9FvY13kpU1hATVa2EJVR9gfSlsg NL5/yqKWtH35kv/OVwtqFnm+VoB6/m+PwHNeNkGNhdYeyZ3AhcejZ5Vy921JdJquHg5+ TpaA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1733916940; x=1734521740; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=jkHtq9idtKzQ6fNQxVQI03Qs+Bo7JPqfFg778nP11bo=; b=tpujvo/pU9kMZ8xurU7ZD/h+PKEwoer/navC8FXPko9xP63bZVTE5t/ztbrnhFvNju SW1ZGENtLMTMddE/9NivobbnSMHiDcWrMJrlGbnG4R0u6B9skkGcIWBUg5BAwDYpui3F 3jSZ1YOuVH0XncmVmLzZ2rnpJ5b7RaPsJQqCUzPgtLCp0xbV8aXFGk/XLjVI4nwC7KoL f5oeBWHkWM83hu/G9oPa74vbHSXq6+BtogXaNZwglnTAt/b5ImquwblN6VoNnJwA8rgQ hgqDUYtFyUg6puFuWXrwLc2JlJC8T+ZVjyC18mIotR4/QWybTRbEGRuZ8yupqfdvMNIJ QYHA== X-Forwarded-Encrypted: i=1; AJvYcCU7VVjgVT7bFDgOV2QDdJDcE+PSS+mSp/MPXEjZKXGLe9mXx7+KKzPLs1SVo78QkuWO3nyC+c9wXxPaqLpE@vger.kernel.org, AJvYcCULmU25DaQkEjUa3av4i+LBTGbW/HfOeCZcQP8j3Px7MlE1b/DkY+PF/8tCmQiKwqwUVtUynbmSDYKJ@vger.kernel.org, AJvYcCXHxh72EzeFrQP4A8djp0tX3o00V95HFgCpQgmN2Ujo+WyhdatuuXJ7H3sisY48dgg9CGUQZAk/wWKJsbA=@vger.kernel.org, AJvYcCXsFB68NUheiTdTin3yGxW4pXNegxCGUhAYa8gxvQmpC/fpMBDFzFzrp+4ZUtLZUc+KRPsIx3M5546vug==@vger.kernel.org X-Gm-Message-State: AOJu0Yx6pT2kJrtD6xdix9rOndIbp0x1pXrYkH6iq+EcDkCQSIJb3//A Ivo/LBl3g6R+/3WvI17znEIBYX2ZGuyiXb/EoqCzjGYymatBccQV X-Gm-Gg: ASbGncs8PbrK4Y7tUxY1bG17xLP19rt7Qx8DXVZoM+707sVMl+9VyNAGc6V9RB7ehTA KbMYwW0+r71GzB+BipfBtBy369LSLVjMB33aK6yvREVM/y9tIk4ml9ojhERP6lSGpExuCStrySr dtiPMy2bRvn1gULtF6dqKV0yot04Kocuc6F1/nXPrbBLjWDRngT7Ub6FmrRDwK+x1fvmb+WEC/f kja/dm3OFqPpiTyCJTi5c1zgxbGzZqRJGaW5SyGfpi6rUd4Xal1WF4DbYg+ck1+5bwO3Q== X-Google-Smtp-Source: AGHT+IEvwlSiJ/A/e/bX+VxNlELH+lImFhyckJUBzQs/9yPJZMVZ9VQiinmNaf4t+2eVGwvQVdZbIA== X-Received: by 2002:a17:902:f646:b0:212:615f:c1 with SMTP id d9443c01a7336-21778534ccamr43159815ad.14.1733916940321; Wed, 11 Dec 2024 03:35:40 -0800 (PST) Received: from localhost.localdomain ([59.188.211.160]) by smtp.googlemail.com with ESMTPSA id d9443c01a7336-216404df131sm56765635ad.203.2024.12.11.03.35.36 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 11 Dec 2024 03:35:39 -0800 (PST) From: Nick Chan To: Lee Jones , Daniel Thompson , Jingoo Han , Pavel Machek , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Helge Deller , Hector Martin , Sven Peter , Alyssa Rosenzweig , Nick Chan , dri-devel@lists.freedesktop.org, linux-leds@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fbdev@vger.kernel.org, asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org Subject: [PATCH v4 2/3] backlight: dwi_bl: Add Apple DWI backlight driver Date: Wed, 11 Dec 2024 19:34:38 +0800 Message-ID: <20241211113512.19009-3-towinchenmi@gmail.com> X-Mailer: git-send-email 2.47.1 In-Reply-To: <20241211113512.19009-1-towinchenmi@gmail.com> References: <20241211113512.19009-1-towinchenmi@gmail.com> Precedence: bulk X-Mailing-List: linux-leds@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Add driver for backlight controllers attached via Apple DWI 2-wire interface, which is found on some Apple iPhones, iPads and iPod touches with a LCD display. Although there is an existing apple_bl driver, it is for backlight controllers on Intel Macs attached via PCI, which is completely different from the Samsung-derived DWI block. Signed-off-by: Nick Chan --- drivers/video/backlight/Kconfig | 12 +++ drivers/video/backlight/Makefile | 1 + drivers/video/backlight/dwi_bl.c | 122 +++++++++++++++++++++++++++++++ 3 files changed, 135 insertions(+) create mode 100644 drivers/video/backlight/dwi_bl.c diff --git a/drivers/video/backlight/Kconfig b/drivers/video/backlight/Kconfig index 3614a5d29c71..debb1326df63 100644 --- a/drivers/video/backlight/Kconfig +++ b/drivers/video/backlight/Kconfig @@ -290,6 +290,18 @@ config BACKLIGHT_APPLE If you have an Intel-based Apple say Y to enable a driver for its backlight. +config BACKLIGHT_APPLE_DWI + tristate "Apple DWI 2-Wire Interface Backlight Driver" + depends on ARCH_APPLE || COMPILE_TEST + default y + help + Say Y to enable the backlight driver for backlight controllers + attached via the Apple DWI 2-wire interface which is found in some + Apple iPhones, iPads and iPod touches. + + To compile this driver as a module, choose M here: the module will + be called dwi_bl. + config BACKLIGHT_QCOM_WLED tristate "Qualcomm PMIC WLED Driver" select REGMAP diff --git a/drivers/video/backlight/Makefile b/drivers/video/backlight/Makefile index 8fc98f760a8a..45cc80d9ae22 100644 --- a/drivers/video/backlight/Makefile +++ b/drivers/video/backlight/Makefile @@ -23,6 +23,7 @@ obj-$(CONFIG_BACKLIGHT_ADP5520) += adp5520_bl.o obj-$(CONFIG_BACKLIGHT_ADP8860) += adp8860_bl.o obj-$(CONFIG_BACKLIGHT_ADP8870) += adp8870_bl.o obj-$(CONFIG_BACKLIGHT_APPLE) += apple_bl.o +obj-$(CONFIG_BACKLIGHT_APPLE_DWI) += dwi_bl.o obj-$(CONFIG_BACKLIGHT_AS3711) += as3711_bl.o obj-$(CONFIG_BACKLIGHT_BD6107) += bd6107.o obj-$(CONFIG_BACKLIGHT_CLASS_DEVICE) += backlight.o diff --git a/drivers/video/backlight/dwi_bl.c b/drivers/video/backlight/dwi_bl.c new file mode 100644 index 000000000000..59e5cad0fbd8 --- /dev/null +++ b/drivers/video/backlight/dwi_bl.c @@ -0,0 +1,122 @@ +// SPDX-License-Identifier: GPL-2.0 OR MIT +/* + * Driver for backlight controllers attached via Apple DWI 2-wire interface + * + * Copyright (c) 2024 Nick Chan + */ + +#include +#include +#include +#include +#include +#include + +#define DWI_BL_CTL 0x0 +#define DWI_BL_CTL_SEND1 BIT(0) +#define DWI_BL_CTL_SEND2 BIT(4) +#define DWI_BL_CTL_SEND3 BIT(5) +#define DWI_BL_CTL_LE_DATA BIT(6) +/* Only used on Apple A9 and later */ +#define DWI_BL_CTL_SEND4 BIT(12) + +#define DWI_BL_CMD 0x4 +#define DWI_BL_CMD_TYPE GENMASK(31, 28) +#define DWI_BL_CMD_TYPE_SET_BRIGHTNESS 0xa +#define DWI_BL_CMD_DATA GENMASK(10, 0) + +#define DWI_BL_CTL_SEND (DWI_BL_CTL_SEND1 | \ + DWI_BL_CTL_SEND2 | \ + DWI_BL_CTL_SEND3 | \ + DWI_BL_CTL_LE_DATA | \ + DWI_BL_CTL_SEND4) + +#define DWI_BL_MAX_BRIGHTNESS 2047 + +struct apple_dwi_bl { + void __iomem *base; +}; + +static int dwi_bl_update_status(struct backlight_device *bl) +{ + struct apple_dwi_bl *dwi_bl = bl_get_data(bl); + + int brightness = backlight_get_brightness(bl); + + u32 cmd = 0; + + cmd |= FIELD_PREP(DWI_BL_CMD_DATA, brightness); + cmd |= FIELD_PREP(DWI_BL_CMD_TYPE, DWI_BL_CMD_TYPE_SET_BRIGHTNESS); + + writel(cmd, dwi_bl->base + DWI_BL_CMD); + writel(DWI_BL_CTL_SEND, dwi_bl->base + DWI_BL_CTL); + + return 0; +} + +static int dwi_bl_get_brightness(struct backlight_device *bl) +{ + struct apple_dwi_bl *dwi_bl = bl_get_data(bl); + + u32 cmd = readl(dwi_bl->base + DWI_BL_CMD); + + return FIELD_GET(DWI_BL_CMD_DATA, cmd); +} + +static const struct backlight_ops dwi_bl_ops = { + .options = BL_CORE_SUSPENDRESUME, + .get_brightness = dwi_bl_get_brightness, + .update_status = dwi_bl_update_status +}; + +static int dwi_bl_probe(struct platform_device *dev) +{ + struct apple_dwi_bl *dwi_bl; + struct backlight_device *bl; + struct backlight_properties props; + struct resource *res; + + dwi_bl = devm_kzalloc(&dev->dev, sizeof(*dwi_bl), GFP_KERNEL); + if (!dwi_bl) + return -ENOMEM; + + dwi_bl->base = devm_platform_get_and_ioremap_resource(dev, 0, &res); + if (IS_ERR(dwi_bl->base)) + return PTR_ERR(dwi_bl->base); + + memset(&props, 0, sizeof(struct backlight_properties)); + props.type = BACKLIGHT_RAW; + props.max_brightness = DWI_BL_MAX_BRIGHTNESS; + + bl = devm_backlight_device_register(&dev->dev, dev->name, &dev->dev, + dwi_bl, &dwi_bl_ops, &props); + if (IS_ERR(bl)) + return PTR_ERR(bl); + + platform_set_drvdata(dev, dwi_bl); + + bl->props.brightness = dwi_bl_get_brightness(bl); + + return 0; +} + +static const struct of_device_id dwi_bl_of_match[] = { + { .compatible = "apple,dwi-bl" }, + {}, +}; + +MODULE_DEVICE_TABLE(of, dwi_bl_of_match); + +static struct platform_driver dwi_bl_driver = { + .driver = { + .name = "dwi-bl", + .of_match_table = dwi_bl_of_match + }, + .probe = dwi_bl_probe, +}; + +module_platform_driver(dwi_bl_driver); + +MODULE_DESCRIPTION("Apple DWI Backlight Driver"); +MODULE_AUTHOR("Nick Chan "); +MODULE_LICENSE("Dual MIT/GPL");