From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (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 D61D722F177 for ; Thu, 27 Feb 2025 13:13:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740662040; cv=none; b=j0J5WgTzy34FvcUn5SqXRfyzAnF6isBQhtn12UvREMaPX0/2A/Emo9WurGdlGeSREB+KSPXCwP4+gKwJa/nqmANNx3M1v0lw0ckPfVPYnh4+QzH2C3TQg4caVkLGzw0E2WevJH6vi/dzFHiGUcP8Aomd15o4Pltv+zz1cTtLUgM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740662040; c=relaxed/simple; bh=RvYJJ5olkrH8g63o2yNdqaycpGOF7BuYhXeUvp7CudE=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=IsycFude76X0StaCmyRxGPRwaAtcWAPhaUHHojuoKRWnKRKaO7PMOhobZzWtXkZqw+13WirvwsTq+5vDmpzQ8CUKhRUYB/fIeiZdkheZBIMUPVjsachqJJqNzbAG3hw22Gr/IZZsEH25CG1Jz7olfOupdnZqFrFR82VjhSKXVXw= 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=w6ULe8Xo; arc=none smtp.client-ip=209.85.128.46 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="w6ULe8Xo" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-4397e5d5d99so5989855e9.1 for ; Thu, 27 Feb 2025 05:13:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1740662036; x=1741266836; 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=RvYJJ5olkrH8g63o2yNdqaycpGOF7BuYhXeUvp7CudE=; b=w6ULe8XoIKz5NaxL+udSppHvVuC25aHUNbRUAONrrDwtNtMdhqhYg00HhkqOFSr6sw l6DIIzGmmRKVzTC+3uZK+x43K80gfPfJBPpJzddK6vJIjt1deSeKiYMoKToFkmb8+Dt7 wV6kuIrYvWTKPs3ofiOFWq+KiXn12Go+0BJWtJKZxjvjwKEFWu/RB5HknYEZCdJKIEp5 ItkEKYP2fy+ys4nweHLm4J8bg9n205EUY7mEwat69NESmQoXvqUNQhglfIcP88WKygLE DFJQSFFakdYokr9m/XLAvp1D6ip3zKttUdu3D5oOT+Ezha3JFJ8AMlNx5x68vUG/9el+ VMOg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740662036; x=1741266836; 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=RvYJJ5olkrH8g63o2yNdqaycpGOF7BuYhXeUvp7CudE=; b=I/ZIvIkQdZdxoNvypSR3AvOf3mnyFX2vtZnsIuFeCihdo0w2x+OgXN5t9C3JFhIZRW V3XB8q7BkpOFZ+lByDBeZ/HsaJWZumkJAlZUkAgujI2bSvM6wIP+FmMPtrlsaIeayEEI DcoKxCv6oUSrejk5Qp+eq3zaoIvrupIYg9ylIYgOiIe2e+OzyEKe8rMBaDK0pM/Y9nc8 8BEYOlNbVY4cKW/5BGESQ/2UDlo9/MurcV/3a4fOeOls6hh+Zqq7aR/5hpLbTzjg1OJz /g5HQq5fJzoiuCOB3psa0eIja/zBd7XZiJc1Rk3s9GvK6PQRR30qYjEJECaUeDhSEz/j QxNw== X-Forwarded-Encrypted: i=1; AJvYcCXH+UwnKKyvFgPX8LdcZEN/SgebhUTiQKWkkdAahQUsDBDnC1nSka/SW2ERxsLXSNiLEhrWJZsM+VaUujs=@vger.kernel.org X-Gm-Message-State: AOJu0YzeoZ4hQevdhZwOK8xV9FHyicPrMbqZ+KA/hg67JRZOktDpOB5m h3QCJGiX/xUZxbDm3Xv20oOsDwiJX4i9b6MjFXHbWTqLLhIlNwQkB60T6vCKgLA= X-Gm-Gg: ASbGnct2icMr+WMOhwOJE2RNvm0P+SwV66FjGM/cNQpYVwvg1FgkfQGxVtBbBNDUzI7 g1uLjlNqRZoOu4/MW3VlCtCNbGZNDRfDAZRMpL4trYTyPFtPwg/HvO1LHWe4xaD3Oco0SO8EDI2 XOBtaC4SaLvC7fvIhA4fKzRDtg5CG6JpScF5qR9auvfCVRLTbwbSZUN+dStLsw3Wxz3h8JXQwHY bU7jFXG4jDXfdd+deVdPSy1Nqaz6nQlpavI3itrECGF5ljnoCjx6Yf/D7SzLQHIdY081r6dK5gc 9tXNGPH2C8jJEh6YqHggOWPWJymjOQ== X-Google-Smtp-Source: AGHT+IE6n/c7/bx5gCSXDTH8aoqU0z7nzFeeaoRQoyzForW+cjpxgZs3kJk+NvWIxEA4pl6DiSKyiA== X-Received: by 2002:a05:6000:1864:b0:390:dbf5:407b with SMTP id ffacd0b85a97d-390dbf541d3mr5027668f8f.28.1740662036128; Thu, 27 Feb 2025 05:13:56 -0800 (PST) Received: from draszik.lan ([80.111.64.44]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-390e4847e62sm2021619f8f.67.2025.02.27.05.13.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 27 Feb 2025 05:13:55 -0800 (PST) Message-ID: <503e105b71fa4271f40a2d3ca18ba13ed7d45a65.camel@linaro.org> Subject: Re: [PATCH v2 3/6] dt-bindings: mfd: add max77759 binding From: =?ISO-8859-1?Q?Andr=E9?= Draszik To: Rob Herring Cc: Lee Jones , Krzysztof Kozlowski , Conor Dooley , Linus Walleij , Bartosz Golaszewski , Srinivas Kandagatla , Kees Cook , "Gustavo A. R. Silva" , Peter Griffin , Tudor Ambarus , Will McVicker , kernel-team@android.com, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-gpio@vger.kernel.org, linux-hardening@vger.kernel.org Date: Thu, 27 Feb 2025 13:13:54 +0000 In-Reply-To: <20250227130451.GA1783593-robh@kernel.org> References: <20250226-max77759-mfd-v2-0-a65ebe2bc0a9@linaro.org> <20250226-max77759-mfd-v2-3-a65ebe2bc0a9@linaro.org> <20250227130451.GA1783593-robh@kernel.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.53.2-1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 On Thu, 2025-02-27 at 07:04 -0600, Rob Herring wrote: > On Wed, Feb 26, 2025 at 05:51:22PM +0000, Andr=C3=A9 Draszik wrote: > > The Maxim MAX77759 is a companion PMIC for USB Type-C applications and > > includes Battery Charger, Fuel Gauge, temperature sensors, USB Type-C > > Port Controller (TCPC), NVMEM, and a GPIO expander. > >=20 > > This describes the top-level device. > >=20 > > Signed-off-by: Andr=C3=A9 Draszik > >=20 > > --- > > v2: > > * rename expected nvmem subdev nodename to 'nvmem-0' > > =C2=A0 I'd have preferred just 'nvmem', but that matches nvmem-consumer= .yaml > > =C2=A0 and fails validation. > >=20 > > Note: MAINTAINERS doesn't need updating, the binding update for the > > first leaf device (gpio) adds a wildcard matching all max77759 bindings > > --- > > =C2=A0.../devicetree/bindings/mfd/maxim,max77759.yaml=C2=A0=C2=A0=C2=A0= | 104 +++++++++++++++++++++ > > =C2=A01 file changed, 104 insertions(+) > >=20 > > diff --git a/Documentation/devicetree/bindings/mfd/maxim,max77759.yaml = b/Documentation/devicetree/bindings/mfd/maxim,max77759.yaml > > new file mode 100644 > > index 0000000000000000000000000000000000000000..87e3737896a289998a18b67= 932dbccacfb8e3150 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/mfd/maxim,max77759.yaml > > @@ -0,0 +1,104 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/mfd/maxim,max77759.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Maxim Integrated MAX77759 PMIC for USB Type-C applications > > + > > +maintainers: > > +=C2=A0 - Andr=C3=A9 Draszik > > + > > +description: | > > +=C2=A0 This is a part of device tree bindings for the MAX77759 compani= on Power > > +=C2=A0 Management IC for USB Type-C applications. > > + > > +=C2=A0 The MAX77759 includes Battery Charger, Fuel Gauge, temperature = sensors, USB > > +=C2=A0 Type-C Port Controller (TCPC), NVMEM, and a GPIO expander. > > + > > +properties: > > +=C2=A0 compatible: > > +=C2=A0=C2=A0=C2=A0 const: maxim,max77759 > > + > > +=C2=A0 interrupts: > > +=C2=A0=C2=A0=C2=A0 maxItems: 1 > > + > > +=C2=A0 interrupt-controller: true > > + > > +=C2=A0 "#interrupt-cells": > > +=C2=A0=C2=A0=C2=A0 const: 2 > > + > > +=C2=A0 gpio-controller: true > > + > > +=C2=A0 "#gpio-cells": > > +=C2=A0=C2=A0=C2=A0 const: 2 >=20 > Why do you have GPIO properties here and in the child node? Either would= =20 > be valid, but both probably not. Putting them here is actually=20 > preferred. That's an oversight, I meant to put them into the child only, not here, since the child is the one providing the gpio functionality. What's the reason to have it preferred inside this parent node? At least some bindings do specify it in the child node, e.g.: delta,tn48m-gpio.yaml kontron,sl28cpld-gpio.yaml xylon,logicvc-gpio.yaml Cheers, Andre'