From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f53.google.com (mail-ej1-f53.google.com [209.85.218.53]) (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 4C0E333D4F2 for ; Fri, 9 Jan 2026 08:26:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767947190; cv=none; b=CYSw3Kyx7ye3Dok4Wezo/PHA+NpQUPfwfSRY2SYcme8NhC+ZOIIovLDPtbUDN/1T3LyB3EZtfXe2U0nkdZFqJw3lk5UYtqU565coV33SiScCOwofKrDF5Z/gT8d2NqDHjWNdCssQMFWmd/kE36/6tQLqKCvcu2FD5Ys6RMcZOpw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767947190; c=relaxed/simple; bh=TegNChNmHEzD8+9WeNnVdBonXBPxxrbHlIIxhxeFES4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=RalB3E7vKq4E60M76rfyAKjnbf7O1t4gH4+oZ4QXX+0Scpew5Py6Th1BACcTn0UZhNasnccMwxj9fVi0Y2cafzPQkYR/j8zeRr/B5uTfKYUTf1OGxdsxCbfGbkBO/T9n0McSoF2nlnQs4CraYcywRgu56mnSQBcktM4JgVmyb6U= 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=dmbj+sQX; arc=none smtp.client-ip=209.85.218.53 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="dmbj+sQX" Received: by mail-ej1-f53.google.com with SMTP id a640c23a62f3a-b736ffc531fso764056766b.1 for ; Fri, 09 Jan 2026 00:26:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1767947181; x=1768551981; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=VpBRuHMiMCleFgRr0io9MPn59FZX7HGviJTaLToKJC0=; b=dmbj+sQX6Npb+NaVsdExvnuWOCZMtcMrRpFdoV+hpQYMyRJUhdyvZXpa3htTf+w0LY sGQfPddcaXXL8nZONwxxvJhhDNA0pQnRNKgE5UL5GGuzggQ181FJhalF2Kfd0uLYswCJ i67qosMgiz+SheSqjdvQb6SYLMGumFy7BOVfL3NPNiKVcgKDEK5X7i1yo4BHZFOcfua5 JZ0bFookxp52Y7DQBbfxgbLn8604BNcHaJNCm7luc8XxArWikZjJckLO0DudgeoXzGDj UqfjKmOw6/r8U4HfhgogMGlcZZcstrA8P0Ur73Vvsur8LRuCeRuhZeaSsnzuqYBKq4sV 8GQA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1767947181; x=1768551981; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=VpBRuHMiMCleFgRr0io9MPn59FZX7HGviJTaLToKJC0=; b=BYdICWNFEcnxZvePeF/vvE+AMGRhx3wR/Vyni0YHC9CCIYDvgOJhrelULxnyl/JbTc u8O+ddyReWJsxRAJ4YfKMsaq82aReL7NYEthgvhC1IdLq2tOHyz+gunjB+hzKCIdkkoB 7qyDVmGXwxaBSG/iaEW7DxrUUs8VpnODvFlBFU/zfDmBronU+X6xiMvXmVxAK2IYRp63 dNymPWVMfpEuKHudDYNKpXVoPnH/3HTSHTcFVy00aRifEMy1eWXViZ6ov9+wCj04ZMyL TK+CnYGXHeGTYwFNw74rjUvKD7SUtTJ+urbMw1B6FWDI5jFkOeSQs/Pz0HrqKHI88KgP Kcrw== X-Forwarded-Encrypted: i=1; AJvYcCXGnoZgsKgTxynn9ozI8uHpH5cDceXVtvyWCQhhD/S8fm+LYPWtikx1kQNP/hLHdpldCzcv5dWWTGk=@vger.kernel.org X-Gm-Message-State: AOJu0YyMubDLXTcXfssEoxauMNgjr/pFgnxsE8p1uGnHEzgruJuW37NA VAdKNZMVBzFE2jwHhPEFKbYSNFzlwtMVMdno0B5RVa2va5liF0Kt45MIC6HQ8e351PM= X-Gm-Gg: AY/fxX5rvVWIRHzvCY3rFj8r9hcxEFhqBEXo4E7HVgv/Kxrg5kWyYIt9ZjmKDFpNIJb WIquoMDFQl/9Xe0wBNvmWbRJGFr2yTbwaPSQoRee1T+fSLlb46yFVdRAr/xGsklEoE4WxiHnKdT RUdl5hwstQDItcfESXqWRATdpQt1L7dkzoF+rSryFNDD++M/DuihNGBXk9PbL8+IcXXPFVzYAhs aAFJWuLU8MIDjlZCQpWbzDe+vsAkkKm9NPa7lMV3iWk2epRxRJQ9FNHvm6G2EskwHdUWS99phbi hBQjWFCJqzOs4HXLjhYqyYxwYR2X/1TIq+zpOWxLkoV+CHMG0ixr8lRw7CdrcQNnZ0YXkS8D2dB PwZmz49Td5tRSt6HlmoXpfkWXbx0w0Lx3jc4sKZfyxjTQBhxcf7lJZWXNKhiKWbQq0eukqZpqIC WPfnBEeTUleXuTA9Cw4Obi2SQsR0/RgcpYDn9FAjwUr0BAizUvTwv4JKAX+UEL4/QDtzW9 X-Google-Smtp-Source: AGHT+IEbfnpAwTBpMLXsSD7GLLq4RdVLr81UJhXWfoMR4g3wMxGRElDJAG31EIjk//ER3TZlJZdKiQ== X-Received: by 2002:a2e:b8d0:0:b0:382:ff8c:c9c3 with SMTP id 38308e7fff4ca-382ff8ccac1mr27076741fa.3.1767940183581; Thu, 08 Jan 2026 22:29:43 -0800 (PST) Received: from ?IPV6:2a10:a5c0:800d:dd00:8fdf:935a:2c85:d703? ([2a10:a5c0:800d:dd00:8fdf:935a:2c85:d703]) by smtp.gmail.com with ESMTPSA id 38308e7fff4ca-382eb7cc214sm21543771fa.22.2026.01.08.22.29.42 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 08 Jan 2026 22:29:42 -0800 (PST) Message-ID: <63bc889a-b97e-43c3-9f46-9ca444873b70@gmail.com> Date: Fri, 9 Jan 2026 08:29:41 +0200 Precedence: bulk X-Mailing-List: linux-clk@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RESEND v6 00/17] Support ROHM BD72720 PMIC To: Lee Jones Cc: Matti Vaittinen , Pavel Machek , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sebastian Reichel , Liam Girdwood , Mark Brown , Michael Turquette , Stephen Boyd , Linus Walleij , Bartosz Golaszewski , Alexandre Belloni , linux-leds@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-rtc@vger.kernel.org, Andreas Kemnade References: <20260108172735.GK302752@google.com> Content-Language: en-US, en-AU, en-GB, en-BW From: Matti Vaittinen In-Reply-To: <20260108172735.GK302752@google.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 08/01/2026 19:27, Lee Jones wrote: > On Mon, 15 Dec 2025, Matti Vaittinen wrote: > >> Resending the v6 >> >> Series is same as v6 _except_ being rebased on v6.19-rc1 - and adding rb >> tags which were replied to v6. >> >> The ROHM BD72720 is a new power management IC for portable, battery >> powered devices. It integrates 10 BUCKs and 11 LDOs, RTC, charger, LEDs, >> GPIOs and a clock gate. To me the BD72720 seems like a successor to the >> BD71828 and BD71815 PMICs. >> >> This series depends on >> 5bff79dad20a ("power: supply: Add bd718(15/28/78) charger driver") >> which is in power-supply tree, for-next. Thus, the series is based on >> it. >> >> The testing since v4 has suffered some hardware-issues after I >> accidentally enabled charging while the PMIC's battery pin was connected >> to the I/O domain. Some heat was generated, not terribly lot smoke >> though... >> >> After the incident I've had occasional I2C failures. I, however, suspect >> the root cause is HW damage in I/O lines. >> >> Revision history: >> v6 resend: >> - Rebased on v6.19-rc1 and collected rb-tags from v6. >> >> v5 => v6: >> - MFD fixes as suggested by Lee >> - Styling mostly >> - New patch to Fix comment style for MFD driver >> More accurate changelog in individual patches >> >> v4 => v5: >> - dt-binding fixes as discussed in v4 reviews. >> - Drop rohm,vdr-battery.yaml and add vdr properties to battery.yaml >> - Drop 'rohm,' -vendor-prefix from vdr properties >> - Link to v4: >> https://lore.kernel.org/all/cover.1763022807.git.mazziesaccount@gmail.com/ >> More accurate changelog in individual patches >> >> v3 => v4: >> - dt-binding fixes to the BD72720 MFD example and regulator bindings >> More accurate changelog in individual patches >> >> v2 => v3: >> - rebased to power-supply/for-next as dependencies are merged to there >> - plenty of dt-binding changes as suggested by reviewers >> - add new patch to better document existing 'trickle-charging' property >> More accurate changelog in individual patches >> >> RFCv1 => v2: >> - Drop RFC status >> - Use stacked regmaps to hide secondary map from the sub-drivers >> - Quite a few styling fixes and improvements as suggested by >> reviewers. More accurate changelog in individual patches. >> - Link to v1: >> https://lore.kernel.org/all/cover.1759824376.git.mazziesaccount@gmail.com/ >> >> --- >> >> Matti Vaittinen (17): >> dt-bindings: regulator: ROHM BD72720 >> dt-bindings: battery: Clarify trickle-charge >> dt-bindings: battery: Add trickle-charge upper limit >> dt-bindings: battery: Voltage drop properties >> dt-bindings: mfd: ROHM BD72720 >> dt-bindings: leds: bd72720: Add BD72720 >> mfd: rohm-bd71828: Use regmap_reg_range() >> mfd: rohm-bd71828: Use standard file header format >> mfd: rohm-bd71828: Support ROHM BD72720 >> regulator: bd71828: rename IC specific entities >> regulator: bd71828: Support ROHM BD72720 >> gpio: Support ROHM BD72720 gpios >> clk: clk-bd718x7: Support BD72720 clk gate >> rtc: bd70528: Support BD72720 rtc >> power: supply: bd71828: Support wider register addresses >> power: supply: bd71828-power: Support ROHM BD72720 >> MAINTAINERS: Add ROHM BD72720 PMIC >> >> .../bindings/leds/rohm,bd71828-leds.yaml | 7 +- >> .../bindings/mfd/rohm,bd72720-pmic.yaml | 339 ++++++ >> .../bindings/power/supply/battery.yaml | 33 +- >> .../regulator/rohm,bd72720-regulator.yaml | 148 +++ >> MAINTAINERS | 2 + >> drivers/clk/Kconfig | 4 +- >> drivers/clk/clk-bd718x7.c | 10 +- >> drivers/gpio/Kconfig | 9 + >> drivers/gpio/Makefile | 1 + >> drivers/gpio/gpio-bd72720.c | 281 +++++ >> drivers/mfd/Kconfig | 18 +- >> drivers/mfd/rohm-bd71828.c | 555 ++++++++- >> drivers/power/supply/bd71828-power.c | 160 ++- >> drivers/regulator/Kconfig | 8 +- >> drivers/regulator/bd71828-regulator.c | 1025 ++++++++++++++++- >> drivers/rtc/Kconfig | 3 +- >> drivers/rtc/rtc-bd70528.c | 21 +- >> include/linux/mfd/rohm-bd72720.h | 634 ++++++++++ >> include/linux/mfd/rohm-generic.h | 1 + >> 19 files changed, 3127 insertions(+), 132 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/mfd/rohm,bd72720-pmic.yaml >> create mode 100644 Documentation/devicetree/bindings/regulator/rohm,bd72720-regulator.yaml >> create mode 100644 drivers/gpio/gpio-bd72720.c >> create mode 100644 include/linux/mfd/rohm-bd72720.h > > The MFD parts LGTM. Thanks Lee! > What Acks are you waiting on? What's the merge strategy? I think everything else has been acked by maintainers, except the power-supply parts. I think those have only been looked at by Andreas and Linus W. Haven't heard anything from Sebastian :( I would love to see the patches 1 - 14 and 17 to be merged (via MFD?). I could then re-spin the 15 and 16 to limited audience as I hope Sebastian had time to take a look at them. However, I don't think any of the other patches in the series depend on the last . Yours, -- Matti. --- Matti Vaittinen Linux kernel developer at ROHM Semiconductors Oulu Finland ~~ When things go utterly wrong vim users can always type :help! ~~