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 phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id CB52BC77B7A for ; Fri, 19 May 2023 08:41:35 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id D5BDE86249; Fri, 19 May 2023 10:41:32 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=denx.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1684485693; bh=hOUiuTAB2x1I8gHkh635YQ6KKM4eDFCAG1lO1f7s4ZI=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=k4WuP40JV0aFXxq9HE2kVnOQiO3/C2gBITKl4+yhflnoplchxwjghIUZxanileP64 wXs0tY8i+ILIdvoiDL4UDV2C04YfeU7dVt5wWKO7wKlCy1o7aAzRgmn+An/yBcBjts DPogRo01yM1mfb1eiVD/K9TWiaOQ8Dkq7N8GeEEwKoiUdYTBhybh1hBWohN7tZKmhm S3IhpY3wO+6CxJzdhttGNf7w9vYAUWmLZXJgyYMTz7x6lOEnOmP7KCeom8CXmjXRto lL7AwaRa4DfSuMJbLTRdPaJR9JzhybnZ/AT9OTePJKJvxlVjKMT8PZuwuaKWHS4kph uIaDo+l9ZxQKQ== Received: from wsk (85-222-111-42.dynamic.chello.pl [85.222.111.42]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) (Authenticated sender: lukma@denx.de) by phobos.denx.de (Postfix) with ESMTPSA id 774028471D; Fri, 19 May 2023 10:41:30 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=denx.de; s=phobos-20191101; t=1684485690; bh=hOUiuTAB2x1I8gHkh635YQ6KKM4eDFCAG1lO1f7s4ZI=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=BjOzc6Gf1DYZEnhRtqLBB10UpiyZT1w1nf7jFTTID2UTYXxwAk9RxtNXqFAqxPjPK QTIKyLub0kvi/WbSQ6CGgyvrU4VXaHrNo+UXIan3irAZ37WQSn7kvYCD3EYMhBF/MS mGtNmVAsLHUFIVS0F7aj4i07YCuNBnQCqLEVIizX/ikbiy7M/KKQOZae18g78CSxxs KqiBn3UzfmOgzqTtMH+lS84OKYM+YjglzHs+va+8u5S7H8wiqx2f4lxfnb3u4baUik xXBpICx40ekjH8zpnX4Xp5PVN99UEbQLkq45g4Mrb9NQiSINKh8Ex7W5fBvgN9yWz4 4+HPPWBsWy05A== Date: Fri, 19 May 2023 10:41:21 +0200 From: Lukasz Majewski To: Marek Vasut Cc: U-Boot Mailing List , Stefano Babic , Peng Fan , Peng Fan , Fabio Estevam , "'NXP i . MX U-Boot Team'" Subject: Re: [PATCH 1/5] arm: mxs: Provide Kconfig option to to not use VDD5V as IMX28 PMU source Message-ID: <20230519104121.5f4c3882@wsk> In-Reply-To: <0d387cf2-3f10-d796-6b07-083761564b79@denx.de> References: <20230509143243.1523791-1-lukma@denx.de> <08c74cf0-5f10-f745-99af-ab8fb4698767@denx.de> <20230509164639.13dc0f22@wsk> <20230509170405.3bd74a16@wsk> <0d387cf2-3f10-d796-6b07-083761564b79@denx.de> Organization: denx.de X-Mailer: Claws Mail 3.19.0 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/gAdA+AFZy.0j8oyix8S6+s6"; protocol="application/pgp-signature"; micalg=pgp-sha512 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean --Sig_/gAdA+AFZy.0j8oyix8S6+s6 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Marek, > On 5/9/23 17:04, Lukasz Majewski wrote: > > Hi Marek, > > =20 > >> On 5/9/23 16:46, Lukasz Majewski wrote: =20 > >>> Hi Marek, =20 > >> > >> Hi, > >> =20 > >>>> On 5/9/23 16:32, Lukasz Majewski wrote: =20 > >>>>> This option sets the current limit for 5V source to zero, so all > >>>>> the PMU outputs are primarily powered from battery source > >>>>> (DCDC_BAT). > >>>>> > >>>>> This option may be set on systems, where the 5V is NOT supposed > >>>>> to be in any scenario powering the system - for example on > >>>>> systems where DCDC_BAT is connected to fixed and regulated 4.2V > >>>>> source (so the "battery" is not present). =20 > >>>> > >>>> Is it possible to dig this information out of DT or is the DT not > >>>> available in SPL for this platform ? =20 > >>> > >>> This platform uses only SPL_OF_PLATDATA to keep the SPL size < 40 > >>> KiB. > >>> > >>> I would prefer to keep this as Kconfig as the PMU driver is not > >>> supporting DT and in fact now ARCH_MX28 only has two boards > >>> supported - XEA and imx28evk. =20 > >> > >> Why not use the platdata then ? > >> This makes it far better, as the code is actually always compiled > >> and doesn't start to bitrot. =20 > >=20 > > As fair as I can tell - the MX28 PMU driver in SPL is not supporting > > OF_PLATDATA. Or have I overlooked something? > >=20 > > I would like to investigate the PMU issue, and avoid extra work to > > provide platdata support for two supported imx287 based boards. =20 >=20 > Hum, considering that this platform is basically legacy and winding=20 > down, I guess that's OK too. Are there any other issues to be solved? Or is this patch set eligible for pulling? (Especially that NXP has confirmed that approach used in this patch set is in sync with their idea of PMU usage). Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Erika Unter HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma@denx.de --Sig_/gAdA+AFZy.0j8oyix8S6+s6 Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCgAdFiEEgAyFJ+N6uu6+XupJAR8vZIA0zr0FAmRnNjEACgkQAR8vZIA0 zr375Qf/ZqNdvEiRjgM1y4DKEDgJPamAqv6jQwx+Ly1agaRy7+dVpPk/onUsNbwu P43omMe8v+pegb7T6uO5xP7wkfR2h/dT1/UpJHKhPZ67YX05I7JDae39q5PeqYM/ xNPN+67BRFVPhDNE6FDLq4uKyzVHhQZ3+dToUqliCRa69L57N4i0pnrH3sOXh+IU y2rBywaEkjSrgvgs19kGHFh3nJGBtGBUXAub8B4j0wPX4S54z9FBB03HBqG9STRv dmICVTd1xTj1Phum/2UVNtx7uhmDqyBc9V83+DeyCgSpYQ7SrgLyiL0//Py2GG9D RCthosVINeLpZ8Icfu4r1h/iFQIjIg== =hUZu -----END PGP SIGNATURE----- --Sig_/gAdA+AFZy.0j8oyix8S6+s6--