[PATCH v3 0/2] Staging: speakup: Misc cleanups and code reformatting.

[PATCH v3 0/2] Staging: speakup: Misc cleanups and code reformatting.

[Sam Muhammed]

Changes to clear Checkpatch.pl CHECKs of
proper use of kmalloc(), and adding identifier names
to function declaration arguments.

All modifications made to match coding style and
to improve readability.

Changes in v3:
- Adding Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
  for both patches.

Changes in v2:
- Drop the first patch in the previous version:
        Staging: speakup: Avoid multiple assignments.
  This code is better with multiple assignments.

Sam Muhammed (2):
  Staging: speakup: Use sizeof(*var) in kmalloc().
  Staging: speakup: Add identifier name to function declaration
    arguments.

 drivers/staging/speakup/spk_ttyio.c | 2 +-
 drivers/staging/speakup/spk_types.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

---
2.20.1

[PATCH v3 1/2] Staging: speakup: Use sizeof(*var) in kmalloc().

[Sam Muhammed]

Modifying struct allocation in kmalloc() to match the
coding standards.

Checkpatch.pl CHECK: Prefer kmalloc(sizeof(*ldisc_data)...)
over kmalloc(sizeof(struct spk_ldisc_data)...)

Signed-off-by: Sam Muhammed <jane.pnx9@gmail.com>
Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
---
Changes in v3:
- Adding Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
  tag.

Changes in v2:
- None.

 drivers/staging/speakup/spk_ttyio.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/speakup/spk_ttyio.c b/drivers/staging/speakup/spk_ttyio.c
index 5a9eff08cb96..9b95f77f9265 100644
--- a/drivers/staging/speakup/spk_ttyio.c
+++ b/drivers/staging/speakup/spk_ttyio.c
@@ -51,7 +51,7 @@ static int spk_ttyio_ldisc_open(struct tty_struct *tty)
 		return -EOPNOTSUPP;
 	speakup_tty = tty;

-	ldisc_data = kmalloc(sizeof(struct spk_ldisc_data), GFP_KERNEL);
+	ldisc_data = kmalloc(sizeof(*ldisc_data), GFP_KERNEL);
 	if (!ldisc_data)
 		return -ENOMEM;

---
2.20.1

[PATCH v3 2/2] Staging: speakup: Add identifier name to function declaration arguments.

[Sam Muhammed]

void (*read_buff_add) argument didn't have an identifier name,
adding a name to it like the rest of all functions' arguments.

Signed-off-by: Sam Muhammed <jane.pnx9@gmail.com>
Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
---
Changes in v3:
- Adding Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
  tag.

Changes in v2:
- None.

 drivers/staging/speakup/spk_types.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/speakup/spk_types.h b/drivers/staging/speakup/spk_types.h
index a2fc72c29894..fc6a9416829c 100644
--- a/drivers/staging/speakup/spk_types.h
+++ b/drivers/staging/speakup/spk_types.h
@@ -189,7 +189,7 @@ struct spk_synth {
 	void (*flush)(struct spk_synth *synth);
 	int (*is_alive)(struct spk_synth *synth);
 	int (*synth_adjust)(struct st_var_header *var);
-	void (*read_buff_add)(u_char);
+	void (*read_buff_add)(u_char c);
 	unsigned char (*get_index)(struct spk_synth *synth);
 	struct synth_indexing indexing;
 	int alive;
---
2.20.1