Page MenuHomeFreeBSD

[new port] systuils/gobi_loader: Firmware Loader for Qualcomm Gobi USB Chipsets
ClosedPublic

Authored by zarychtam_plan-b.pwste.edu.pl on Dec 27 2019, 9:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 3 2024, 12:45 AM
Unknown Object (File)
Mar 3 2024, 12:45 AM
Unknown Object (File)
Mar 3 2024, 12:45 AM
Unknown Object (File)
Mar 3 2024, 12:45 AM
Unknown Object (File)
Mar 3 2024, 12:45 AM
Unknown Object (File)
Mar 3 2024, 12:45 AM
Unknown Object (File)
Mar 3 2024, 12:45 AM
Unknown Object (File)
Mar 3 2024, 12:45 AM
Subscribers

Details

Summary

Many people are still using this old piece of software but it's still missing in the ports tree. For source is used a trimmed fork of known to be working for years https://github.com/kicer/gobi_loader project. Original http://www.codon.org.uk/~mjg59/gobi_loader probably never worked on FreeBSD.

Test Plan

Initial version. Needs more testing and quality assurance.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

gobi_loader/Makefile
24

Remove blank line

26

share/examples/devd/gobi_loader.conf.sample might be a better idea as it's not going to work out of box due to missing firmware?

gobi_loader/files/gobi.conf.in
7 ↗(On Diff #66038)

Is it possible to avoid hardcoding /dev/cuaU0?
/boot/firmware/gobi/ should probably be /usr/local/lib/firmware? (use ports path variables)

gobi_loader/pkg-message
7

Multiple typos and what should the firmware file name be?

zarychtam_plan-b.pwste.edu.pl added inline comments.
gobi_loader/Makefile
17

This one could be useful

linimon retitled this revision from gobi_loader port to [new port] systuils/gobi_loader: Firmware Loader for Qualcomm Gobi USB Chipsets.Dec 28 2019, 1:49 AM
gobi_loader/Makefile
17

Not sure why the line shifted but I meant GH_PROJECT=

Removed GH_PROJECT= and a few whitespaces; reachable e-mail address of port maintainer added.

So what's the status of this port?

I have updated patch to clarify how to obtain firmware files. Port builds and works 100% fine. It was tested with portilint and poudriere testport.

imp added inline comments.
sysutils/gobi_loader/Makefile
22 ↗(On Diff #90816)

This could use another tab indent...

But I'll fix that when committing.

This revision is now accepted and ready to land.Jun 13 2021, 8:01 PM