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 EFF2BC0015E for ; Fri, 21 Jul 2023 11:41:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230143AbjGULlv (ORCPT ); Fri, 21 Jul 2023 07:41:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43542 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230045AbjGULlr (ORCPT ); Fri, 21 Jul 2023 07:41:47 -0400 Received: from mail11.truemail.it (mail11.truemail.it [217.194.8.81]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C2C202D4D; Fri, 21 Jul 2023 04:41:45 -0700 (PDT) Received: from francesco-nb.int.toradex.com (93-49-2-63.ip317.fastwebnet.it [93.49.2.63]) by mail11.truemail.it (Postfix) with ESMTPA id 15021209F8; Fri, 21 Jul 2023 13:41:40 +0200 (CEST) Date: Fri, 21 Jul 2023 13:41:35 +0200 From: Francesco Dolcini To: James Hilliard Cc: Krzysztof Kozlowski , devicetree@vger.kernel.org, Gregory CLEMENT , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Marek Vasut , Frieder Schrempf , Stefan Wahren , Philippe Schenker , Andreas Kemnade , Marcel Ziswiler , Christoph Niedermaier , Li Yang , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v6 2/3] ARM: dts: imx6qdl: Add Variscite VAR-SOM-MX6 SoM support Message-ID: References: <20230721103350.3684483-1-james.hilliard1@gmail.com> <20230721103350.3684483-2-james.hilliard1@gmail.com> <3492152a-d668-54ba-7cb4-4db8bfec6f03@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jul 21, 2023 at 05:05:51AM -0600, James Hilliard wrote: > On Fri, Jul 21, 2023 at 4:43 AM Krzysztof Kozlowski > wrote: > > On 21/07/2023 12:33, James Hilliard wrote: > > > + compatible = "regulator-fixed"; > > > + regulator-name = "touch_3v3_supply"; > > > + regulator-always-on; > > > > Also, missing constraints. > > Hmm, what's the correct way to determine the constraint values? You should know/understand the hardware design. Normally when writing a DT file you have the hardware schematics and the relevant datasheets available. > I'm guessing they should just be set to this based on the name?: > regulator-min-microvolt = <3300000>; > regulator-max-microvolt = <3300000>; This looks like a good guess, assuming that the regulator name is correct. Francesco