USB/U3G: Added support for Panasonic CF-F9 GOBI 3G modem to U3G module

[Tomasz CEDRO]

On Sun, Oct 1, 2017 at 8:23 PM, Ian Lepore <ian at freebsd.org> wrote:
> On Sun, 2017-10-01 at 20:17 +0200, Tomasz CEDRO wrote:
>> On Sun, Oct 1, 2017 at 7:21 PM, Hans Petter Selasky <hps at selasky.org>
>> wrote:
>> >
>> > On 10/01/17 19:09, Tomasz CEDRO wrote:
>> > >
>> > >
>> > > On Sun, Oct 1, 2017 at 6:40 PM, Ian Lepore <ian at freebsd.org>
>> > > wrote:
>> > > >
>> > > >
>> > > > On Sun, 2017-10-01 at 18:33 +0200, Tomasz CEDRO wrote:
>> > > > >
>> > > > >
>> > > > > On Wed, Sep 27, 2017 at 8:32 AM, Hans Petter Selasky <hps at sel
>> > > > > asky.org
>> > > > > >
>> > > > > >
>> > > > > > wrote:
>> > > > > >
>> > > > > > On 09/27/17 00:37, Ian Lepore wrote:
>> > > > > > >
>> > > > > > >
>> > > > > > >
>> > > > > > >
>> > > > > > > On Wed, 2017-09-27 at 00:30 +0200, Hans Petter Selasky
>> > > > > > > wrote:
>> > > > > > > >
>> > > > > > > >
>> > > > > > > >
>> > > > > > > >
>> > > > > > > > On 09/27/17 00:11, Tomasz CEDRO wrote:
>> > > > > > > > >
>> > > > > > > > >
>> > > > > > > > >
>> > > > > > > > >
>> > > > > > > > >
>> > > > > > > > > https://github.com/freebsd/freebsd/pull/115
>> > > > > > > > >
>> > > > > > > > > :-)
>> > > > > > > > >
>> > > > > > > > The devd.conf rule should match more than just vendor
>> > > > > > > > and
>> > > > > > > > product:
>> > > > > > > >
>> > > > > > > >
>> > > > > > > > +# Load GOBI 2000/3000 U3G QDL modem firmware on attach
>> > > > > > > > / boot.
>> > > > > > > > +# Note: This requires additional "gobi_loader" utility
>> > > > > > > > to be
>> > > > > > > > installed,
>> > > > > > > > +#   as well as valid QDL driver firmware files placed
>> > > > > > > > in
>> > > > > > > > /boot/firmware/gobi.
>> > > > > > > > +#   If modem does not accept valid firmware try
>> > > > > > > > gobi_loader
>> > > > > > > > -2000
>> > > > > > > > switch.
>> > > > > > > > +#   Please adjust modem VID/PID to match your device
>> > > > > > > > supported
>> > > > > > > > by
>> > > > > > > > u3g
>> > > > > > > > module.
>> > > > > > > > +#attach 100 {
>> > > > > > > > +#      match "vendor" "0x04da";
>> > > > > > > > +#      match "product" "0x250e";
>> > > > > > > > +#      action "/usr/local/sbin/gobi_loader /dev/cuaU0
>> > > > > > > > /boot/firmware/gobi/";
>> > > > > > > > +#};
>> > > > > > > >
>> > > > > > > > Else patch looks good.
>> > > > > > > >
>> > > > > > > > --HPS
>> > > > > > >
>> > > > > > > Hard-coding /dev/cuaU0 cannot possibly be right.
>> > > > > > >
>> > > > > > > -- Ian
>> > > > > > >
>> > > > > > These three lines are missing:
>> > > > > >
>> > > > > >          match "system"          "DEVFS";
>> > > > > >          match "subsystem"       "CDEV";
>> > > > > >          match "type"            "CREATE";
>> > > > > >
>> > > > > >
>> > > > > > --HPS
>> > > > > Thanks! Updated! :-)
>> > > > >
>> > > > > https://github.com/freebsd/freebsd/pull/115
>> > > > >
>> > > > If this is to be an example, it should be correct.  Please
>> > > > replace the
>> > > > "cuaU0" with "$cdev".  (See /etc/devd/uath.conf for an
>> > > > example).
>> > > >
>> > > > -- Ian
>> > >
>> > > Thanks Ian! Is it okay now? I have moved this example to
>> > > dedicated
>> > > /etc/devd/u3g.conf file, and added u3g load to
>> > > /etc/devd/usb.conf.. if
>> > > syntax is okay I will verify on my laptop..
>> > >
>> > > https://github.com/freebsd/freebsd/pull/115
>> > >
>> > Looks good to me. Don't forget to MFC!
>> >
>> > --HPS
>> /etc/devd/u3g.conf:
>>
>> notify 100 {
>>  match "system" "USB";
>>  match "subsystem" "DEVICE";
>>  match "type" "ATTACH";
>>  match "vendor" "0x04da";
>>  match "product" "0x250e";
>>  action "/usr/local/bin/gobi_loader /dev/$cdev /boot/firmware/gobi/";
>> };
>>
>> This does not work when /dev/$cdev is used.. but it works when
>> /dev/cuaU0 is used. Ian, could you please advise? :-)
>>
>
> Hmmm, then I'd have to surmise that everything in /etc/devd/uath.conf
> will fail the same way.
>
> I noticed earlier the events mentioned were
>
>          match "system"          "DEVFS";
>          match "subsystem"       "CDEV";
>          match "type"            "CREATE";
>
> and I would expect $cdev to have the right value on a CDEV/CREATE
> event, but not necessarily on a DEVICE/ATTACH event.
>
> -- Ian

I have verified on uath device (more on that below) and it turns out
$cdev works fine.. but it returns /dev/usb/X.Y.Z not the /dev/cuaU0
which does not work with this "gobi_loader" utility which requires
/dev/cuaU0 (CDC / serial port device)... any clues how to replace
$cdev with cuaUX? :-)

Regarding the UATH, I have TP-LINK TL-WN822N Ver2.0 based on Atheros
9002[1] and it seems to work with modified /dev/devd/uath.conf but the
uathload returns "Operation not permitted" when executed as root and
during boot..

[1] https://wikidevi.com/wiki/TP-LINK_TL-WN822N_v2

-- 
CeDeROM, SQ7MHZ, http://www.tomek.cedro.info