From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f43.google.com (mail-wm1-f43.google.com [209.85.128.43]) (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 5DBA72F39CF for ; Thu, 27 Nov 2025 17:53:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.43 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764266024; cv=none; b=Zk3xNadgsBum9n/po4Ac/xx8eZSgVVso5hgJKmSr3GL+LkivgmUT4axOY230xSJ3TPFSXtyiPrdAcshCichQ63GHmDT47gw70lmyOvu49nI+2+KlyeCFUKHLlNqP6WWpkymIpbQccW17H1fEcd/wKTd6dXGOY+tn3aEE6lZJDmQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764266024; c=relaxed/simple; bh=nEj2rUx8G9gla85oszVk7WefBks8vjxYZLcGA4NKCZc=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=MsgpEucd5k495MlMdraYUJ1pdjDD9ngUXTMALlBsUvjNQ5pBWDOwttuDhhbJhEg+ZCEmm68QK93W4d3mOi3tti5y7H5qGIl++g4Um2rZx0nhr2i3Wc1dkF7sT+GDoT50ZRhamBO8lyBuZWWv3I20R2RYkse/1tlvuhpOJKsCW3s= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=cXTvrBSL; arc=none smtp.client-ip=209.85.128.43 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="cXTvrBSL" Received: by mail-wm1-f43.google.com with SMTP id 5b1f17b1804b1-4779cb0a33fso10212485e9.0 for ; Thu, 27 Nov 2025 09:53:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1764266021; x=1764870821; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=ETweoxGWZm2ziwTrSjwftUK80iZE4y2BuRacSXs/oVk=; b=cXTvrBSLRu//G1uEOfJgmZMZuu1swGKTUfHUrYQIH5KALFTMeO1dJ6OlO+mxtcutZA Xw0J9FxnkFtJbbEwzXurObhNlfrGQFPmTGqiwcU3DbLp6wYSakVij065Iv1upwP606Ry Z00jNN0GebI8uu8WEhvn69S1x0VFxgEtEvODkii/dIK5zn1L9QmNForyLi40tRO8uriN zhFmjuu0V/cMkX507V/O+Mc8wxqCeFb1ds6XHnB4kpvflP7F/jgUDMXR/Mmx54Da74Ll mM21Svm8U3YWoVyFZQSaurmG6rXLL9RRWolCWpXcpNooFIC53JBpiCBLPCXb6NJVG9Im cAxQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764266021; x=1764870821; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language: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=ETweoxGWZm2ziwTrSjwftUK80iZE4y2BuRacSXs/oVk=; b=riHw4yEN3rdo14XFZH42Sq2eaHiOE1L794/FS8aYzdQ7MzJaVta1E4LHflHN0aG+zm ya8jbXlt2bnka1/NQBB+w75ltmCNtRQa8okzoPQjW7kK60udWUYDy85g6AOh/cv48P4E L4lh2v0PJ5qRML0MFay/8s9EhNNczSuPADSqpRwG41zERtOiKdaX7KO0Kd6mbjVVX5C9 QoWVqRB45pIy8A+FXE36T+J2AE8JL3PNbVH/cFUEVcDW+KgDup0BYR56lvvB6eFsXPoh wsQMiYIy7ExIWZa2w96d65iszimzMWb+AVRq5OgLEcjpp8zFGLPr2wVyXwYFG9tO6lnm elLQ== X-Gm-Message-State: AOJu0YxcSxZaAWU/aqasVSI61P7ke7BHKHZPazjFXZzR6uzZx8SUEHhc JVnYsJx1aRVa5QEp1T3ju27cjKFiteyJFFitfmEAqZzjwdjIJPZvcPYqAm2nv8NnU0k= X-Gm-Gg: ASbGncu7Gspq2pRVjR8MxTCS+oBd9ukD0qlAiJjVXOytv0/g4u2/vofseki1Dnv6r1M CqQDMJynAEcPL8gO+aH/ehKDFPe0xSI+IgPGwVzFoiJgy3iOMY+OJ9l7CuiJAkSICqRaKQlNyNA CWxFmFL6DCHUs+JsfElJOYp9oSaitu2B483ZoRhaQUoWvwjueYwHQD8unIf4g6qo8r68zlaofgU v5OuCbVZfFqgfvh84BPecXzFtu7/Ai7MP2omh6JOh83bnOcD0S4lmo9ZNG77cX4uAwZ+8ge7wUe etlADngfOxypBopV3p0J2fyvXzNYIeNN0mD/Sx783FiU+QUa+2aJqdwGbxNu8hr2xjPG74ADRy4 Rg7YDb2kt8P7WEwlH/lCaDQezQEcTwc0WIlMmEh4sLpILUySgX2E5MEAUkkXlZD05u7XpHSwUhz eDy+q8AUxP6Fb9j55C4Luuq/f5ycA5PlAssSOSwiIuPMoYD5DxF69Q X-Google-Smtp-Source: AGHT+IGfQcg2rnX4fdDDkIwWfOlTxEg1XWFtGmpFpSHYpBftRve1U7d6ObiP5sKkjk0IWCH4pIYGKg== X-Received: by 2002:a05:600c:3595:b0:477:755b:5587 with SMTP id 5b1f17b1804b1-477c0184b34mr251138045e9.8.1764266020721; Thu, 27 Nov 2025 09:53:40 -0800 (PST) Received: from [192.168.1.36] (p549d4195.dip0.t-ipconnect.de. [84.157.65.149]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-479052b3f5fsm72574625e9.4.2025.11.27.09.53.39 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 27 Nov 2025 09:53:40 -0800 (PST) Message-ID: Date: Thu, 27 Nov 2025 18:53:38 +0100 Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC 3/8] arm64: dts: qcom: pmi8998: Add fuel gauge Content-Language: en-US To: david@ixit.cz, Sebastian Reichel , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Casey Connolly , Joel Selvaraj , Yassine Oudjana , Bjorn Andersson , Konrad Dybcio , Alexander Martinz , =?UTF-8?B?QmFybmFiw6FzIEN6w6ltw6Fu?= , Richard Acayan , Alexey Minnekhanov Cc: linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, phone-devel@vger.kernel.org References: <20251124-pmi8998_fuel_gauge-v1-0-dd3791f61478@ixit.cz> <20251124-pmi8998_fuel_gauge-v1-3-dd3791f61478@ixit.cz> From: Casey Connolly In-Reply-To: <20251124-pmi8998_fuel_gauge-v1-3-dd3791f61478@ixit.cz> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 24/11/2025 22:53, David Heidelberg via B4 Relay wrote: > From: Casey Connolly > > Introduce the fuel gauge node for pmi8998. > Signed-off-by: Casey Connolly > Signed-off-by: David Heidelberg > --- > arch/arm64/boot/dts/qcom/pmi8998.dtsi | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/pmi8998.dtsi b/arch/arm64/boot/dts/qcom/pmi8998.dtsi > index cd3f0790fd420..ab3bc66502657 100644 > --- a/arch/arm64/boot/dts/qcom/pmi8998.dtsi > +++ b/arch/arm64/boot/dts/qcom/pmi8998.dtsi > @@ -44,6 +44,17 @@ pmi8998_rradc: adc@4500 { > reg = <0x4500>; > #io-channel-cells = <1>; > }; > + > + pmi8998_fg: fuel-gauge@4000 { > + compatible = "qcom,pmi8998-fg"; > + reg = <0x4000>; > + > + interrupts = <0x2 0x40 0x3 IRQ_TYPE_EDGE_RISING>; > + interrupt-names = "soc-delta"; I think it makes sense to add all the interrupts here, even if thr driver only uses one of them currently. interrupts = <0x2 0x40 0x0 IRQ_TYPE_EDGE_BOTH>, <0x2 0x40 0x1 IRQ_TYPE_EDGE_BOTH>, <0x2 0x40 0x2 IRQ_TYPE_EDGE_RISING>, <0x2 0x40 0x3 IRQ_TYPE_EDGE_RISING>, <0x2 0x40 0x4 IRQ_TYPE_EDGE_BOTH>, <0x2 0x40 0x5 IRQ_TYPE_EDGE_RISING>, <0x2 0x40 0x6 IRQ_TYPE_EDGE_BOTH>, <0x2 0x40 0x7 IRQ_TYPE_EDGE_BOTH>; interrupt-names = "soc-update", "soc-ready", "bsoc-delta", "msoc-delta", "msoc-low", "msoc-empty", "msoc-high", "msoc-full"; https://github.com/LineageOS/android_kernel_oneplus_sdm845/blob/lineage-22.2/arch/arm64/boot/dts/qcom/pmi8998.dtsi#L292 Not sure what the difference is between bsoc-delta and msoc-delta, maybe Richard or Yassine can recall? DT bindings would need updating too. > + > + status = "disabled"; > + }; > + > }; > > pmi8998_lsid1: pmic@3 { > -- // Casey (she/her)