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 linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 18 Aug 2016 10:25:55 +0900 From: Chanwoo Choi <cw00.choi@...sung.com> To: Guenter Roeck <groeck@...gle.com> Cc: Guenter Roeck <linux@...ck-us.net>, MyungJoo Ham <myungjoo.ham@...sung.com>, linux-kernel <linux-kernel@...r.kernel.org>, Guenter Roeck <groeck@...omium.org>, Chris Zhong <zyw@...k-chips.com>, "cpgs (cpgs@...sung.com)" <cpgs@...sung.com> Subject: Re: [PATCH] extcon: Introduce EXTCON_PROP_USB_SUPERSPEED property On 2016년 08월 18일 09:41, Guenter Roeck wrote: > On Wed, Aug 17, 2016 at 5:37 PM, Chanwoo Choi <cw00.choi@...sung.com> wrote: >> Hi Guenter, >> >> How about changing the property name for SuperSpeed as following? >> It is more simple and reduce the length of property name. >> - EXTCON_PROP_USB_SUPERSPEED -> EXTCON_PROP_USB_SS >> > Whatever you prefer is fine. OK. Thanks. I'll change the property name. Regards, Chanwoo Choi > > Thanks, > Guenter > >> Regards, >> Chanwoo Choi >> >> On 2016년 08월 16일 10:41, Guenter Roeck wrote: >>> On Mon, Aug 15, 2016 at 5:55 PM, Chanwoo Choi <cw00.choi@...sung.com> wrote: >>>> Hi Guenter, >>>> >>>> Looks good to me. >>>> I'll add the reference[1] information on patch description and applied it. >>>> [1] https://en.wikipedia.org/wiki/USB#Overview >>>> >>> >>> Thanks! >>> >>> Guenter >>> >>>> Thanks, >>>> Chanwoo Choi >>>> >>>> On 2016년 08월 15일 22:15, Guenter Roeck wrote: >>>>> From: Guenter Roeck <groeck@...omium.org> >>>>> >>>>> EXTCON_PROP_USB_SUPERSPEED is necessary to distinguish between USB/USB2 >>>>> and USB3 connections on USB Type-C cables. >>>>> >>>>> Cc: Chris Zhong <zyw@...k-chips.com> >>>>> Signed-off-by: Guenter Roeck <groeck@...omium.org> >>>>> --- >>>>> Applies on top of extcon-next. >>>>> >>>>> include/linux/extcon.h | 8 +++++++- >>>>> 1 file changed, 7 insertions(+), 1 deletion(-) >>>>> >>>>> diff --git a/include/linux/extcon.h b/include/linux/extcon.h >>>>> index ad7a1606a7f3..38d2c0dec2c1 100644 >>>>> --- a/include/linux/extcon.h >>>>> +++ b/include/linux/extcon.h >>>>> @@ -107,12 +107,18 @@ >>>>> * @type: integer (intval) >>>>> * @value: 0 (normal) or 1 (flip) >>>>> * @default: 0 (normal) >>>>> + * - EXTCON_PROP_USB_SUPERSPEED >>>>> + * @type: integer (intval) >>>>> + * @value: 0 (USB/USB2) or 1 (USB3) >>>>> + * @default: 0 (USB/USB2) >>>>> + * >>>>> */ >>>>> #define EXTCON_PROP_USB_VBUS 0 >>>>> #define EXTCON_PROP_USB_TYPEC_POLARITY 1 >>>>> +#define EXTCON_PROP_USB_SUPERSPEED 2 >>>>> >>>>> #define EXTCON_PROP_USB_MIN 0 >>>>> -#define EXTCON_PROP_USB_MAX 1 >>>>> +#define EXTCON_PROP_USB_MAX 2 >>>>> #define EXTCON_PROP_USB_CNT (EXTCON_PROP_USB_MAX - EXTCON_PROP_USB_MIN + 1) >>>>> >>>>> /* Properties of EXTCON_TYPE_CHG. */ >>>>> >>>> >>> >>> >>> >> > > >
Powered by blists - more mailing lists