Page MenuHomeFreeBSD
Differential D54620

ads111x: Convert driver to CTLFLAG_MPSAFE
ClosedPublic
Actions

Authored by guest-seuros on Jan 9 2026, 3:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 4, 3:11 PM
Unknown Object (File)
Sun, Mar 1, 10:49 AM
Unknown Object (File)
Sat, Feb 28, 5:52 PM
Unknown Object (File)
Sun, Feb 15, 12:45 AM
Unknown Object (File)
Wed, Feb 11, 5:18 PM
Unknown Object (File)
Jan 20 2026, 10:28 AM
Unknown Object (File)
Jan 20 2026, 2:25 AM
Unknown Object (File)
Jan 20 2026, 1:28 AM
View All 18 Files
Subscribers
None

Details

Reviewers
jhb
imp
markj
Commits
rG714d5b382fc4: ads111x: Convert driver to CTLFLAG_MPSAFE
rG2c029cff5ba3: ads111x: Convert driver to CTLFLAG_MPSAFE
Summary

Replace CTLFLAG_NEEDGIANT with CTLFLAG_MPSAFE for all ADC sensor
sysctls. The driver uses I2C bus transfers which handle locking
internally via the I2C bus layer.

Sysctls converted:

  • dev.ads111x.X.channelN.gain_index (PGA setting)
  • dev.ads111x.X.channelN.rate_index (sample rate)
  • dev.ads111x.X.channelN.voltage (sampled voltage)
  • dev.ads111x.X.config (configuration register)
  • dev.ads111x.X.lo_thresh (comparator low threshold)
  • dev.ads111x.X.hi_thresh (comparator high threshold)

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

guest-seuros created this revision.Jan 9 2026, 3:53 PM
guest-seuros held this revision as a draft.
Harbormaster completed remote builds in B69755: Diff 169391.Jan 9 2026, 3:53 PM
guest-seuros published this revision for review.Jan 9 2026, 3:58 PM
markj accepted this revision.Jan 12 2026, 12:50 AM

The driver uses I2C bus transfers which handle locking internally via the I2C bus layer.

Well, really it serializes all of the sysctl handlers using an internal lock.

This revision is now accepted and ready to land.Jan 12 2026, 12:50 AM
Closed by commit rG2c029cff5ba3: ads111x: Convert driver to CTLFLAG_MPSAFE (authored by guest-seuros, committed by markj). · Explain WhyJan 12 2026, 12:58 AM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG2c029cff5ba3: ads111x: Convert driver to CTLFLAG_MPSAFE.
markj added a commit: rG714d5b382fc4: ads111x: Convert driver to CTLFLAG_MPSAFE.Jan 26 2026, 12:52 AM

Revision Contents
Changeset List

PathSize
sys/
dev/
iicbus/
adc/
12 lines

Diff 169496

View Options

sys/dev/iicbus/adc/ads111x.c

Loading...