- 17 Oct, 2015 40 commits
-
-
H Hartley Sweeten authored
Rename this register and its bit defines so they have namespace associated with the driver. Use the BIT macro to define the bits. Signed-off-by:
H Hartley Sweeten <hsweeten@visionengravers.com> Reviewed-by:
Ian Abbott <abbotti@mev.co.uk> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
H Hartley Sweeten authored
Rename this register and its bit defines so they have namespace associated with the driver. Use the BIT macro to define the bits. Signed-off-by:
-
H Hartley Sweeten authored
Rename this register and its bit defines so they have namespace associated with the driver. Use the BIT macro to define the bits. For aesthetics, rename the associated member of the private data. Signed-off-by:
-
H Hartley Sweeten authored
For aesthetics, convert the inline functions that return the bits used to set the DAC range and enable bits to macros. Signed-off-by:
-
H Hartley Sweeten authored
Rename the defines for the PCI BAR2 register offsets so they have namespace associated with the driver and convert the offsets to hex. Signed-off-by:
-
H Hartley Sweeten authored
Rename the defines for the PCI BAR3 register offsets so they have namespace associated with the driver and convert the offsets to hex. Signed-off-by:
-
H Hartley Sweeten authored
Instead of relying on the IS_UNIPOLAR mask value, use the comedi helper function to check if the range is unipolar. Signed-off-by:
-
H Hartley Sweeten authored
Convert the inline function DAC_DATA_REG() into a simple macro. Rename the defines for the PCI BAR4 register offsets so they have namespace associated with the driver and convert the offsets to hex. Signed-off-by:
-
H Hartley Sweeten authored
This member of the private data holds the PCI BAR4 resource starting address. For aesthetics, rename this member to 'pcibar4'. Signed-off-by:
-
H Hartley Sweeten authored
This member of the private data holds the PCI BAR2 resource starting address. For aesthetics, rename this member to 'pcibar2'. Signed-off-by:
-
H Hartley Sweeten authored
This member of the private data holds the PCI BAR1 resource starting address. For aesthetics, rename this member to 'pcibar1'. Signed-off-by:
-
H Hartley Sweeten authored
All the boards, except the 'pci-das1001', use the normal analog input range table 'cb_pcidas_ranges'. Convert the 'ranges' boardinfo into a bit-field flag 'use_alt_range' to save a bit of space. For aesthetics, reword the comments in the boardinfo for the bit-field members so they align. Signed-off-by:
-
H Hartley Sweeten authored
All the boards supported by this driver have 16 analog input channels. Remove this unnecessary information from the boardinfo. Signed-off-by:
-
H Hartley Sweeten authored
Only some of the boards supported by this driver have analog outputs but they always have 2 channels. Refactor the 'ao_nchan' member of the boardinfo into a bit-field flag 'has_ao' to save a bit of space. Also, convert the 'has_ao_fifo' member into a bit-field flag. Signed-off-by:
-
H Hartley Sweeten authored
The boards supported by this driver either have 16-bit or 12-bit analog input/output resolution. For aesthetics, replace the 'ai_bits' member of the boardinfo with a bit-field flag 'is_16bit'. Also, change the types of the other bit-field flags to unsigned int. Signed-off-by:
-
H Hartley Sweeten authored
These defines don't add any additional clarity to the driver. Just remove them. Signed-off-by:
-
H Hartley Sweeten authored
Reformat the multi-line comments in the kernel CodingStyle. Signed-off-by:
-
Sakshi Bansal authored
Fixed block comments usage of * on subsequent lines Signed-off-by:
Sakshi Bansal <sakshi.april5@gmail.com> Signed-off-by:
-
Ronit Halder authored
This patch replaces dma_alloc_coherent with the corresponding managed interface. Signed-off-by:
Ronit Halder <ronit.crj@gmail.com> Signed-off-by:
-
Christian Gromm authored
This patch prevents code from crossing the 80 character margin. Signed-off-by:
Christian Gromm <christian.gromm@microchip.com> Signed-off-by:
-
Christian Gromm authored
This patch changes the description of a function parameter for a better understanding. Signed-off-by:
-
Christian Gromm authored
This patch replaces memcpy() by the preferred function ether_addr_copy(). Signed-off-by:
-
Christian Gromm authored
This patch removes and fixes constants being misplaced in comparisons. Signed-off-by:
-
Christian Gromm authored
This patch fixes a couple of issues of the hdm-usb module found by checkpatch.pl. Signed-off-by:
-
Christian Gromm authored
This patch fixes the issues of HDM module i2c found by checkpatch.pl Signed-off-by:
-
Christian Gromm authored
This patch removes constants from comparisons and fixes checkpatch warnings about constants being put in the wrong place inside a comparison. Signed-off-by:
-
Christian Gromm authored
This patch fixes line-over-80-characters violation and removes the splitting of quoted strings. Signed-off-by:
-
Shivani Bhardwaj authored
The variable mbo is already NULL tested so, check on it should be removed. Semantic patch used: @@ expression E; statement S; @@ if(E==NULL) {... return ...;} - if(E) S Signed-off-by:Shivani Bhardwaj <shivanib134@gmail.com> Signed-off-by:
-
Gavin Thomas Claugus authored
Fix 80+ character line in cdev.c to stop checkpatch from whining. Signed-off-by:
Gavin Thomas Claugus <gclaugus@gmail.com> Signed-off-by:
-
Aybuke Ozdemir authored
Problem found using checkpatch.pl CHECK: Comparison to NULL could be written "(!)dcon_device" Signed-off-by:
Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by:
-
Aybuke Ozdemir authored
This patch "uint16_t" type instead of "u16" type was used. checkpatch.pl issue. Signed-off-by:
-
Aybuke Ozdemir authored
Remove multiple blank lines. Problem found using checkpatch.pl CHECK: Please don't use multiple blank lines. Signed-off-by:
-
Alison Schofield authored
Replace explicit NULL comparison and zero comparison test with ! operator to simplify code. Signed-off-by:
Alison Schofield <amsfield22@gmail.com> Signed-off-by:
-
Shivani Bhardwaj authored
struct bd is already NULL tested so, any further NULL checks should be removed. Signed-off-by:
-
Shivani Bhardwaj authored
Compiler can typecast variables implicitly so, explicit type cast is not required and should be removed. Semantic patch used: @@ type T; T e; identifier x; @@ * T x = (T)e; Signed-off-by:
-
Amitoj Kaur Chawla authored
Remove intialisation of a variable that is immediately reassigned. The semantic patch used to find this is: // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
Amitoj Kaur Chawla <amitoj1606@gmail.com> Signed-off-by:
-
Amitoj Kaur Chawla authored
Remove intialisation of a variable that is immediately reassigned. The semantic patch used to find this is: // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
-
Amitoj Kaur Chawla authored
Replace bit shifting on 1 with the BIT(x) Macro The semantic patch used to find this is: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
Amitoj Kaur Chawla authored
Remove intialisation of a variable that is immediately reassigned. The semantic patch used to find this is: // <smpl> @@ type T; identifier x; constant C; expression e; @@ T x - = C ; x = e; // </smpl> Signed-off-by:
-
Amitoj Kaur Chawla authored
Replace bit shifting on 1 with the BIT(x) Macro The semantic patch used to find this is: @@ int g; @@ -(1 << g) +BIT(g) Signed-off-by:
-
