Change subject: CHROMIUM: config: enable usb net drivers (dm9601/net1080/rtl8150) for all targets
......................................................................

Patch Set 3: I would prefer that you didn't submit this

Yup, this is just what I was looking for.  Thanks!

* For NET1080 and DM9601: no problem.  If we include these modules for x86 then I agree that we should include them for ARM and have no problem giving the +2.
* For RTL8150: this one is a little different because it was only in the Chromium OS config.  I believe that we are less picky about what we put in there since that doesn't get used with shipping products.  By moving it you're adding a new driver for shipping x86 (and ARM) products.  See the CL where it was purposely added only to the Chromium OS config <https://gerrit.chromium.org/gerrit/#/c/2599/>.  

Are there common USB ethernet adapters that use the RTL8150 driver?  Do we think the driver quality is good enough that we want users to be using it?

Marking -1 until sufficient justification for RTL8150.  ...if Olof (or someone else) wants to +2 I have no problem with that, though.

--
To view, visit https://gerrit.chromium.org/gerrit/37296
To unsubscribe, visit https://gerrit.chromium.org/gerrit/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9615109a5b17af293855595ba8f06f397db360c4
Gerrit-PatchSet: 3
Gerrit-Project: chromiumos/third_party/kernel
Gerrit-Branch: chromeos-3.4
Gerrit-Owner: Mike Frysinger <vap...@chromium.org>
Gerrit-Reviewer: Doug Anderson <diand...@chromium.org>
Gerrit-Reviewer: Gerrit <chrome-...@google.com>
Gerrit-Reviewer: Jon Kliegman <kli...@chromium.org>
Gerrit-Reviewer: Mike Frysinger <vap...@chromium.org>
Gerrit-Reviewer: Olof Johansson <ol...@chromium.org>