[PATCH v3 0/5] qcserial: refactoring support for Sierra Wireless devices

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

 



From: Bjørn Mork <bjorn@xxxxxxx>

I just have to admit that using the per-interface number matching
here wasn't such a good idea after all. It just does not scale
well when adding new devices, and I think we should break now
before we have added even more redundant match entries.

This series refactors the device layout selecting code to simplify
adding more layouts, and then define a Sierra Wireless specific 
layout.  This layout is applied to the existing entries.

The old support for single interface number matching is no longer
necessary after this, and is therefore removed.

Comments are appreciated.  I realize that this will complicate
stuff a bit for stable backports while both schemes are in use,
but I do believe it represents a necessary simplification in the
long run.  The purpose of the first patch is to assist stable
backporting of the new Dell devices added in this series.


v2:
 - rebased on top of recent qcserial device additions

v3: (based on review comments from Johan Hovold)
 - adding new Dell devices first to support stable backporting
 - fixing up multiline comment style 

Bjørn Mork (5):
  usb: qcserial: add a number of Dell devices
  usb: qcserial: fix multiline comment coding style
  usb: qcserial: refactor device layout selection
  usb: qcserial: define and use Sierra Wireless layout
  usb: qcserial: remove interface number matching

 drivers/usb/serial/qcserial.c | 121 ++++++++++++++++++++++++++----------------
 1 file changed, 75 insertions(+), 46 deletions(-)

-- 
2.0.0.rc0

--
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




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux