On Mon, Sep 21, 2020 at 10:43:12AM +0200, Oliver Neukum wrote: > Am Montag, den 21.09.2020, 10:10 +0200 schrieb Johan Hovold: > > Add support for Whistler radio scanners TRX series, which have a union > > descriptor that designates a mass-storage interface as master. Handle > > that by generalising the NO_DATA_INTERFACE quirk to allow us to fall > > back to using the combined-interface detection. > > Hi, > > it amazes me what solutions people can come up with. Yet in this case > using a quirk looks like an inferior solution. If your master > is a storage interface, you will have a condition on the device you > can test for without the need for a quirk. Sure, and I mentioned that as an alternative, another would be checking for a control interface with three endpoints directly. My fear is that any change in this direction risk introducing regression if there are devices out there with broken descriptors that we currently happen to support by chance. Then again, probably better to try to handle any such breakage if/when reported. I'll respin. Johan
