Re: [PATCH 2/3] usb: type-c: USB Type-C Connector System Software Interface

From: Andy Shevchenko
Date: Thu Feb 11 2016 - 03:26:37 EST


On Wed, Feb 10, 2016 at 5:11 PM, BjÃrn Mork <bjorn@xxxxxxx> wrote:
> Andy Shevchenko <andy.shevchenko@xxxxxxxxx> writes:
>> On Wed, Feb 10, 2016 at 3:21 PM, Oliver Neukum <oneukum@xxxxxxxx> wrote:
>>> On Wed, 2016-02-10 at 13:56 +0200, Andy Shevchenko wrote:
>>>> > +out:
>>>>
>>>> CodingStyle suggests to do a better label naming.

[...]

> Exactly what is it about "out" that is unclear to you here? Could you
> propose a better alternative if you seriously mean that this needs to be
> changed?

Those switches of course an extra stuff, but I would propose
out_connect:

To me out: sounds out, like printing error and return code, or
something like that. Here the case is different.

And since we have two full switches it might be hard to have on one
screen out code and exact goto. See my point now?

--
With Best Regards,
Andy Shevchenko