lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Date:	Mon, 29 Dec 2014 10:31:50 -0800
From:	Dmitry Torokhov <dmitry.torokhov@...il.com>
To:	nick <xerofoify@...il.com>
Cc:	linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: Vendor Setup for  walkera_dev structure argument in the
 function,walkera0701_connect for the file,walera0701.c

Hi Nick,

On Mon, Dec 29, 2014 at 12:24:14AM -0500, nick wrote:
> Greetings Dmirty,
> I am wondering about the below code in the function,walkera701_connect in the file,walera0701.c.
> /* TODO what id vendor/product/version ? */
>          w->input_dev->id.vendor = 0x0001;
>          w->input_dev->id.product = 0x0001;
>          w->input_dev->id.version = 0x0100;
> As the maintainer I am wondering if you known the values these need to
> be set to as it seems a rather trivial patch for me to write if you
> can send me the correct values or a reference sheet.

I think we should not have initialized them at all as I don't think we
have good values for IDs for it. But now that we had done that, current
values are as good as any... Just leave the driver alone.

Also, as you mentioned, producing a patch is trivial. If you want to get
involved in kernel development you need to do some research and actually
_propose_ the changes (with explanation why the change is needed and why
you chose to make the change in a certain way), rather just asking
maintainers for explicit instructions on what changes to make.

Thanks.

-- 
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists