[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1014017: soundmodem: Fails to build with HID support <linux/hidraw.h>

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), 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 <n2ygk@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 <iu5hkx@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:
> >
> > 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
> >

Reply to: