Page MenuHomeFreeBSD
Differential D34433

bcm5974: fix endian conversion signedness bug
ClosedPublic
Actions

Authored by val_packett.cool on Mar 3 2022, 11:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 8, 3:21 PM
Unknown Object (File)
Sun, May 5, 10:08 PM
Unknown Object (File)
Sun, May 5, 4:33 PM
Unknown Object (File)
Sun, May 5, 9:54 AM
Unknown Object (File)
Wed, May 1, 11:09 PM
Unknown Object (File)
Sat, Apr 27, 8:08 AM
Unknown Object (File)
Mar 22 2024, 11:16 PM
Unknown Object (File)
Mar 22 2024, 11:16 PM
View All 27 Files
Subscribers
imp
Contributor Reviews (src)

Details

Reviewers
wulf
Group Reviewers
desktop
Contributor Reviews (src)
Commits
rGfda9ac06aaf2: bcm5974(4): fix endian conversion signedness bug
Summary

This fixes wrong coordinates resulting in weird behavior.

This was correct in the wsp driver originally.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44653
Build 41541: arc lint + arc unit

Event Timeline

val_packett.cool created this revision.Mar 3 2022, 11:12 PM
Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptMar 3 2022, 11:12 PM
val_packett.cool requested review of this revision.Mar 3 2022, 11:12 PM
Harbormaster completed remote builds in B44639: Diff 103464.Mar 3 2022, 11:12 PM
val_packett.cool updated this revision to Diff 103465.Mar 3 2022, 11:14 PM

remove junk

Harbormaster completed remote builds in B44640: Diff 103465.Mar 3 2022, 11:14 PM
val_packett.cool edited the summary of this revision. (Show Details)Mar 3 2022, 11:17 PM
val_packett.cool added reviewers: wulf, desktop, Contributor Reviews (src).
val_packett.cool changed the repository for this revision from rS FreeBSD src repository - subversion to rG FreeBSD src repository.
wulf added inline comments.Mar 4 2022, 10:00 AM
sys/dev/hid/bcm5974.c
698

Don't modify input report. It can be exported via hidraw latter.

May be something like this will help?

#define WSP_LE2H(x) ((int32_t)(int16_t)le16toh(x))

With changing type of members of struct tp_finger to uint16_t

Or do sign extension with shifts

val_packett.cool retitled this revision from bcm5974: fix endianness bug to bcm5974: fix endian conversion signedness bug.Mar 4 2022, 3:12 PM
val_packett.cool updated this revision to Diff 103489.Mar 4 2022, 3:12 PM

switch to macro

Harbormaster completed remote builds in B44653: Diff 103489.Mar 4 2022, 3:12 PM
wulf accepted this revision.Mar 4 2022, 9:07 PM

Ouch. Actually, the macro should be named as BCM5974_LE2H rather than WSP_LE2H to follow style. I'll handle that.

This revision is now accepted and ready to land.Mar 4 2022, 9:07 PM
Closed by commit rGfda9ac06aaf2: bcm5974(4): fix endian conversion signedness bug (authored by val_packett.cool, committed by wulf). · Explain WhyMar 8 2022, 12:52 PM
This revision was automatically updated to reflect the committed changes.
wulf added a commit: rGfda9ac06aaf2: bcm5974(4): fix endian conversion signedness bug.

Revision Contents
Changeset List

PathSize
sys/
dev/
hid/
62 lines

Diff 103489

View Options

sys/dev/hid/bcm5974.c

Loading...