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 E75F33AC0CD for ; Mon, 4 May 2026 13:22: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=1777900974; cv=none; b=Qgpd6PX3LPmVy3XOjTEJ/XS/jVjNXyvweDCWT1MBAoNl64g6/olJi0OgQiJjGVs6CpJbfHt8SnM2nQhaKAO8EXxy/gLnVDb4Gv22K83ZpP2isBv1Hv+GYQFEJ64s9PH5s10sA8hUuBTTm/zoGPmhWq0LM8X1ewxErxt785p1YAQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777900974; c=relaxed/simple; bh=hPP1W5oM+PMZffUDhm7eGNzZLrmVscrBig8GQKpA9pQ=; h=Message-ID:Date:MIME-Version:From:Subject:To:Cc:References: In-Reply-To:Content-Type; b=YJlWuz7u3OS5SOnLehYeFMFNIKNII9XMV93zbxrYiLzrKRcmN27MNWx8TyWtrbgAFdr3XVnRcIl9+ZRV0HjhWvjzGwNGXVlFg29wR/uDmvz9wDKi80XJRRnaejohMwVQ5sSivDUwl0nWT0HAwdJ+UfVGKckF8kFUkoqymR07N0k= 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=P8M82mnW; 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="P8M82mnW" Received: by mail-wr1-f51.google.com with SMTP id ffacd0b85a97d-445795cf6f1so2404850f8f.1 for ; Mon, 04 May 2026 06:22:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1777900970; x=1778505770; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:organization:autocrypt :content-language:references:cc:to:subject:reply-to:from:user-agent :mime-version:date:message-id:from:to:cc:subject:date:message-id :reply-to; bh=nK3BUe90hxbNx22uyW4aJ4Z5JKnLCY2TLAKp1mVu7aA=; b=P8M82mnWRjQ4KsaLJGn59H3R7oJL4XweJWj5QO+8V+ngq95ac843BGwxZVbnHoSzIF jo9lqyXMIbs/tLdXE92mAL3hDetkvTwfk1vya868/N18ktEZVCzJb9OL0ak6jIFpTRwT vpWqQaA9L9TIPCVXo57yNwFgXUgrwUI1cDXDMWi9B7S9O2UCA74z4zRi8aUBTLn8th/b dp6y4W8sydPuDaWrkgTX1PI8+IKc6kt3GzeP/ou2RtXEHdwUd2LIPIpREbOSbieGtNqI zihy8keo1oAhxYI4LG08I0NYMGPSwIFvuEwinzpsmvrjjkCedN2nIjvq/4x5pd801uKX SGyw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777900970; x=1778505770; h=content-transfer-encoding:in-reply-to:organization:autocrypt :content-language:references:cc:to:subject:reply-to:from:user-agent :mime-version:date:message-id:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=nK3BUe90hxbNx22uyW4aJ4Z5JKnLCY2TLAKp1mVu7aA=; b=WLjjD4gmXm8my/dNKsf+xStURd9M4H292qPXxKXS12E8oECUZaiFRlXmMnSOZlA9fE +j+5Mgx2gMV7wYPB3bg6NemSoRG+bnbheH/susDQbIMmlMWp8Eyhfe7BYwzDydhRB06p D5VyGXDvN1QwxjxUY3Qo+vPM5zododH7JhhEBYCE21+vTszwpZBRe45U8UhTWceuFRjH UWh7JAMbCBAu/6oh9mU7MgkSvzUyArCD9k3qaqlao6EnS88YutrgsVVnkfDeqMqKS096 XllqKgAvasiZLxs3HpiCiWBRwFAx7i/R7MnGgh3DABATVODkkgllZy+nX2r3x7FBBLAP MLHQ== X-Gm-Message-State: AOJu0YxByhJwDLl4/2n5jNgwo/yMB3wdDWw3VeQRkNUIcpN47UemOt6S zcSM5zEBc3za9xxxOjt5aDSPhlIvmnD2JVAKYNcrKc93uwmi2u1BnKQqnPtQZmvXEws= X-Gm-Gg: AeBDiev+np4Wd0wioc7YQQLwa8X04VtYQyv6M5CevsgAm4mPWOlX1eOsTHR9KJHxjA6 96dSWoB3ExNo+lRHixaTKm0GA4tpLY7w9xFCWHCmuAXpqvuH1gzA1Fj5zvhVRllSVKAl6s+EV8K DbVv/gncz+zQ0K9iL84iJOtnhen1gJNP49kRucLykCTN5pbcfleG5M/CoA3lh3CBhwMmWpSDkW9 7quO1gEjzQDOWZ4u4qQsGkpgnXrVvoItRlyM8jClHADGy1rbHnglCHfUjT+/+O1LaDWYGxcnMIh 27fjsg2bAYNghnVd0x4hm6nTNHgHM3oyAfR6W7Z/z87hm7U2pziag8P8SW+hiMMQFSHAPUDzpWB 8iLe893Lod+wsuMSfpgur2P07wVMf45paXxRStb9/vzIhZXr7YgHXOMvMqAvUspk2t2zIvOg5WB 41g18x41ma1rSVGkSORKqRiIXIVGAU+EnUArb85PEOhsiQcxdtUqeK8euHAYhUegTL5RLUDTWCS 9U5y8TSYnRti/cb3LOV4UX6WO+S X-Received: by 2002:a05:6000:1ac6:b0:439:c661:3245 with SMTP id ffacd0b85a97d-44bb5f246b8mr16110067f8f.34.1777900970043; Mon, 04 May 2026 06:22:50 -0700 (PDT) Received: from ?IPV6:2a01:e0a:106d:1080:4086:6e90:21c8:1555? ([2a01:e0a:106d:1080:4086:6e90:21c8:1555]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-44a8ea7d035sm24106863f8f.5.2026.05.04.06.22.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 04 May 2026 06:22:49 -0700 (PDT) Message-ID: <7683723f-df3b-418e-958d-9815d79b6a10@linaro.org> Date: Mon, 4 May 2026 15:22:49 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird From: Neil Armstrong Reply-To: Neil Armstrong Subject: Re: [PATCH v3 1/2] regulator: dt-bindings: document the SGM3804 Dual Output regulator To: Liam Girdwood , Mark Brown , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, KancyJoe References: <20260504-topic-sm8650-ayaneo-pocket-s2-sgm3804-v3-0-c4783443890b@linaro.org> <20260504-topic-sm8650-ayaneo-pocket-s2-sgm3804-v3-1-c4783443890b@linaro.org> Content-Language: en-US, fr Autocrypt: addr=neil.armstrong@linaro.org; keydata= xsBNBE1ZBs8BCAD78xVLsXPwV/2qQx2FaO/7mhWL0Qodw8UcQJnkrWmgTFRobtTWxuRx8WWP GTjuhvbleoQ5Cxjr+v+1ARGCH46MxFP5DwauzPekwJUD5QKZlaw/bURTLmS2id5wWi3lqVH4 BVF2WzvGyyeV1o4RTCYDnZ9VLLylJ9bneEaIs/7cjCEbipGGFlfIML3sfqnIvMAxIMZrvcl9 qPV2k+KQ7q+aXavU5W+yLNn7QtXUB530Zlk/d2ETgzQ5FLYYnUDAaRl+8JUTjc0CNOTpCeik 80TZcE6f8M76Xa6yU8VcNko94Ck7iB4vj70q76P/J7kt98hklrr85/3NU3oti3nrIHmHABEB AAHNKk5laWwgQXJtc3Ryb25nIDxuZWlsLmFybXN0cm9uZ0BsaW5hcm8ub3JnPsLAkQQTAQoA OwIbIwULCQgHAwUVCgkICwUWAgMBAAIeAQIXgBYhBInsPQWERiF0UPIoSBaat7Gkz/iuBQJk Q5wSAhkBAAoJEBaat7Gkz/iuyhMIANiD94qDtUTJRfEW6GwXmtKWwl/mvqQtaTtZID2dos04 YqBbshiJbejgVJjy+HODcNUIKBB3PSLaln4ltdsV73SBcwUNdzebfKspAQunCM22Mn6FBIxQ GizsMLcP/0FX4en9NaKGfK6ZdKK6kN1GR9YffMJd2P08EO8mHowmSRe/ExAODhAs9W7XXExw UNCY4pVJyRPpEhv373vvff60bHxc1k/FF9WaPscMt7hlkbFLUs85kHtQAmr8pV5Hy9ezsSRa GzJmiVclkPc2BY592IGBXRDQ38urXeM4nfhhvqA50b/nAEXc6FzqgXqDkEIwR66/Gbp0t3+r yQzpKRyQif3OwE0ETVkGzwEIALyKDN/OGURaHBVzwjgYq+ZtifvekdrSNl8TIDH8g1xicBYp QTbPn6bbSZbdvfeQPNCcD4/EhXZuhQXMcoJsQQQnO4vwVULmPGgtGf8PVc7dxKOeta+qUh6+ SRh3vIcAUFHDT3f/Zdspz+e2E0hPV2hiSvICLk11qO6cyJE13zeNFoeY3ggrKY+IzbFomIZY 4yG6xI99NIPEVE9lNBXBKIlewIyVlkOaYvJWSV+p5gdJXOvScNN1epm5YHmf9aE2ZjnqZGoM Mtsyw18YoX9BqMFInxqYQQ3j/HpVgTSvmo5ea5qQDDUaCsaTf8UeDcwYOtgI8iL4oHcsGtUX oUk33HEAEQEAAcLAXwQYAQIACQUCTVkGzwIbDAAKCRAWmrexpM/4rrXiB/sGbkQ6itMrAIfn M7IbRuiSZS1unlySUVYu3SD6YBYnNi3G5EpbwfBNuT3H8//rVvtOFK4OD8cRYkxXRQmTvqa3 3eDIHu/zr1HMKErm+2SD6PO9umRef8V82o2oaCLvf4WeIssFjwB0b6a12opuRP7yo3E3gTCS KmbUuLv1CtxKQF+fUV1cVaTPMyT25Od+RC1K+iOR0F54oUJvJeq7fUzbn/KdlhA8XPGzwGRy 4zcsPWvwnXgfe5tk680fEKZVwOZKIEuJC3v+/yZpQzDvGYJvbyix0lHnrCzq43WefRHI5XTT QbM0WUIBIcGmq38+OgUsMYu4NzLu7uZFAcmp6h8g Organization: Linaro In-Reply-To: <20260504-topic-sm8650-ayaneo-pocket-s2-sgm3804-v3-1-c4783443890b@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 5/4/26 14:44, Neil Armstrong wrote: > Document the SG Micro SGM3804 Single Inductor Dual Output Buck/Boost > Converter used to power LCD panels a provide positive and negative > power rails with configurable voltage and active discharge function > for each output. > > Signed-off-by: Neil Armstrong > --- > .../bindings/regulator/sgmicro,sgm3804.yaml | 77 ++++++++++++++++++++++ > 1 file changed, 77 insertions(+) > > diff --git a/Documentation/devicetree/bindings/regulator/sgmicro,sgm3804.yaml b/Documentation/devicetree/bindings/regulator/sgmicro,sgm3804.yaml > new file mode 100644 > index 000000000000..5ea475f8f8b6 > --- /dev/null > +++ b/Documentation/devicetree/bindings/regulator/sgmicro,sgm3804.yaml > @@ -0,0 +1,77 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/regulator/sgmicro,sgm3804.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: SG Micro SGM3804 Single Inductor Dual Output Buck/Boost Converter > + > +maintainers: > + - Neil Armstrong > + > +description: > + The SGM3804 is a dual voltage regulator, designed to support positive/negative > + supply for driving LCD panels. It support software-configurable output > + switching. The output voltages can be programmed via an I2C compatible interface. > + > +properties: > + compatible: > + const: sgmicro,sgm3804 > + > + reg: > + maxItems: 1 > + > + enable-gpios: > + minItems: 2 > + description: > + GPIO specifiers to enable the positive and negative outputs. > + > + vin-supply: true > + > +patternProperties: > + "^(pos|neg)$": > + type: object > + $ref: regulator.yaml# > + unevaluatedProperties: false > + > +required: > + - compatible > + - reg > + - enable-gpios > + - pos > + - neg > + > +additionalProperties: false > + > +examples: > + - | > + #include > + > + i2c { > + #address-cells = <1>; > + #size-cells = <0>; > + > + regulator@3e { > + compatible = "sgmicro,sgm3804"; > + reg = <0x3e>; > + > + vin-supply = <&vin_reg>; > + > + enable-gpios = <&gpio 17 GPIO_ACTIVE_HIGH>, > + <&gpio 18 GPIO_ACTIVE_HIGH>; So this fails with Rob's bot and with DTBS_CHECK with board DT but I can't make it fail with dt_binding_check on my side. enable-gpios: [[88, 59, 0], [88, 58, 0]] is too long But I found out in Documentation/devicetree/bindings/gpio/gpio-consumer-common.yaml : enable-gpios: maxItems: 1 description: GPIO connected to the enable control pin. This was rather unexpected... So I'll probably need update the bindings to mode the enable gpios in the subnodes instead. Neil > + > + pos { > + regulator-name = "outpos"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + }; > + > + neg { > + regulator-name = "outneg"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + }; > + }; > + }; > +... > + >