Received-SPF: pass (google.com: domain of linux-kernel+bounces-138142-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Wed, 10 Apr 2024 10:08:48 +0200
From: Sebastian Reichel <sebastian.reichel@collabora.com>
To: Hermes Zhang <chenhuiz@axis.com>
Cc: Andrew Davis <afd@ti.com>, Hermes Zhang <Hermes.Zhang@axis.com>, 
	kernel@axis.com, Pali =?utf-8?B?Um9ow6Fy?= <pali@kernel.org>, 
	linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] power: supply: bq27xxx: Divide the reg cache to each
 register
Message-ID: <i4tusers6p4ofzusgalshmiekfeczfvfvkjrmz42n5saaq2nmy@th6eweffbu6k>
References: <20240306100904.1914263-1-Hermes.Zhang@axis.com>
 <c4u27576oazfrlcp5avy3ect3i4jlsmdvi7nlun5qvez3ipti2@ue5jxbydmevs>
 <c3c219ca-e126-42fb-8f20-5df0dec58d7c@ti.com>
 <ff34e22e-702f-acc9-469d-33f092bf20bd@axis.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="n3zerunypjlzwp5a"
Content-Disposition: inline
In-Reply-To: <ff34e22e-702f-acc9-469d-33f092bf20bd@axis.com>


--n3zerunypjlzwp5a
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi,

On Tue, Apr 02, 2024 at 10:04:22AM +0800, Hermes Zhang wrote:
> On 2024/4/1 21:57, Andrew Davis wrote:
> > On 4/1/24 8:15 AM, Sebastian Reichel wrote:
> > > [+cc Andrew Davis]
> > >=20
> > > Hello Hermes,
> > >=20
> > > Sorry for the delay. This arrived too close to the 6.9 merge window.
> > > I had a look now and while the patch looks fine to me on a conceptual
> > > level, it did not apply. It looks like you used a pre-2024 kernel tree
> > > to generate the patch against. Please always use something recent base
> > > tree (and ideally use git's --base option to document the used
> > > parent commit).
> > >=20
> > > Other than that I just applied a series from Andrew, which cleans up
> > > the register caching in bq27xxx and removed most registers from the
> > > cache. That's something I did not consider earlier, since I thought
> > > the cache was introduced to fix a different issue. But that was
> > > apparently sbs-battery and not bq27xxx.
> >=20
> > The original BQ27000 device did not have an interrupt pin to signal
> > when important updates to the battery occurred, so early devices
> > using it would have userspace poll those values. As the kernel driver
> > added its own polling for updates, it seems the early driver authors
> > simply decided to cache the values between kernel reads and return
> > those to userspace when it reads.
> >=20
> > This is a problem though for two reasons.
> > 1. If no one is interested in these values the kernel will still poll
> > =A0=A0 them anyway, wasting I2C bus time and power.
> > 2. If userspace is actually interested in some value and so checks it
> > =A0=A0 often, it will only get real updated values when the kernel next
> >    polls, which might not be often enough for some use-cases.
> >=20
> Agree! Also good to know the history.

well, there was another driver, which had to introduce caching to
reduce I2C bus load when userspace polls data too quickly. Since
bq27xxx already had caching, its removal might have regressed some
platform. If nobody complains I'm of course happy with the much
simpler code :)

-- Sebastian

--n3zerunypjlzwp5a
Content-Type: application/pgp-signature; name="signature.asc"

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

iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAmYWSQwACgkQ2O7X88g7
+poY8A/9G1KVulGheVYOGNJYFbUgTPbeVzesFW2K4/x1y9k/kk1Y43Afl4uujE/g
94h+7hWylbvYGuJ0MfwbmaokV0J9ST2qsWjRpPBSgm4dw9NVXZ3ks5qHZtRFJJWX
3XHOIaJinQ8JTcU3QUlp0soMgOKl78c+5yAPzSeg8cGsorE1LwNdVgE73VMku+fc
4yPeOsbKxZphsfvPrJkJfA4FanWHqwdIygYxiiqXKIsjz2V/xQmdX+OGR4HwKuqC
B4CIhQx0SgaHr6txIBI0sgpCrhcLNQhwy/5LWH3RRWmKKwkkXmSqLKnHl7DxtHD+
W/45spUkCeRN1izobpXMsh26xwcSLDatJvoCxE3m49SdglI0SjLXpz7jv2er1JWz
xrgVJJ7f+SkXVQMo3sLwJ8ITe18MmyEUn0dT1kvVnDmIulhyXoWXmXXnD1bevwgr
Du2+gxn1ih35eJ0mP3ZnWxeix2OJG+Q5//d73OkY+hQyijOv8rOIKbqD2FwNC+Sc
qXvJxASoKBAz1N/RwkfUe9bxGSuqJtgVjuWD2VAiRGFL92KUBsdlC4phvvThWhb5
Xz774Y67R8RKhBCRZXWIfH/PnHbnZEcOp7jnQngfsBkBhYMGb+7nYhQkcpkZ2KcY
TTYDAqfstI8UMWeRf3tSIeurRciDl6u9PPpTLXyYieytch1LbtY=
=EEZn
-----END PGP SIGNATURE-----

--n3zerunypjlzwp5a--