From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 228CDC433FE for ; Sat, 29 Oct 2022 07:48:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229932AbiJ2Hs3 (ORCPT ); Sat, 29 Oct 2022 03:48:29 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46910 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229739AbiJ2HsR (ORCPT ); Sat, 29 Oct 2022 03:48:17 -0400 Received: from mail-ed1-x535.google.com (mail-ed1-x535.google.com [IPv6:2a00:1450:4864:20::535]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9CF62BE2D4 for ; Sat, 29 Oct 2022 00:48:15 -0700 (PDT) Received: by mail-ed1-x535.google.com with SMTP id i21so10851612edj.10 for ; Sat, 29 Oct 2022 00:48:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=mind.be; s=google; 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=pg+RytIDlq54zISxpQtl2L8fZN1+m9I00gnazEQCvBw=; b=VZRajbXPCAtrrJfcWiqzHT9mbVUjHfnuf9m3BveCxFmOAp0ZyKxznlo5NXRV9jfpjm tdswy1Dc8Gwwdr9bSgLmhB5Lp+rtEYHjKEY2CjKHzqpWHxtEfJg08482LckxO435gbTz QIndJA3KIrJh21NXJTOBBrRJTbPb1lbvT0W4AfasLiZWgJhD6bbOh95K08gz7wEMM3Mr vC5+zaX8PfVbmRpWLyn1ByH1d3RhiYUZKbme1ECUiaGh5sRjfG2jkzL4IHoxQ3hPW5YN xt1dwzz6PCCdQ1MZJoPtoDUOWd6fFi1H77ICXLwMZ10AfC5QAqvMuOe6xnvosin/gpUY v1bg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=pg+RytIDlq54zISxpQtl2L8fZN1+m9I00gnazEQCvBw=; b=B320+dV/ika30aiSjGRWaXPL9N5YTy2pQgtuf73R+vWD38Gx40kRqRk2+ANvGy4v9+ mr0IkJIb1EVYmiCDKgE//V1cW1Vn1A/YvQBXvPUls4YT545KFIrWMyO61/Q+8VNvPBDc GiMZrPxzEYhHI/ogZqrqDkPezbE44Gg7qy+BcTI+hZuyrz7eMwjfC+QQMbizZ+FUYYr0 xl6xdHQPdzPtWBuucljiHUqOQCBUnf+HzCCK/E9DNFrCV3318t6JYu17PuAAayadFkNV lHLx3q1F01AI1LFB3AEjsVd73MKF//wcGjq1YrdZ9tHR5jDMBk4NyfVQY+1LOdpf2uB0 HrfQ== X-Gm-Message-State: ACrzQf1ZoAZGcgUhOQUGrPO7aGQO6LXPVRonOHMcOkIpJE33wLNbjGCk /4Vq094aRbu+ZG26ye1eIWtOTw== X-Google-Smtp-Source: AMsMyM580yL+VrmPLJoT1pN0Qr+g2h9lTiNJM6CALSo57nrSxd+LUPZJvrxkCwrilpLOTIvui9gUBA== X-Received: by 2002:a05:6402:5489:b0:43b:b935:db37 with SMTP id fg9-20020a056402548900b0043bb935db37mr3320635edb.347.1667029694230; Sat, 29 Oct 2022 00:48:14 -0700 (PDT) Received: from [192.168.2.9] (78-22-137-109.access.telenet.be. [78.22.137.109]) by smtp.gmail.com with ESMTPSA id gx3-20020a1709068a4300b0078c468bd604sm396505ejc.57.2022.10.29.00.48.13 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 29 Oct 2022 00:48:13 -0700 (PDT) Message-ID: <7cbb1cc5-e333-db25-77e2-7a1ebda70e24@mind.be> Date: Sat, 29 Oct 2022 09:48:12 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.3.1 Subject: Re: [PATCH 1/2] dt-bindings: leds-lp55xx: add ti,disable-charge-pump To: Krzysztof Kozlowski , Pavel Machek , Rob Herring , Krzysztof Kozlowski , Jacek Anaszewski Cc: linux-leds@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20221028133501.95833-1-maarten.zanders@mind.be> <20221028133501.95833-2-maarten.zanders@mind.be> <34c4037b-d152-412d-e551-dd3ea27b6b46@linaro.org> Content-Language: en-US From: Maarten Zanders In-Reply-To: <34c4037b-d152-412d-e551-dd3ea27b6b46@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/29/22 00:07, Krzysztof Kozlowski wrote: > Drop | OK. >> + Disable the internal charge pump. > Does it mean that it is enabled by default? The device default after reset is "off". The current implementation sets it to "auto". Other possible modes are "bypass" and "boost". Should I change to an optional "ti,charge_pump_mode" as string and set default to "auto"? Thanks!