Page MenuHomeFreeBSD
Differential D58563

apex: add a driver for the Google Apex Edge TPU
AcceptedPublic
Actions

Authored by seuros on Jul 30 2026, 8:12 PM.
Tags
Referenced Files
F176028076: D58563.id186603.diff
Thu, Oct 15, 10:36 AM
F176002796: D58563.id186612.diff
Thu, Oct 15, 5:14 AM
F175998621: D58563.id186603.diff
Thu, Oct 15, 4:20 AM
Unknown Object (File)
Wed, Oct 14, 7:16 AM
Unknown Object (File)
Wed, Oct 14, 4:06 AM
Unknown Object (File)
Wed, Oct 14, 4:06 AM
Unknown Object (File)
Wed, Oct 14, 4:04 AM
Unknown Object (File)
Wed, Oct 14, 12:48 AM
View All Files
Subscribers
imp
ivy
ziaee

Details

Reviewers
adrian
ziaee
Group Reviewers
drivers
manpages
Summary

The Apex is the accelerator on Coral PCI Express and M.2 modules,
appearing as 1ac1:089a
Google's Linux driver left staging in 5.16 and the out of tree
repository was archived; no BSD has had one.

Tested on: amd64, Coral M.2 Accelerator, running MobileNet v1
with the TensorFlow Lite 2.16 and LiteRT runtimes

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 76851
Build 73734: arc lint + arc unit

Event Timeline

seuros created this revision.Jul 30 2026, 8:12 PM
Herald added subscribers: ivy, ziaee, imp. · View Herald TranscriptJul 30 2026, 8:12 PM
seuros requested review of this revision.Jul 30 2026, 8:12 PM
Harbormaster completed remote builds in B75304: Diff 183133.Jul 30 2026, 8:12 PM
seuros added a comment.Jul 30 2026, 8:18 PM

The USB variant is more common than the PCIE/M2 in the wild, but i did not want to port it without testing it on hardware.

I will check with some frigate or home-assistant friends if they have they can undust the coral .

PS: the linux driver has a bug that will never be fixed (the repo is archived) which pushed people in stopped the coral usage. this port address it.

seuros added a parent revision: D58561: gasket: add a framework for Google accelerator ASICs.Jul 30 2026, 8:40 PM
seuros updated this revision to Diff 183142.Jul 30 2026, 10:36 PM

fix docs

Harbormaster completed remote builds in B75310: Diff 183142.Jul 30 2026, 10:36 PM
adrian added a comment.Sep 12 2026, 8:41 PM

in general good; just update your email address!

sys/dev/apex/apex.c
60

Maybe create a #define for how big this can get instead of '3' and use that for bounds checking the array?

ziaee requested changes to this revision.Sep 13 2026, 12:19 AM
ziaee added a reviewer: manpages.

looks cool; please also relnotes: yes for any new driver

share/man/man4/apex.4
14–27

can also add tunables and sysctls if you want, see share/examples/mdoc/example.4

44

see share/man/man5/style.mdoc.5

This revision now requires changes to proceed.Sep 13 2026, 12:19 AM
seuros updated this revision to Diff 186603.Sep 13 2026, 2:49 PM

refactor/doc/split driver

Harbormaster completed remote builds in B76843: Diff 186603.Sep 13 2026, 2:49 PM
seuros updated this revision to Diff 186612.Sep 13 2026, 5:18 PM

rebase on main

Harbormaster completed remote builds in B76851: Diff 186612.Sep 13 2026, 5:18 PM
adrian accepted this revision as: adrian.Mon, Oct 12, 2:01 AM
seuros updated this revision to Diff 189630.Mon, Oct 12, 4:59 PM

rebase

Harbormaster completed remote builds in B78077: Diff 189630.Mon, Oct 12, 4:59 PM
ziaee accepted this revision.Mon, Oct 12, 7:56 PM

Accepted provided feedback is addressed, thanks for tagging me and waiting for me.

share/man/man4/apex.4
8

Am I correct in understanding that this only works on amd64 for now? If so, please specify in the manpage. There is an optional architectural specifier here where you can do that.

44

This is not done, and it is specified in the style guide I linked.

72

This is not a sentence so is not traditionally capitalized/punctuated, see intro(5)

This revision is now accepted and ready to land.Mon, Oct 12, 7:56 PM
ziaee added a comment.Mon, Oct 12, 7:57 PM

wait omg please dont forget Relnotes: yes

adrian accepted this revision.Mon, Oct 12, 8:11 PM
seuros updated this revision to Diff 189676.Mon, Oct 12, 8:54 PM
seuros marked 3 inline comments as done.

relnotes

This revision now requires review to proceed.Mon, Oct 12, 8:54 PM
Harbormaster completed remote builds in B78107: Diff 189676.Mon, Oct 12, 8:54 PM
adrian accepted this revision.Tue, Oct 13, 1:16 AM
This revision is now accepted and ready to land.Tue, Oct 13, 1:16 AM
seuros marked 3 inline comments as done.Tue, Oct 13, 8:43 AM

Revision Contents
Changeset List

PathSize
etc/
mtree/
2 lines
include/
5 lines
share/
man/
man4/
1 line
92 lines
sys/
amd64/
conf/
6 lines
conf/
1 line
dev/
apex/
1178 lines
39 lines
145 lines
modules/
2 lines
apex/
8 lines

Diff 186612

View Options

etc/mtree/BSD.include.dist

Loading...
View Options

include/Makefile

Loading...
View Options

share/man/man4/Makefile

Loading...
View Options

share/man/man4/apex.4

Loading...
View Options

sys/amd64/conf/NOTES

Loading...
View Options

sys/conf/files.amd64

Loading...
View Options

sys/dev/apex/apex.c

Loading...
View Options

sys/dev/apex/apex_ioctl.h

Loading...
View Options

sys/dev/apex/apexreg.h

Loading...
View Options

sys/modules/Makefile

Loading...
View Options

sys/modules/apex/Makefile

Loading...