Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
Date:   Mon, 6 Nov 2023 21:06:05 +0100
From:   Wolfram Sang <wsa+renesas@sang-engineering.com>
To:     Johan Hovold <johan@kernel.org>
Cc:     linux-renesas-soc@vger.kernel.org,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4 3/3] gnss: ubx: add support for the reset gpio
Message-ID: <ZUlHLZ1mm9IpSPal@ninjato>
Mail-Followup-To: Wolfram Sang <wsa+renesas@sang-engineering.com>,
        Johan Hovold <johan@kernel.org>, linux-renesas-soc@vger.kernel.org,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        linux-kernel@vger.kernel.org
References: <20231103225601.6499-1-wsa+renesas@sang-engineering.com>
 <20231103225601.6499-4-wsa+renesas@sang-engineering.com>
 <ZUj3fzArgPU4kYYT@hovoldconsulting.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="XtrtXT+bQEBVz+kB"
Content-Disposition: inline
In-Reply-To: <ZUj3fzArgPU4kYYT@hovoldconsulting.com>
Precedence: bulk


--XtrtXT+bQEBVz+kB
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable


> 	1.5 I/O pins
> 	RESET_N: Reset input
> 	Driving RESET_N low activates a hardware reset of the system.
> 	Use this pin only to reset the module. Do not use RESET_N to
> 	turn the module on and off, since the reset state increases
> 	power consumption.
>=20
> (and AFAIU you should generally not try to use reset this way unless it
> is explicitly said to be supported).

Oh! That's the opposite of my intention :/ Okay, today I learnt
something. Thank you for pointing this out. I will remember this and
double check reset handling in the future.

That means I only need to de-assert reset in probe() for now, right?


--XtrtXT+bQEBVz+kB
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmVJRy0ACgkQFA3kzBSg
Kbblgw/9HkWUa0iBE7Mo2bg3mjgmftVGc9rJBWSmmDiocs9jX9ksuSH6a5AHiqMe
+xpfG6MI4xjUzLGL1+YaITrYQyKE3ZUMSLeKqwJSV+qxhBo+biDtWq/EKExyO+91
PAb553VbrW+QmdK2+bcsHKi+LqZ0kSy18IF9vaCV7fQozNXwRvxocyTBLGZmfrZo
xQ0+Mt1+T5nNkU8dHtyKENHuMlgFYSlGbUiLEdIUPYBNA9TAew+7JDiyx6ZS88hu
Njkso91lBK981JBPS3kmpL+/WsRXsm+tEuc/6snQ4TWu9LlIDv+TsYXZ6rh3Oe7L
B+3L+mMa+fUcRkt0rfCh18hbLASgtQZ67HfT5t5jUqlq5L4aR0wYNHNt1Gu2S3g/
Of/XiEfvD54yjwled7izZEW9U7yQcSrPSi49DSzuibavicjYK+dKTw7P/v21z2Pc
UWlAuFsoyamyGyswY2Z1vIBSnFtIgySFcr6cS5YGwTHomzxWUkH81zjKtZfIHmD8
hafMaegBE40vE+9wBMp0+UpAv5Zt5KgI7W6FCHIiu8offra9dch54SiaARjJlMe4
etaFgu//W+bxGQDUuIfNJER8V8FMsb+3gip1eR7dQq+MnZqIVubaVIRhCOYoxDZf
7Dxkr1o28KP3W4ETDirAAuR8/BtaZimx/9uXm9CEewYtGX7GANU=
=TGul
-----END PGP SIGNATURE-----

--XtrtXT+bQEBVz+kB--