On Thu, 2018-02-01 at 09:17 +1100, Johan Hovold wrote: > On Wed, Jan 31, 2018 at 09:56:01AM +0100, Kristian Evensen wrote: > > Hi Johan, > > > > On Wed, Jan 31, 2018 at 7:38 AM, Johan Hovold <johan@xxxxxxxxxx> > > wrote: > > > This will probably have to do for now, but we already have > > > another > > > blacklist struct with the same content which we could rename and > > > reuse. > > > > I noticed the same, but wasn't quite sure about the policy on > > renaming/recycling and added a new blacklist entry. I can rename > > the > > entry and update references as part of this commit. What would be > > an > > appropriate name, something straight-forward like > > "net_intf4_intf5_blacklist"? > > Yeah, the policy isn't entirely clear to me either. ;) The > net_blacklist > are used to blacklist a single network interface, but here the other > interface was used for ADB and for the other driver it was for an > audio > interface I think. When I added/consolidated this feature a long time back I didn't think we'd end up with as many common entries as we have. I think it's fine to re-use use a common entry, but if you do and the common entry is named after a vendor/model, then make it generic. Dan > You can just add the duplicate entry for now and if this comes up > again, > we'll figure out a new (naming) policy. > > Thanks, > Johan > -- > To unsubscribe from this list: send the line "unsubscribe linux-usb" > in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html