lkml.org 
[lkml]   [2016]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH net-next 2/3] r8152: modify rtl ops init
Date
Hayes Wang <hayeswang <at> realtek.com> writes:
> Replace using VID/PID with using tp->version to initialize the ops.

Hayes,
This patch breaks new HW with existing drivers. more below.

>
> Signed-off-by: Hayes Wang <hayeswang <at> realtek.com>
> ---
> drivers/net/usb/r8152.c | 79 ++++++++++++++++++--------------------------
-----
> 1 file changed, 28 insertions(+), 51 deletions(-)
>
> diff --git a/drivers/net/usb/r8152.c b/drivers/net/usb/r8152.c
> index 4b6db8a..cf1b8a7 100644
> --- a/drivers/net/usb/r8152.c
> +++ b/drivers/net/usb/r8152.c
> <at> <at> -3742,66 +3742,43 <at> <at> static void
rtl8153_unload(struct r8152 *tp)
> r8153_power_cut_en(tp, false);
> }
>
> -static int rtl_ops_init(struct r8152 *tp, const struct usb_device_id *id)
> +static int rtl_ops_init(struct r8152 *tp)
> {
> struct rtl_ops *ops = &tp->rtl_ops;
> - int ret = -ENODEV;
> -
> - switch (id->idVendor) {
> - case VENDOR_ID_REALTEK:
> - switch (id->idProduct) {
...
> default:
> + ret = -ENODEV;
> + netif_err(tp, probe, tp->netdev, "Unknown Device\n");

This hunk causes r8152 driver to refuse to bind to a device that was working
(AFAICT) before this change. This means every new version of the HW
requires a driver update for every kernel in every distribution that
incorporates this patch.

I've tried to "align" chromeos-3.18 branch with kernel.org r8152 driver:

I'm getting this output in dmesg:
r8152 2-1:1.0 (unnamed net_device) (uninitialized): Unknown version 0x6010
r8152 2-1:1.0 (unnamed net_device) (uninitialized): Unknown Device

Could you take a look at the hack I've uploaded for chromeos-3.18 kernel:
https://chromium-review.googlesource.com/348951

cheers,
grant



\
 
 \ /
  Last update: 2016-06-02 04:01    [W:0.108 / U:0.252 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site