Bitfields
@avisconti is already working on this.
Since Jan 9, 2025.
Assessment
This issue has not been assessed yet.
Description
This driver source code still associates compiler bitfield layout with host endianness - but the two are not associated. Bitfield layout is determined by the compiler and may not line up with the layout of the sensor device registers. Bitfields are to be avoided for this purpose.
Refer to #1
The endianness of the host is irrelevant in the first place.
Refer to #2
- Dominant language
- C
- Stars
- 5
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Similar issues
-
level/task module/gcp type/bug
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
-
Difficulty 1/5 Under an hour Newbie friendliness 86/100
hapostgres/pg_auto_failover#1190 ·
-
docs
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
P3 sonic-vpp
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
sonic-net/sonic-buildimage#29662 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 94/100
spack/spack-packages#6586 ·