linux-stable.git/drivers/staging/sep, branch linux-3.9.y Linux kernel stable tree staging: Remove unnecessary OOM messages 2013-02-11T18:10:33+00:00 Joe Perches joe@perches.com 2013-02-11T17:41:29+00:00 78110bb8dc4a7ff331bfa3cfe7d4e287cfb3f22b alloc failures already get standardized OOM messages and a dump_stack. For the affected mallocs around these OOM messages: Converted kzallocs with multiplies to kcalloc. Converted kmallocs with multiplies to kmalloc_array. Converted a kmalloc/strlen/strncpy to kstrdup. Moved a spin_lock below a removed OOM message and removed a now unnecessary spin_unlock. Neatened alignment and whitespace. Signed-off-by: Joe Perches <joe@perches.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
alloc failures already get standardized OOM
messages and a dump_stack.

For the affected mallocs around these OOM messages:

Converted kzallocs with multiplies to kcalloc.
Converted kmallocs with multiplies to kmalloc_array.
Converted a kmalloc/strlen/strncpy to kstrdup.
Moved a spin_lock below a removed OOM message and
removed a now unnecessary spin_unlock.
Neatened alignment and whitespace.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sep: remove assignment to i and j in sep_crypto_setup 2013-01-22T16:58:44+00:00 Devendra Naga devendra.aaru@gmail.com 2013-01-22T08:56:08+00:00 c095454e4b6acc8a5e7908647ec60d6088be90c8 the i and j are used in for loop and they assigned to zeros in the for loop, so no need to assign them to zeros again. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
the i and j are used in for loop and they assigned to zeros in the
for loop, so no need to assign them to zeros again.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sep: remove use of __devinit 2012-11-21T23:05:01+00:00 Bill Pemberton wfp5p@virginia.edu 2012-11-19T18:22:03+00:00 df1f35ebffc3739e43b96d1df689c2ecabf09552 CONFIG_HOTPLUG is going away as an option so __devinit is no longer needed. Signed-off-by: Bill Pemberton <wfp5p@virginia.edu> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.

Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sep: return -EFAULT on copy_from_user failure 2012-10-29T21:59:39+00:00 Devendra Naga devendra.aaru@gmail.com 2012-10-27T17:23:47+00:00 b58d6021d4dbc215dc975510bb2509e66e6061be copy_from_user() returns the number of bytes remaining to be copied, and it fails if user pointer is invalid, so return -EFAULT so that the user gets a FAULT. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
copy_from_user() returns the number of bytes remaining to be copied,
and it fails if user pointer is invalid, so return -EFAULT
so that the user gets a FAULT.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Staging: sep: Correct misspelled "remap_page_range" -> "remap_pfn_range" 2012-09-04T21:27:41+00:00 Robert P. J. Day rpjday@crashcourse.ca 2012-08-29T22:48:30+00:00 375d8886bc76871f012a337b4d6d753970d9fb36 Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sep_crypto.c: remove duplicated include 2012-07-20T00:20:06+00:00 Duan Jiong djduanjiong@gmail.com 2012-07-18T14:33:38+00:00 194ccca70ec29795ef8db006fd557a14783e734e Signed-off-by: Duan Jiong <djduanjiong@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
Signed-off-by: Duan Jiong <djduanjiong@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sep_main.c: remove duplicated include 2012-07-20T00:20:06+00:00 Duan Jiong djduanjiong@gmail.com 2012-07-18T14:30:30+00:00 33792df47ca5d9c654a5286dfb521b0ef81569ef Signed-off-by: Duan Jiong <djduanjiong@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
Signed-off-by: Duan Jiong <djduanjiong@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging/sep: use module_pci_driver macro 2012-07-12T00:26:01+00:00 Devendra Naga devendra.aaru@gmail.com 2012-07-10T06:41:48+00:00 8e4763f67fe7279822c131bde4eadf5f7e456b2c this driver duplicates the module_pci_driver code, so remove the duplicated code and use module_pci_driver and also remove the obvious comments about _init and _exit. Signed-off-by: Devendra Naga <devendra.aaru@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
this driver duplicates the module_pci_driver code, so remove the
duplicated code and use module_pci_driver and also remove the
obvious comments about _init and _exit.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sep: sep_crypto.c: Remove useless functions sep_dump and sep_dump_sg 2012-07-06T23:14:21+00:00 Marcos Paulo de Souza marcos.souza.org@gmail.com 2012-06-29T04:19:23+00:00 0e7e10fe49213ba4e9dc8a74615b81dd8a57ba2a These two functions only has commented code, so remove it. Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
These two functions only has commented code, so remove it.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
staging: sep: sep_crypto.c: Remove useless function crypto_sep_dump_message 2012-07-06T23:14:20+00:00 Marcos Paulo de Souza marcos.souza.org@gmail.com 2012-06-29T04:19:22+00:00 68cacda10dc0fd1e57153055473c4030873bcea1 The function crypto_sep_dump_message don't have any use in this driver. So remove it. Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> 
The function crypto_sep_dump_message don't have any use in this driver. So remove it.

Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>