Re: [Xen-devel] [PATCH] xenbus: Neaten xenbus_va_dev_error

2017-02-08 Thread Joe Perches
On Wed, 2017-02-08 at 12:14 -0500, Boris Ostrovsky wrote: > On 02/08/2017 11:05 AM, Joe Perches wrote: > > On Wed, 2017-02-08 at 10:33 -0500, Boris Ostrovsky wrote: > > > On 02/08/2017 06:33 AM, Joe Perches wrote: > > > > This function error patch can be simplified, so do so. > > > > > > > > Remov

Re: [Xen-devel] [PATCH] xenbus: Neaten xenbus_va_dev_error

2017-02-08 Thread Boris Ostrovsky
On 02/08/2017 11:05 AM, Joe Perches wrote: > On Wed, 2017-02-08 at 10:33 -0500, Boris Ostrovsky wrote: >> On 02/08/2017 06:33 AM, Joe Perches wrote: >>> This function error patch can be simplified, so do so. >>> >>> Remove fail: label and somewhat obfuscating, used once "error_path" >>> function. >

Re: [Xen-devel] [PATCH] xenbus: Neaten xenbus_va_dev_error

2017-02-08 Thread Joe Perches
On Wed, 2017-02-08 at 10:33 -0500, Boris Ostrovsky wrote: > On 02/08/2017 06:33 AM, Joe Perches wrote: > > This function error patch can be simplified, so do so. > > > > Remove fail: label and somewhat obfuscating, used once "error_path" > > function. btw: I left it alone, but likely #define PRI

Re: [Xen-devel] [PATCH] xenbus: Neaten xenbus_va_dev_error

2017-02-08 Thread Boris Ostrovsky
On 02/08/2017 06:33 AM, Joe Perches wrote: > This function error patch can be simplified, so do so. > > Remove fail: label and somewhat obfuscating, used once "error_path" > function. > > Signed-off-by: Joe Perches Applied to for-linus-4.11. -boris __