- 30 Aug, 2014 20 commits
-
-
Mario Schlegel authored
fixed a coding style issue Signed-off-by:
Mario Schlegel <m.schlegel@posteo.de> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Oleg Drokin authored
Client policies do not make much sense to have in client-only code, so remove all references to everything but fifo. This also removes lustre/libcfs/heap.c and its header, since it was only used by certain request policies (crr and orr). Signed-off-by:
Oleg Drokin <green@linuxhacker.ru> Signed-off-by:
-
Oleg Drokin authored
Also unused header file lustre_linkea.h Signed-off-by:
-
Oleg Drokin authored
This contains server-side llog operations. Also remove declaration of public structure from lustre_log.h Signed-off-by:
-
Oleg Drokin authored
It's only used by server side luster for on-disk structure swabbing. And also remove forward declarations for its functions from md_object.h and lustre_idl.h Signed-off-by:
-
Julia Lawall authored
The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // </smpl> Signed-off-by:
Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by:
-
Julia Lawall authored
This file contains unresolved references and is not mentioned in the Makefile. Signed-off-by:
-
Julia Lawall authored
This file contains unresolved references and is not mentioned in the Makefile. Signed-off-by:
-
Julia Lawall authored
The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // </smpl> Signed-off-by:
-
Julia Lawall authored
The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // </smpl> Signed-off-by:
-
Julia Lawall authored
The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ identifier i; constant c; @@ return - ( \(i\|-i\|i(...)\|c\) - ) ; // </smpl> Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl space required after that ',' errors Signed-off-by:
Greg Donald <gdonald@gmail.com> Signed-off-by:
-
Julia Lawall authored
Use c99 initializers for structures. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @decl@ identifier i1,fld; type T; field list[n] fs; @@ struct i1 { fs T fld; ...}; @bad@ identifier decl.i1,i2; expression e; initializer list[decl.n] is; @@ struct i1 i2 = { is, + .fld = e - e ,...}; // </smpl> Signed-off-by:
-
Greg Donald authored
Fix checkpatch.pl that open brace { should be on the previous line errors Signed-off-by: -
Greg Donald authored
Fix checkpatch.pl space required before the open parenthesis '(' errors Signed-off-by: -
Greg Donald authored
Fix checkpatch.pl "foo * bar" should be "foo *bar" errors Signed-off-by:
-
Hema Prathaban authored
This patch fixes the following error using checkpatch.pl ERROR: inline keyword should sit between storage class and type Signed-off-by:
Hema Prathaban <hemaklnce@gmail.com> Signed-off-by:
-
Phong Tran authored
This patch fixes some non static function declarations that cause sparse warning. Tested by compilation. Signed-off-by:
Phong Tran <tranmanphong@gmail.com> Signed-off-by:
-
Oleg Drokin authored
This adds drivers/staging/lustre/README.txt with some basic information about Lustre and how to use it. Eventually this will need to move to Documentation/filesystems/lustre.txt Signed-off-by:
-
Oleg Drokin authored
Just add the entry with some info. Signed-off-by:
-
- 26 Aug, 2014 11 commits
-
-
Andrew Morton authored
drivers/staging/unisys/visorchipset/file.c:31: error: redefinition of typedef 'GUEST_PHYSICAL_ADDRESS' drivers/staging/unisys/common-spar/include/channels/controlvmchannel.h:23: note: previous declaration of 'GUEST_PHYSICAL_ADDRESS' was here Cc: Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by:
Andrew Morton <akpm@linux-foundation.org> Signed-off-by:
-
Honggang Li authored
drivers/staging/unisys/virthba/virthba.c:520:2: warning: format ‘%u’ expects argument of type ‘unsigned int’, but argument 4 has type ‘u64’ [-Wformat=] drivers/staging/unisys/virthba/virthba.c:749:2: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘u64’ [-Wformat=] drivers/staging/unisys/virthba/virthba.c:1142:4: warning: format ‘%d’ expects argument of type ‘int’, but argument 7 has type ‘u64’ [-Wformat=] drivers/staging/unisys/virthba/virthba.c:1151:5: warning: format ‘%d’ expects argument of type ‘int’, but argument 5 has type ‘u64’ [-Wformat=] After apply the patch: linux-next]$ make C=1 drivers/staging/unisys/virthba/virthba.o CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrelease.h CALL scripts/checksyscalls.sh CHECK drivers/staging/unisys/virthba/virthba.c CC [M] drivers/staging/unisys/virthba/virthba.o Signed-off-by:
Honggang Li <enjoymindful@gmail.com> Signed-off-by:
-
Benjamin Romer authored
There are several else clauses that aren't necessary listed by checkpatch.pl in visorchipset_main.c. Remove these. Signed-off-by:
Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by:
-
Benjamin Romer authored
Fix the missing blank lines between declarations and code in visorchipset_main.c. Signed-off-by:
-
Benjamin Romer authored
Fix the two spacing errors in parser.c. Signed-off-by:
-
Benjamin Romer authored
There's a missing blank line between code and declarations in file.c. Signed-off-by:
-
Benjamin Romer authored
The function uisutil_copy_fragsinfo_from_skb() in uisutils.c is messy and over-indented. This patch fixes the function's indentation so it is cleaner. Signed-off-by:
-
Benjamin Romer authored
Fix the missing blank line between code and declarations in uisthread.c. Signed-off-by:
-
Benjamin Romer authored
Remove the extraneous return from the uislib module exit function. Signed-off-by:
-
Benjamin Romer authored
The destroy_bus() function is heavily indented and many lines extend past the right margin. This patch simplifies the function without changing its effects. Signed-off-by:
-
Benjamin Romer authored
Fix the line spacing between code and local variables in uislib.c. Signed-off-by:
-
- 25 Aug, 2014 1 commit
-
-
git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iioGreg Kroah-Hartman authored
Jonathan writes: 1st round of new IIO drivers, functionality and cleanups for the 3.18 cycle. Maintainer Updates * Add 3 designated reviewers for IIO. Lars, Peter and Hartmut have been actively reviewing a lot of patches for a while now so this reflects the status quo. These three are probably the only reason I keep my head above the water! New drivers and device support * max5821 DAC * Rockchip SARADC * TI ADC128S052 ADC * BMC150 Accelerometer * exynos ADC driver gains support for s3c24xx and s3c64xx parts. * kxcjk-1013 gainst range control and runtime PM support to drive down it's power usage. Driver removals * Drop ad5930, ad99850, ad9852, ad9910 and ad9951 drivers on the simple basis that they drivers just provided a register write function with no compliant user space ABI whatsoever. Much better to drop them and start again for these in the fullness of time. Core Enhancements * Join together neighbouring elements in the demux units that feeds the binary interfaces. This cuts down on the number of individual copies needed when splitting out individual channels from the incoming channel scans. * Other demux related cleanups such as using roundup instead of a local implementation. Cleanups * Drop an unnecessary double setting of the owner field in xilinx adc. * Some more patches to use managed (devm) interfaces to cut down on complexity of removal code. * adis16060 coding style fixlets. * Fix some incorrect error returns in the Xilinx ADC driver. * Coding style fixlets for various accelerometer drivers. * Some sparse warning fixes to do with endianness and sign of variables. * Fix an incorrect and entirely pointless use of sizeof on a dynamic pointer in hid-sensor-magn-3d by dropping the relevant code.
-
- 19 Aug, 2014 4 commits
-
-
Michal Simek authored
There is no need to init .owner field. Based on the patch from Peter Griffin <peter.griffin@linaro.org> "mmc: remove .owner field for drivers using module_platform_driver" This patch removes the superflous .owner field for drivers which use the module_platform_driver API, as this is overriden in platform_driver_register anyway." Signed-off-by:
Michal Simek <michal.simek@xilinx.com> Signed-off-by:
Jonathan Cameron <jic23@kernel.org>
-
Lars-Peter Clausen authored
All what these 'drivers' do is expose a single (non standard ABI) sysfs attribute that when written to does a direct pass-through to spi_write(). This is rather ugly and does not justify the existence of a driver as the same can easily done by using the spidev interface. The drivers will eventually be rewritten as proper IIO ABI compliant drivers which do have the proper abstraction layers between userspace and the device. But in the meantime these driver do not add any extra value and just clutter up the staging area. So just remove them. Signed-off-by:
Lars-Peter Clausen <lars@metafoo.de> Signed-off-by:
-
Lars-Peter Clausen authored
Add Michael and myself as the maintainer for the Analog Devices IIO drivers. The entry matches on all files in drivers/staging/iio and drivers/iio/ starting with the 'ad' prefix, except for 'adjd' as that one is used by Avago Technologies. Signed-off-by:
-
Lars-Peter Clausen authored
Add those persons who generally tend to review new IIO patches to the list of designated reviewers to make sure that they are Cc'ed on new patches. Signed-off-by:
Peter Meerwald <pmeerw@pmeerw.net> Acked-by:
Hartmut Knaack <knaack.h@gmx.de> Signed-off-by:
-
- 17 Aug, 2014 4 commits
-
-
Konrad Zapalowicz authored
The dgnc_Major_Control_Registered variable purpose was to act as a flag to indicate if the character device has been successfully registered into the kernel. This flag was later checked in the module cleanup function to know if the character device needs to be deregistered. However the {device,class}_destroy and unregister_chrdev functions may be called with 'invalid' data perfectly fine. This means that this variable is not needed and can safely be removed which is what this commit does. Signed-off-by:Konrad Zapalowicz <bergo.torino@gmail.com> Signed-off-by:
-
Konrad Zapalowicz authored
The dgnc_driver_start variable purpose was to indicate if the driver 'start' routine has been called. Now, because the 'start' routine can only be called once this variable is not needed thus this commit removes it. Signed-off-by:
-
Konrad Zapalowicz authored
The polling counter variable is only exposed via sysfs and has no other purpose. Now, since the polling shall be implemented as a board specific feature rather than being global in the driver this counter is obsolete. Signed-off-by:
-
Konrad Zapalowicz authored
This commit removes the driver's global state variable. This is ok because the state was changed only once at the end of init phase thus the future usage of this variable is pointless. Signed-off-by:
-
