Page MenuHomeFreeBSD
Differential D35195

comms/opencbm-kmod: Remove unused DRIVER_MODULE devclass on recent main.
ClosedPublic
Actions

Authored by jhb on May 13 2022, 4:54 PM.
Tags
None
Referenced Files
F152979157: D35195.id105946.diff
Sat, Apr 18, 11:05 AM
F152964470: D35195.id106216.diff
Sat, Apr 18, 9:07 AM
F152964422: D35195.id106216.diff
Sat, Apr 18, 9:07 AM
F152953101: D35195.diff
Sat, Apr 18, 7:24 AM
Unknown Object (File)
Fri, Apr 17, 10:47 AM
Unknown Object (File)
Fri, Apr 17, 3:23 AM
Unknown Object (File)
Wed, Apr 15, 8:42 AM
Unknown Object (File)
Mon, Mar 30, 2:59 AM
View All Files
Subscribers
mat

Details

Reviewers
zirias
Commits
R11:7175e1ab5ff1: comms/opencbm-kmod: Remove unused DRIVER_MODULE devclass on recent main.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb created this revision.May 13 2022, 4:54 PM
Herald added a subscriber: mat. · View Herald TranscriptMay 13 2022, 4:54 PM
jhb requested review of this revision.May 13 2022, 4:54 PM
Harbormaster completed remote builds in B45565: Diff 105946.May 13 2022, 4:54 PM
jhb added a reviewer: zirias.May 13 2022, 4:54 PM
zirias accepted this revision.May 13 2022, 5:57 PM

Looks fine for now, trusting your word the version constant is correct, hehe. Should probably try to take this upstream (and then update the port) ;)

This revision is now accepted and ready to land.May 13 2022, 5:57 PM
jhb added a child revision: D35168: *DRIVER_MODULE*: Remove the compat shims to accept the devclass argument..May 17 2022, 10:05 PM
jhb added a comment.May 20 2022, 5:09 PM

Do you have a relationship with upstream to post the patch there?

Closed by commit R11:7175e1ab5ff1: comms/opencbm-kmod: Remove unused DRIVER_MODULE devclass on recent main. (authored by jhb). · Explain WhyMay 20 2022, 5:14 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: R11:7175e1ab5ff1: comms/opencbm-kmod: Remove unused DRIVER_MODULE devclass on recent main..
zirias added a comment.May 21 2022, 7:12 AM
In D35195#799397, @jhb wrote:

Do you have a relationship with upstream to post the patch there?

Kind of, I originally ported the Linux driver to FreeBSD ;) So, I'll open a pull request soon.

zirias added a comment.Jul 11 2022, 7:04 PM

Patch just landed upstream: https://github.com/OpenCBM/OpenCBM/commit/6e438c1b32d39e5238f65246e192c89b22fdfd82 – sorry for the long delay.

Revision Contents
Changeset List

PathSize
comms/
opencbm/
files/
25 lines

Diff 106216

View Options

comms/opencbm/files/patch-opencbm_sys_freebsd_opencbm.c

Loading...