Cover Letter Detail
Show a cover letter.
GET /api/covers/2217444/?format=api
{ "id": 2217444, "url": "http://patchwork.ozlabs.org/api/covers/2217444/?format=api", "web_url": "http://patchwork.ozlabs.org/project/rtc-linux/cover/20260329-sc27xx-mfd-cells-v3-0-9158dee41f74@abscue.de/", "project": { "id": 9, "url": "http://patchwork.ozlabs.org/api/projects/9/?format=api", "name": "Linux RTC development", "link_name": "rtc-linux", "list_id": "linux-rtc.vger.kernel.org", "list_email": "linux-rtc@vger.kernel.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260329-sc27xx-mfd-cells-v3-0-9158dee41f74@abscue.de>", "list_archive_url": null, "date": "2026-03-29T07:27:44", "name": "[v3,0/5] mfd: sc27xx: Use MFD cells and devm_mfd_add_devices()", "submitter": { "id": 85322, "url": "http://patchwork.ozlabs.org/api/people/85322/?format=api", "name": "Otto Pflüger", "email": "otto.pflueger@abscue.de" }, "mbox": "http://patchwork.ozlabs.org/project/rtc-linux/cover/20260329-sc27xx-mfd-cells-v3-0-9158dee41f74@abscue.de/mbox/", "series": [ { "id": 497909, "url": "http://patchwork.ozlabs.org/api/series/497909/?format=api", "web_url": "http://patchwork.ozlabs.org/project/rtc-linux/list/?series=497909", "date": "2026-03-29T07:27:46", "name": "mfd: sc27xx: Use MFD cells and devm_mfd_add_devices()", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/497909/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/2217444/comments/", "headers": { "Return-Path": "\n <linux-rtc+bounces-6263-incoming=patchwork.ozlabs.org@vger.kernel.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "linux-rtc@vger.kernel.org" ], "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n secure) header.d=abscue.de header.i=@abscue.de header.a=rsa-sha256\n header.s=dkim header.b=gIV0Iifz;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2600:3c0a:e001:db::12fc:5321; helo=sea.lore.kernel.org;\n envelope-from=linux-rtc+bounces-6263-incoming=patchwork.ozlabs.org@vger.kernel.org;\n receiver=patchwork.ozlabs.org)", "smtp.subspace.kernel.org;\n\tdkim=pass (2048-bit key) header.d=abscue.de header.i=@abscue.de\n header.b=\"gIV0Iifz\"", "smtp.subspace.kernel.org;\n arc=none smtp.client-ip=89.58.28.240", "smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=abscue.de", "smtp.subspace.kernel.org;\n spf=pass smtp.mailfrom=abscue.de" ], "Received": [ "from sea.lore.kernel.org (sea.lore.kernel.org\n [IPv6:2600:3c0a:e001:db::12fc:5321])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fk5Zw0Vfgz1xrn\n\tfor <incoming@patchwork.ozlabs.org>; Sun, 29 Mar 2026 18:28:24 +1100 (AEDT)", "from smtp.subspace.kernel.org (conduit.subspace.kernel.org\n [100.90.174.1])\n\tby sea.lore.kernel.org (Postfix) with ESMTP id B7679302AE2E\n\tfor <incoming@patchwork.ozlabs.org>; Sun, 29 Mar 2026 07:28:04 +0000 (UTC)", "from localhost.localdomain (localhost.localdomain [127.0.0.1])\n\tby smtp.subspace.kernel.org (Postfix) with ESMTP id 84BF92D77F5;\n\tSun, 29 Mar 2026 07:28:01 +0000 (UTC)", "from srv01.abscue.de (abscue.de [89.58.28.240])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby smtp.subspace.kernel.org (Postfix) with ESMTPS id 117FC175A6D;\n\tSun, 29 Mar 2026 07:27:57 +0000 (UTC)", "from fluffy-mammal.metal.fwg-cag.de (87-94-78-46.co.dnainternet.fi\n [87.94.78.46])\n\tby srv01.abscue.de (Postfix) with ESMTPSA id 85C511C0010;\n\tSun, 29 Mar 2026 09:27:48 +0200 (CEST)" ], "ARC-Seal": "i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116;\n\tt=1774769281; cv=none;\n b=lrP/1X1o7HGCM683dK43RnfJRoQZzdLB8dYK1uFDbCJTy8yscqUgi/JekB3bOJUN7HLMLAjEwyVp/bJM81xdG+O04o4Q0hTfh/0B8beuJyiSNWsmUx3tcZ2z6fp8ULdhkGjAJ8UYEd9i7BoPexgSfDBqiczUS/hXMzXdE/j6mnc=", "ARC-Message-Signature": "i=1; a=rsa-sha256; d=subspace.kernel.org;\n\ts=arc-20240116; t=1774769281; c=relaxed/simple;\n\tbh=TyOGRacq1l8VhjrXar+eTNydzKWkr7ceew9w7rmxyfM=;\n\th=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc;\n b=iceVTH4XBIjnwgTKFQwxXKMsP6o2ikdCPXQ1VA6C1YI82tP3xbXasVP049lQaC36USfOBGCa/mz1lOChPfjUK9Q+ybv2JDV6Y9b9GrZ5hlPGKtMq7/qW8ZEJ99RzFLGdA774zSLYFOgwPZ24/SLrbw3gXigoSSUDAYF9KBXRUpQ=", "ARC-Authentication-Results": "i=1; smtp.subspace.kernel.org;\n dmarc=none (p=none dis=none) header.from=abscue.de;\n spf=pass smtp.mailfrom=abscue.de;\n dkim=pass (2048-bit key) header.d=abscue.de header.i=@abscue.de\n header.b=gIV0Iifz; arc=none smtp.client-ip=89.58.28.240", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=abscue.de; s=dkim;\n\tt=1774769269;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\t to:to:cc:cc:mime-version:mime-version:content-type:content-type:\n\t content-transfer-encoding:content-transfer-encoding;\n\tbh=bwKg5f4/9X84xc/R7pQkuqBj+q/DTdDssJZt7wCnFjQ=;\n\tb=gIV0IifziZIlQJ0rYwDTlaqqhEm0A+ovDuqFYb/SHDiXwEyJfw/NeuYqe6zISgn3K6zMoo\n\tPmmZn2RnPDZm1ZuhWkWlaE41SVyF+LCa2/WEtH9E+M2TUJEbUsMyL0Ua1gqCAghHaqeTwA\n\tVsKh1zBZ+Z1ASeNv8qvk+tgNUFlpBQG7IeAP2K9j2y+LSXUJy9fsgDbHpIJpMmo/d6E13F\n\tT3bsOS8V24HnzKE2kde7zLx7AMUh1oWefDQ4AKtj8PPwOHY3m65R6prMWJRtrT4DQAxgdY\n\ta3CUnFTtQd1FhdJfdWjMXpr1dVBMDaCc73cwwBhZGn990dZzNsMbkLWCJQwXiA==", "From": "=?utf-8?q?Otto_Pfl=C3=BCger?= <otto.pflueger@abscue.de>", "Subject": "[PATCH v3 0/5] mfd: sc27xx: Use MFD cells and\n devm_mfd_add_devices()", "Date": "Sun, 29 Mar 2026 09:27:44 +0200", "Message-Id": "<20260329-sc27xx-mfd-cells-v3-0-9158dee41f74@abscue.de>", "Precedence": "bulk", "X-Mailing-List": "linux-rtc@vger.kernel.org", "List-Id": "<linux-rtc.vger.kernel.org>", "List-Subscribe": "<mailto:linux-rtc+subscribe@vger.kernel.org>", "List-Unsubscribe": "<mailto:linux-rtc+unsubscribe@vger.kernel.org>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "8bit", "X-B4-Tracking": "v=1; b=H4sIAHDUyGkC/23NQQrCMBCF4auUrI2k0zahrryHuJgkExuorSQaK\n qV3Ny2IiC7/B/PNzCIFT5EdipkFSj76cchR7QpmOhwuxL3NzUCAFAAljwbUNPGrs9xQ30duUat\n WNK5CJJbPboGcnzbydM7d+Xgfw3P7kMp1fWPwi6WSCy7bBqQjVRuljqijedDeEluxBB+gguYPA\n BmwgrSWArHGL2BZlhczVPyx8QAAAA==", "X-Change-ID": "20260221-sc27xx-mfd-cells-dab7905f3aae", "To": "Alexandre Belloni <alexandre.belloni@bootlin.com>,\n Rob Herring <robh@kernel.org>, Krzysztof Kozlowski <krzk+dt@kernel.org>,\n Conor Dooley <conor+dt@kernel.org>, Orson Zhai <orsonzhai@gmail.com>,\n Baolin Wang <baolin.wang@linux.alibaba.com>,\n Chunyan Zhang <zhang.lyra@gmail.com>, Lee Jones <lee@kernel.org>,\n Pavel Machek <pavel@kernel.org>, Liam Girdwood <lgirdwood@gmail.com>,\n Mark Brown <broonie@kernel.org>, Sebastian Reichel <sre@kernel.org>", "Cc": "linux-rtc@vger.kernel.org, devicetree@vger.kernel.org,\n linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org,\n linux-pm@vger.kernel.org,\n =?utf-8?q?Otto_Pfl=C3=BCger?= <otto.pflueger@abscue.de>,\n Sebastian Reichel <sebastian.reichel@collabora.com>", "X-Mailer": "b4 0.14.3" }, "content": "These changes resulted from the need to decouple the the Linux device\ndriver hierarchy from the device tree bindings for two different series\nintroducing regulator [1] and poweroff [2] support for the SC2730 PMIC.\n\nThere are different PMICs in the SC27xx series, including SC2730 and\nSC2731. These have a lot of similarities, but some differences too. For\ninstance, they contain compatible RTC blocks, but completely different\nsets of regulators.\n\nOn the Linux side, each PMIC block needs its own driver. The MFD driver\ncurrently uses devm_of_platform_populate() to load the drivers for the\ncomponents of the PMIC, which only works when each component has its own\nsub-node with a \"compatible\" property that is used to select a driver\nfor the device.\n\nWhen viewed from the device tree side, the parent node representing the\nPMIC already contains a \"compatible\" property that distinguishes the\ndifferent PMICs. While the device tree bindings currently do require a\nseparate \"compatible\" property for each sub-node (ADC, fuel gauge,\nregulators, ...), this is essentially redundant since the node name and\nthe parent compatible uniquely identify the component. Moreover, some\nparts of the PMIC such as the poweroff/reboot controller do not even\nneed a corresponding device tree node.\n\nChange the MFD driver to use MFD cells instead, which allows it to\ninstantiate sub-devices both with and without device tree nodes.\nDevices that do not have a separate device tree node with its own\n\"compatible\" property can be matched by their platform device ID.\nUse this to hook up the existing SC2731 poweroff and regulator drivers,\nwhich were previously not loaded at all due to the lack of an ID table.\n\nIn the device tree bindings, deprecate the redundant \"compatible\"\nproperty for the \"regulators\" node. While it might make sense to do this\nfor the other components too, there are a few reasons to only change the\nregulators at this point:\n - The regulators node is special since it is not as independent as the\n other components. For instance, it is the only child node of the PMIC\n that does not have a \"reg\" property. The set of regulators also\n differs much more between different PMIC models than the register\n layout of the other components.\n - We already have some other PMICs where only the regulators are\n treated specially like this, such as MediaTek MT6359 and MT6370.\n - It was suggested to remove the \"compatible\" property for the new\n SC2730 regulator bindings I am preparing in [2]. The bindings for\n the other components do not need any significant changes at the\n moment.\n - Unlike the poweroff and regulator components, the other parts are\n already working with the existing drivers and bindings.\n\nFor the other components that still have a \"compatible\" property used\nfor matching MFD cells, ensure that an SC2730-specific compatible is\ndefined in the bindings so that it can be listed in the SC2730-specific\ndevice table in the MFD driver.\n\nSigned-off-by: Otto Pflüger <otto.pflueger@abscue.de>\n\n[1]: https://lore.kernel.org/all/20250926-sc2730-reboot-v1-0-62ebfd3d31bb@abscue.de/\n[2]: https://lore.kernel.org/all/20260220-sc2730-regulators-v1-0-3f2bbc9ecf14@abscue.de/\n\n---\nChanges in v3:\n- Fixed warning about pointer-to-integer cast by using uintptr_t.\n- Changed device ID enum to start with 1 so that 0 is invalid.\n- Link to v2: https://lore.kernel.org/r/20260325-sc27xx-mfd-cells-v2-0-d0ebb60aa4a7@abscue.de\n\nChanges in v2:\n- Changed PMIC type matching in MFD driver to use an identifier like\n other drivers instead of passing pointers through of_device_id.\n- Rebased on next-20260324.\n- Link to v1: https://lore.kernel.org/r/20260222-sc27xx-mfd-cells-v1-0-69526fe74c77@abscue.de\n\n---\nOtto Pflüger (5):\n dt-bindings: rtc: sc2731: Add compatible for SC2730\n regulator: dt-bindings: sc2731: Deprecate compatible property\n mfd: sprd-sc27xx: Switch to devm_mfd_add_devices()\n power: reset: sc27xx: Add platform_device_id table\n regulator: sc2731: Add platform_device_id table\n\n .../devicetree/bindings/mfd/sprd,sc2731.yaml | 2 -\n .../bindings/regulator/sprd,sc2731-regulator.yaml | 4 +-\n .../devicetree/bindings/rtc/sprd,sc2731-rtc.yaml | 7 ++-\n drivers/mfd/sprd-sc27xx-spi.c | 62 ++++++++++++++++++----\n drivers/power/reset/sc27xx-poweroff.c | 8 +++\n drivers/regulator/sc2731-regulator.c | 10 +++-\n 6 files changed, 77 insertions(+), 16 deletions(-)\n---\nbase-commit: 85964cdcad0fac9a0eb7b87a0f9d88cc074b854c\nchange-id: 20260221-sc27xx-mfd-cells-dab7905f3aae\n\nBest regards," }