Dave: This patch would be mostly invisible to existing users. It would add the gpio attribute to the soundmodem configuration with a default empty value.
73 On Thu, Nov 2, 2023 at 11:25 AM Dave Hibberd <d...@vehibberd.com> wrote: > Hi Alan, > > I'm part of quite an active packet communtiy ( > https://ukpacketradio.network/) - DINAH looks like a cool item a number > of folks would be itnerested in! Not many of our users are on soundmodem - > direwolf, G8BPQ's QtSoundmodem (which I plan to upload to Debian) and (in > hardware) NinoTNC are the flavour of the month for us. > > I know of a few CM108 mods (our sister communtiy even has a guide - > https://wiki.oarc.uk/cm108_sound_interface_smd) > <https://wiki.oarc.uk/cm108_sound_interface_smd?s[]=cm108>, so extending > functionality to more people would be generally of benefit to all. > > Soundmodem is a little old, but it's also not moving very fast so I don't > see too much overhead in maintaining a patch for it unless it's likely to > degrade the experience for other users? > > Cheers > > DH > > -- > Hibby > MM0RFN > > On Thu, 2 Nov 2023, at 2:45 PM, Alan Crosswell wrote: > > Hey Daniele, > > It's been about a year and I've just now gotten around to building a > Raspberry Pi connected to a DINAH and can confirm that this PTT patch still > works on the latest Raspi Bullseye distro. I don't know if there's any > interest in carrying this forward to a committed patch for soundmodem. I > can always keep patching it myself if I'm the only one who still thinks > soundmodem is a nice small tool for AX.25. > > Regarding many more CM108's, I wonder how many of them are integrated such > that a spare GPIO pin is used for PTT? Given it's probably not a lot, I > wouldn't think removing the device test entirely would be a huge issue. > Would you like me to submit a revised PR to do that? > > 73 de N2YGK > > On Mon, 24 Oct 2022 09:14:23 -0400 Alan Crosswell <n2...@weca.org> wrote: > > Yeah I don't know that ignoring the device code would be much of a > problem. > > It's not like it searches available devices to see which one to use; the > > specific device to use is specified. > > > > On Sun, Oct 23, 2022 at 3:10 PM Daniele Forsi <iu5...@gmail.com> wrote: > > > > > Hello Alan, > > > > > > I committed your patch to configure.ac in a branch and I think that we > > > should merge it to master: > > > https://salsa.debian.org/debian-hamradio-team/soundmodem/-/tree/hidraw > > > > > > I didn't commit your patch to ptt.c yet. > > > What happens if we drop the check for hiddevinfo.product for C-Media > > > entirely? > > > > > > You changed the test to work with your hardware, which is fine, but it > > > seems that there are many more CM108s out there (I have one with ID > > > 0d8c:013c). > > > I'm copying the list from https://usb-ids.gowdy.us/read/UD/0d8c so it > > > is archived with this bug report. > > > > > > Id Name > > > 0001 Audio Device > > > 0002 Composite Device > > > 0003 Sound Device > > > 0004 CM6631A Audio Processor > > > 0005 Blue Snowball > > > 0006 Storm HP-USB500 5.1 Headset > > > 000c Audio Adapter > > > 000d Composite Device > > > 000e Audio Adapter (Planet UP-100, Genius G-Talk) > > > 0012 > > > 0014 Audio Adapter (Unitek Y-247A) > > > 001f CM108 Audio Controller > > > 0029 > > > 0102 CM106 Like Sound Device > > > 0103 CM102-A+/102S+ Audio Controller > > > 0104 CM103+ Audio Controller > > > 0105 CM108 Audio Controller > > > 0107 CM108 Audio Controller > > > 010f CM108 Audio Controller > > > 0115 CM108 Audio Controller > > > 0134 > > > 0139 Multimedia Headset [Gigaware by Ignition L.P.] > > > 013c CM108 Audio Controller > > > 0201 CM6501 > > > 5000 Mass Storage Controller > > > 5200 Mass Storage Controller(0D8C,5200) > > > b213 USB Phone CM109 (aka CT2000,VPT1000) > > > > > > -- > > > 73 de IU5HKX Daniele > > > > > >
