[PATCH v3 3/4] staging: emxx_udc: Remove unecessary temporary variable

2016-12-20 Thread Afonso Bordado
This improves code readability. Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index 0963533..45808ed 100644

[PATCH v3 1/4] staging: emxx_udc: Fix CamelCase function name

2016-12-20 Thread Afonso Bordado
Change EP0_out_PIO to use the kernel convention. Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index 3f42fa8..51ad04b 100644

[PATCH v3 0/4] staging: emxx_udc: Fix checkpatch.pl CamelCase issues

2016-12-20 Thread Afonso Bordado
Fix checkpatch.pl issues with CamelCase. Improves readability by removing temporary variables. Afonso Bordado (4): staging: emxx_udc: Fix CamelCase function name staging: emxx_udc: Rename CamelCase variable staging: emxx_udc: Remove unecessary temporary variable staging: emxx_udc: Fix

[PATCH v3 4/4] staging: emxx_udc: Fix CamelCase variable name

2016-12-20 Thread Afonso Bordado
Changes from CamelCase to a kernel format Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index 45808ed..c19ce17 100644

[PATCH v3 2/4] staging: emxx_udc: Rename CamelCase variable

2016-12-20 Thread Afonso Bordado
The new name complies with the kernel styling guidelines and is more descriptive. Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging

[PATCH v2] staging: emxx_udc: Fix styling issues

2016-12-20 Thread Afonso Bordado
Fix CamelCase styling issues with EP0_out_PIO Remove unecessary temporary variables Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 29 - 1 file changed, 12 insertions(+), 17 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b

[PATCH v2] staging: emxx_udc: Fix styling issues

2016-12-18 Thread Afonso Bordado
Fix CamelCase styling issues with EP0_out_PIO Remove unecessary temporary variables Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 29 - 1 file changed, 12 insertions(+), 17 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b

Re: [PATCH] staging: emxx_udc: Fix CamelCase styling issue

2016-12-18 Thread Afonso Bordado
> Instead of merely converting Hungarian CamelCase to lowercase > with underscores where word transitions occurred, try reading > the code and making sense of what it does to perhaps find a > better variable name instead. Maybe eliminate the variable > altogether. > > iWordLength could be reads or

[PATCH] staging: emxx_udc: Fix CamelCase styling issue

2016-12-18 Thread Afonso Bordado
Converts from CamelCase to the recommended style. Signed-off-by: Afonso Bordado --- drivers/staging/emxx_udc/emxx_udc.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/emxx_udc/emxx_udc.c b/drivers/staging/emxx_udc/emxx_udc.c index 3f42fa8