Hi > > I believe that in the future all gadget drivers will define their own > SS > > descriptors with values that are suitable for that gadget driver and > the > > create_ss_descriptors() will become obsolete. As I've already > mentioned this > > is just a way to allow all the existing drivers to operate in SS mode > with > > minimum changes to their code. So this patch doesn't contradict your > > approach, just extends it. > > but that's not a good approach. You're adding bloat which will be > removed soon. Problem is that once it's there, we will have to support > it for a long long time as we won't know if there are any out of tree > gadget drivers depending on that, forcing us on publising of > Documentation/feature-removal-schedule.txt that we will remove that > layer in e.g. 4 major revisions. So, IMHO, better not to add it. > I see your point. I'll generate a different patch for this function (create_ss_descriptors()) so it will be easy to take it out of the patch series if we eventually decide against it. I think it's still useful to have this ability since it can help developers at the beginning even if not adding it to the formal kernel release and just have it in the mailing list archives. Best regards, Tanya Brokhman Consultant for Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum -- 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