From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f51.google.com (mail-wr1-f51.google.com [209.85.221.51]) (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 BDC3F1E1C16 for ; Tue, 7 Jan 2025 09:06:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736240816; cv=none; b=WZIiJ7XxpO+BoRfNai+na7TEjXjBmGH/o6zO+9vTdSVB8jjH52JyamYL4qytyfqEwtxZoEnyhi3xSHEQbO2XpqJM/7ehP/sQ47ummyOI7/oS3QKbR0xOMLMHO3xRCRUTfGlL2097HmcmgNxk5oymf774z+TdKDFwkJJpEhDW1cs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736240816; c=relaxed/simple; bh=LvDd8U5d1JAExU6nJWpv5tD30jOU1E0HMWgRrqOF98o=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=RLPDvsF8CmhET++TJUkzMudY9q0qEORLfOvruHLlPChBikaCuruKt2XhNqiyJd3cgHlGEl7NVAUOgBZG7t40iK5jI3CGff1iAa52Wx9IEh6n3GUkJBfc8iiHgVvw/bM7p9HxACI2uIaYT7ChtaDfMU1VAsPyqubQg7YxvBT0LHs= 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=SfAdx9pD; arc=none smtp.client-ip=209.85.221.51 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="SfAdx9pD" Received: by mail-wr1-f51.google.com with SMTP id ffacd0b85a97d-385eed29d17so7239450f8f.0 for ; Tue, 07 Jan 2025 01:06:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1736240810; x=1736845610; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=A4jDBOd4uA6iWq10QaHFHWmxRK0/Onl9k6WANOitcUI=; b=SfAdx9pD8u00Jk9Zxg0HjjKtk1PVJmVuNIy9xCxr+lvGr1MG556UlGirUbNwzin2fM B5W9X7aSnTOHrW8TNLLqMjEAGtpkxIMPrFtw8ZPHmGK0UONtJ5a2vIMgtg0+EaGp5kav Ig0vwgueYHJeWZsDXVngEJXFKhSiAnZ+t+NwYmhTq1J/Sl01WQDkO67RunPulI8W7AH9 pE5K+IxSLUFtn2kaKDhnggjHeSqmgKATS8F7RAV42vQvgXmlvLUqV43qGko9Bfo8ltdO 5kx/qTdsKN6PzJhJfnKV8FaQC4bopzDcLPpLLhkATo5R72zc0t2bY5otgiMnJqVJPnGi Svgw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736240810; x=1736845610; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=A4jDBOd4uA6iWq10QaHFHWmxRK0/Onl9k6WANOitcUI=; b=TF696rRN7TU2ZIIRaU9c2HlYRMDCpmJ3imDtnn4H+KDJMRTXAj/C6pVqKQBbpszsCt W/6GxAxNgfYlfFij+jJ394HssiPOb6Os1m2lXE8uIFd0oBV8lKi0hGGq1MiMSoUmd+D9 zJ1qHkQWcqdXUa3yFKdB5QET0cqDkcc/44l6a2ZKbe0Pyy+JSpBFxUKbAvPKGZxzX/Jh SCI6w5qdz12ODc9mfUv24UbvF43Z871/wBkC5bN8Ip6PU8IUIhgkvf5rq9e0h/Tx8e5g MYNTf419NN+Rnag0yJmvge/Fpgnh4n3WGDNAz2Eiu+B6jBtUU+G3vb/HGSe2na7tKiac Xs/A== X-Forwarded-Encrypted: i=1; AJvYcCUYYA5Msmm1EQMEE2Uo7y30CNAa3gXmqIa9PZptaG2J2l3ab2OEhsP5Rb6OVdZcJ0Af0WBHwjgdu/vELzk=@vger.kernel.org X-Gm-Message-State: AOJu0YyVW/o74vffpwIA6ZZa6f7rve2NF8qm1Ypb0FvJHlEIjC7/0btH EFUUZoLXVXJG6g6U2vhMMGA6AQYvfsVpxEjnsa70CS18waU6flEVi8/3Qye9dh8= X-Gm-Gg: ASbGncvsXmlvx2CceQ/B3i8aN7DYhFO3k1Bu7bjDwbK5a6lGG0cF0fR+8IUo+GTeCjO UXD5AOotv08tOyTgydH8p7H1g7WYereX5VuAV+tqp90pcos1tXK+2iEXeGlrkQ4fCKXujoRu7RQ Ji3mLXDqKGijJ+sqZGXugNCyI7rtGbKqPXUVcpxA9/ls6IqKyEvfUJ06LAwLShoW/mUINMod8AB N0kByaGdK3uWCFNYAlSo3rTUCsecOOmuNpai73B+lxfXyjJdrNAVOLurPvKhw== X-Google-Smtp-Source: AGHT+IHZAZxqzkt43hjIPdmiX16LWtAmIlQjSGTOiM5YyAVZqEoQExXofh761RH379rxPGTtkqw/RA== X-Received: by 2002:a05:6000:480a:b0:386:1cd3:89fa with SMTP id ffacd0b85a97d-38a221fa8c6mr57479039f8f.33.1736240809665; Tue, 07 Jan 2025 01:06:49 -0800 (PST) Received: from [10.1.1.109] ([80.111.64.44]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43656b013e1sm619089305e9.12.2025.01.07.01.06.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Jan 2025 01:06:49 -0800 (PST) Message-ID: <74430be84cb49dc0a7413656a2923facc021cd15.camel@linaro.org> Subject: Re: [PATCH v2 4/4] arm64: dts: exynos: gs101-oriole: enable Maxim max77759 fuel gauge From: =?ISO-8859-1?Q?Andr=E9?= Draszik To: t.antoine@uclouvain.be, Sebastian Reichel , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Dimitri Fedrau , Catalin Marinas , Will Deacon , Peter Griffin , Alim Akhtar Cc: linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org Date: Tue, 07 Jan 2025 09:06:48 +0000 In-Reply-To: <20250102-b4-gs101_max77759_fg-v2-4-87959abeb7ff@uclouvain.be> References: <20250102-b4-gs101_max77759_fg-v2-0-87959abeb7ff@uclouvain.be> <20250102-b4-gs101_max77759_fg-v2-4-87959abeb7ff@uclouvain.be> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.52.1-4 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Hi Thomas, Thanks for your patch! On Thu, 2025-01-02 at 12:15 +0100, Thomas Antoine via B4 Relay wrote: > From: Thomas Antoine >=20 > Add the node for the max77759 fuel gauge as a slave of the i2c. >=20 > The todo is still applicable given there are other slaves on the > bus (pca9468, other max77759 functions and the max20339 OVP). >=20 > The fuel gauge has been tested and seems to give coherent results. > Manual activation of the charger via i2cset shows that the sign of > the current does indicate charging/discharging status. >=20 > Signed-off-by: Thomas Antoine > --- > =C2=A0arch/arm64/boot/dts/exynos/google/gs101-oriole.dts | 10 ++++++++++ > =C2=A01 file changed, 10 insertions(+) >=20 > diff --git a/arch/arm64/boot/dts/exynos/google/gs101-oriole.dts b/arch/ar= m64/boot/dts/exynos/google/gs101-oriole.dts > index 387fb779bd29ea3812331a7951f03b181c5fe659..6c83ee6f8a6b0327c576573d0= 3a8d2bcc93f9e16 100644 > --- a/arch/arm64/boot/dts/exynos/google/gs101-oriole.dts > +++ b/arch/arm64/boot/dts/exynos/google/gs101-oriole.dts > @@ -10,6 +10,7 @@ > =C2=A0 > =C2=A0#include > =C2=A0#include > +#include > =C2=A0#include "gs101-pinctrl.h" > =C2=A0#include "gs101.dtsi" > =C2=A0 > @@ -90,6 +91,15 @@ eeprom: eeprom@50 { > =C2=A0&hsi2c_12 { > =C2=A0 status =3D "okay"; > =C2=A0 /* TODO: add the devices once drivers exist */ > + > + fuel-gauge@36 { > + compatible =3D "maxim,max77759-fg"; > + reg =3D <0x36>; > + reg-names =3D "m5"; > + shunt-resistor-micro-ohms =3D <5000>; > + interrupt-parent =3D <&gpa9>; > + interrupts =3D <3 IRQ_TYPE_LEVEL_LOW>; > + }; The order of properties within a node should follow Documentation/devicetree/bindings/dts-coding-style.rst In particular shunt-resistor-micro-ohms should come last in this case. Cheers, Andre'