[PATCH] return same error value from spk_set_key_info

2017-02-21 Thread Pranay Kr. Srivastava
This patch makes spk_set_key_info return -EINVAL in case of failure instead of returning 4 different values for the type of error that occurred. Retain the previous error values as debug messages instead. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 24

[PATCH Speakup v2] return same error value from spk_set_key_info

2017-02-21 Thread Pranay Kr. Srivastava
This patch makes spk_set_key_info return -EINVAL in case of failure instead of returning 4 different values for the type of error that occurred. Print the offending values instead as debug message. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 28

[PATCH] remove unnecessary initial allocation of vc

2017-02-21 Thread Pranay Kr. Srivastava
This patch removes the unnecessary allocation of current foreground vc during initialization. Also change the prototype of speakup_allocate to take extra argument of gfp_* flags. Thus not requiring GFP_ATOMIC during initialization. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging

[no subject]

2017-02-21 Thread Pranay Kr. Srivastava
Sending both patches as series instead. Made the required changes as suggessted in earlier versions and fixed the warnings reported from kbuild test robot. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailm

[PATCH-SPEAKUP 2/2] remove unecessary initial allocation of vc

2017-02-21 Thread Pranay Kr. Srivastava
requiring GFP_ATOMIC during initialization. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 19 --- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup/main.c index 6667cf2..f81a936 100644

[PATCH-SPEAKUP 1/2] return same error value from spk_set_key_info

2017-02-21 Thread Pranay Kr. Srivastava
This patch makes spk_set_key_info return -EINVAL in case of failure instead of returning 4 different values for the type of error that occurred. Print the offending values instead as debug message. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 27

[PATCH SPEAKUP v2 0/3] cleanup error and initilization

2017-02-28 Thread Pranay Kr. Srivastava
Changelog from v1: 1. fixed kbuild warning for i386 build as reported by kbuild robot 2. split initialization code in two patches. Pranay Kr. Srivastava (3): return same error value from spk_set_key_info remove unecessary initial allocation of vc use spkeaup_allocate as per

[PATCH SPEAKUP v2 3/3] use spkeaup_allocate as per required context

2017-02-28 Thread Pranay Kr. Srivastava
speakup_allocate used GFP_ATOMIC for allocations even while during initialization due to it's use in notifier call. Pass GFP_ flags as well to speakup_allocate depending on the context it is called in. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 8 1

[PATCH SPEAKUP v2 1/3] return same error value from spk_set_key_info

2017-02-28 Thread Pranay Kr. Srivastava
This patch makes spk_set_key_info return -EINVAL in case of failure instead of returning 4 different values for the type of error that occurred. Print the offending values instead as debug message. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 27

[PATCH SPEAKUP v2 2/3] remove unecessary initial allocation of vc

2017-02-28 Thread Pranay Kr. Srivastava
This patch removes the unnecessary allocation of current foreground vc during initialization. This initialization is already handled in the loop that follows it for all available virtual consoles. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 11 --- 1 file

[PATCH staging/speakup v3 0/3] cleanup error and initilization

2017-03-21 Thread Pranay Kr. Srivastava
Changelog from v2: Fixed the message subject line. Changelog from v1: 1. fixed kbuild warning for i386 build as reported by kbuild robot 2. split initialization code in two patches. Pranay Kr. Srivastava (3): return same error value from spk_set_key_info remove

[PATCH staging/speakup v3 2/3] remove unnecessary initial allocation of vc

2017-03-21 Thread Pranay Kr. Srivastava
This patch removes the unnecessary allocation of current foreground vc during initialization. This initialization is already handled in the loop that follows it for all available virtual consoles. Signed-off-by: Pranay Kr. Srivastava Reviewed-by: Samuel Thibault --- drivers/staging/speakup

[PATCH staging/speakup v3 1/3] return same error value from spk_set_key_info

2017-03-21 Thread Pranay Kr. Srivastava
This patch makes spk_set_key_info return -EINVAL in case of failure instead of returning 4 different values for the type of error that occurred. Print the offending values instead as debug message. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 27

[PATCH staging/speakup v3 3/3] use speakup_allocate as per required context

2017-03-21 Thread Pranay Kr. Srivastava
speakup_allocate used GFP_ATOMIC for allocations even while during initialization due to it's use in notifier call. Pass GFP_ flags as well to speakup_allocate depending on the context it is called in. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 8 1

[[RESEND]PATCH staging/speakup v3 3/3] use speakup_allocate as per required context

2017-03-24 Thread Pranay Kr. Srivastava
speakup_allocate used GFP_ATOMIC for allocations even while during initialization due to it's use in notifier call. Pass GFP_ flags as well to speakup_allocate depending on the context it is called in. Signed-off-by: Pranay Kr. Srivastava --- drivers/staging/speakup/main.c | 8 1