linux-stable.git/drivers/staging/dgrp, branch linux-3.14.y

Revert "Staging: dgrp: Refactor the function dgrp_receive() in drrp_net_ops.c"

2014-02-07T17:16:38+00:00

This reverts commit b73db54750482cf3910046c82a84ce8c1684dfbe.

Reported-by: Geert Uytterhoeven 
Reported-by: Chen Gang 
Cc: Rashika Kheria 
Cc: James Hogan 
Signed-off-by: Greg Kroah-Hartman

staging: delete non-required instances of include

2013-12-17T18:08:14+00:00

None of these files are actually using any __init type directives
and hence don't need to include .  Most are just a
left over from __devinit and __cpuinit removal, or simply due to
code getting copied from one driver to the next.

Signed-off-by: Paul Gortmaker 
Signed-off-by: Greg Kroah-Hartman

staging: dgrp: Fix space prohibited before semicolon

2013-12-17T17:50:11+00:00

This patch fixed "WARNING: space prohibited before semicoloni"
found by checkpatch.pl within dgrp_tty.c

Signed-off-by: Masanari Iida 
Signed-off-by: Greg Kroah-Hartman

Staging: dgrp: Refactor the function dgrp_receive() in drrp_net_ops.c

2013-11-11T23:52:00+00:00

The function dgrp_receive() in dgrp_net_ops.c is too long and can be
refactored. It uses various switch statements and goto labels. I have
removed a label called data and tried to extract a new function out of
it called as handle_data_in_packet().

This helps to make the code more modularize and simple to read and
understand.

Signed-off-by: Rashika Kheria 
Signed-off-by: Greg Kroah-Hartman

staging: dgrp: Remove casting the return value which is a void pointer

2013-09-17T14:47:43+00:00

Casting the return value which is a void pointer is redundant.
The conversion from void pointer to any other pointer type is
guaranteed by the C programming language.

Signed-off-by: Jingoo Han 
Signed-off-by: Greg Kroah-Hartman

Merge tag 'tty-3.12-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty

2013-09-03T18:38:36+00:00

Pull tty/serial driver patches from Greg KH:
 "Here's the big tty/serial driver pull request for 3.12-rc1.

  Lots of n_tty reworks to resolve some very long-standing issues,
  removing the 3-4 different locks that were taken for every character.
  This code has been beaten on for a long time in linux-next with no
  reported regressions.

  Other than that, a range of serial and tty driver updates and
  revisions.  Full details in the shortlog"

* tag 'tty-3.12-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty: (226 commits)
  hvc_xen: Remove unnecessary __GFP_ZERO from kzalloc
  serial: imx: initialize the local variable
  tty: ar933x_uart: add device tree support and binding documentation
  tty: ar933x_uart: allow to build the driver as a module
  ARM: dts: msm: Update uartdm compatible strings
  devicetree: serial: Document msm_serial bindings
  serial: unify serial bindings into a single dir
  serial: fsl-imx-uart: Cleanup duplicate device tree binding
  tty: ar933x_uart: use config_enabled() macro to clean up ifdefs
  tty: ar933x_uart: remove superfluous assignment of ar933x_uart_driver.nr
  tty: ar933x_uart: use the clk API to get the uart clock
  tty: serial: cpm_uart: Adding proper request of GPIO used by cpm_uart driver
  serial: sirf: fix the amount of serial ports
  serial: sirf: define macro for some magic numbers of USP
  serial: icom: move array overflow checks earlier
  TTY: amiserial, remove unnecessary platform_set_drvdata()
  serial: st-asc: remove unnecessary platform_set_drvdata()
  msm_serial: Send more than 1 character on the console w/ UARTDM
  msm_serial: Add support for non-GSBI UARTDM devices
  msm_serial: Switch clock consumer strings and simplify code
  ...

staging: dgrp: add init/exit macros

2013-08-19T22:31:58+00:00

add __init/__exit macros to related init/cleanup functions.

Signed-off-by: navin patidar 
Signed-off-by: Greg Kroah-Hartman

tty: Ensure single-threaded flip buffer consumer with mutex

2013-07-23T23:47:09+00:00

The buffer work may race with parallel tty_buffer_flush. Use a
mutex to guarantee exclusive modify access to the head flip
buffer.

Remove the unneeded spin lock.

Signed-off-by: Peter Hurley 
Signed-off-by: Greg Kroah-Hartman

drivers: avoid format string in dev_set_name

2013-07-03T23:07:41+00:00

Calling dev_set_name with a single paramter causes it to be handled as a
format string.  Many callers are passing potentially dynamic string
content, so use "%s" in those cases to avoid any potential accidents,
including wrappers like device_create*() and bdi_register().

Signed-off-by: Kees Cook 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

drivers: staging: dgrp: using time_is_before_jiffies() instead of open coding 'jiffies'

2013-06-09T05:20:59+00:00

Need use the proper jiffy comparison macros/functions instead of open
coding 'jiffies'.

The original 'comparation' is always true.

The related warning (with 'EXTRA_CFLAGS=-W')
  drivers/staging/dgrp/dgrp_net_ops.c:1610:10: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]

Signed-off-by: Chen Gang 
Signed-off-by: Greg Kroah-Hartman