Hi, On 05/03/2016 07:49 PM, Mark Brown wrote: > On Tue, May 03, 2016 at 09:43:58AM +0800, Lu Baolu wrote: >> On 05/02/2016 07:00 PM, Mark Brown wrote: >>> On Fri, Apr 29, 2016 at 02:26:32PM +0800, Lu Baolu wrote: >>>> + gpiod = gpiod_get(dev, "vbus_en", GPIOD_ASIS); >>>> + if (IS_ERR(gpiod)) >>>> + return PTR_ERR(gpiod); >>> This is clearly an inappropriate name for the signal in generic code, >>> it's specific to your use case. >> I will change the gpio name to "gpio". Is that okay? > Yes, that looks good (and lines up with DT so hopefully the code can be > shared). Fair enough. Thanks. Best regards, Lu Baolu -- 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