Re: [PATCH v2] hv: utils: enhance code for human read

2019-03-25 Thread Jesús Castro
On Mon, Mar 11, 2019 at 06:29:49PM -0600, Jesús Castro wrote: > The if/else block from hvt_op_open function can be written > as a switch/case block, and the bool issue_reset variable > is avoided. > > No functional change made. > > Suggested-by: Joe Perches > Sig

[PATCH v2] hv: utils: enhance code for human read

2019-03-11 Thread Jesús Castro
The if/else block from hvt_op_open function can be written as a switch/case block, and the bool issue_reset variable is avoided. No functional change made. Suggested-by: Joe Perches Signed-off-by: Jesús Castro --- v2: Refactoring hvt_op_open function with a switch/case block. Change

Re: [PATCH] hv: utils: fix coding style

2019-03-11 Thread Jesús Castro
> > Checkpatch script is showing a coding style error and is showing: > > > > ERROR: else should follow close brace '}' > > + } > > + else if (hvt->mode == HVUTIL_TRANSPORT_NETLINK) { > > > > This commit fixes the coding style, not a functional change. > [] > > diff --git a/drivers/hv/hv_util

[PATCH] hv: utils: fix coding style

2019-03-05 Thread Jesús Castro
Checkpatch script is showing a coding style error and is showing: ERROR: else should follow close brace '}' + } + else if (hvt->mode == HVUTIL_TRANSPORT_NETLINK) { This commit fixes the coding style, not a functional change. Signed-off-by: Jesús Castro --