tree 86f268b539e173cab9f747889df30a0ae0c21830
parent 9c9f1e3a7453be7d12286000aa4fd825a7d48a25
author Jonathon Reinhart <jrreinhart@pigweed.infra.roller.google.com> 1726507438 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> 1726507438 +0000

roll: third_party/pigweed pw_spi_linux: Avoid unneccessary ioctl()s in Configure()

Initiator::Configure() is called on every Device::WriteRead() call, in
case different devices on the bus use different configuration.

Avoid unnecessary ioctl() calls by recording the current configuration
and skipping Configure() if the new config is not actually different.

If the pw::spi::Initiator base class were using the non-virtual
interface (NVI) pattern, this could be implemented there and apply to
all initiator implementations.

Test: Verified downstream project still works and redundant ioctls no
      longer happen
Original-Bug: 366541694
Original-Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/235877
Original-Revision: eefd313bdb13098552cd713598b937debe80d3d4

Rolled-Repo: https://pigweed.googlesource.com/pigweed/pigweed
Rolled-Commits: f66eafb0d8d1b5..eefd313bdb1309
Roller-URL: https://ci.chromium.org/b/8736620221636290081
GitWatcher: ignore
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: I21dbc82d6613640adadc0e8dc47c08218085d119
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/examples/+/236194
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Lint: Lint 🤖 <android-build-ayeaye@system.gserviceaccount.com>