- 25 Oct, 2015 40 commits
-
-
Amitoj Kaur Chawla authored
This patch removes unnecessary braces around if-else single statement blocks and unnecessary tab space aligning Tool used is uncrustify. Signed-off-by:
Amitoj Kaur Chawla <amitoj1606@gmail.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Shraddha Barke authored
Introduce the use of function put_unaligned_le32. A simplified version of Coccinelle semantic patch used - @@ identifier tmp; expression ptr; expression y,e; type T; @@ - tmp = cpu_to_le32(y); <+... when != tmp - memcpy(ptr, (T)&tmp, ...); + put_unaligned_le32(y,ptr); ...+> ? tmp = e @@ type T; identifier tmp; @@ - T tmp; ...when != tmp Signed-off-by:
Shraddha Barke <shraddha.6596@gmail.com> Signed-off-by:
-
Dennis Menschel authored
This patch fixes some internal voltage settings for the C-Berry28 display. The original example source files for the C-Berry28 as provided by its vendor admatec contained six issues where a command parameter's value didn't match its corresponding comment. I've informed admatec about these discrepancies on 2015-08-25. In the meantime, I've assumed the comments to be more reliable than the code, and thus have used these values to write the initial C-Berry28 driver. After having received a reply from admatec on 2015-10-16 that the issues have been fixed in their example code, it has turned out that for the voltages VCOM and AVDD, the code was indeed correct, but the comments were wrong. This patch is meant to fix these two pending mistakes. Signed-off-by:
Dennis Menschel <menschel-d@posteo.de> Signed-off-by:
-
Dennis Menschel authored
This patch converts the default init sequence of the ST7789V display controller into an init function, as init sequences are considered deprecated by the maintainers of fbtft. Signed-off-by:
-
Dennis Menschel authored
This patch removes the function set_addr_win() from fb_st7789v.c, as its definition is redundant to the default implementation fbtft_set_addr_win() which can be found in fbtft-core.c. Signed-off-by:
-
Dennis Menschel authored
This patch makes use of the standard MIPI Display Command Set to remove redundant entries from the command enum of the ST7789V display controller and also some of the magic constants found in the init sequence of the C-Berry28 display. Signed-off-by:
-
Alison Schofield authored
Add local variable scanned_queue to rtw_createbss_cmd23a_callback() and use it (5x) to simplify references. Addresses checkpatch.pl WARNING: line over 80 characters Signed-off-by:
Alison Schofield <amsfield22@gmail.com> Signed-off-by:
-
Alison Schofield authored
Move constant to right of comparison test to improve readability. Addresses checkpatch.pl: WARNING: Comparisons should place the constant on the right side of the test Signed-off-by:
-
Alison Schofield authored
Replace printk() with netdev_err() for uniform error reporting. Issue found by checkpatch. Signed-off-by:
-
Burcin Akalin authored
Add space around operator '*'. Problem found using checkpatch.pl CHECK: spaces preferred around that '*' (ctx:VxV) Signed-off-by:
Burcin Akalin <brcnakalin@gmail.com> Signed-off-by:
-
Burcin Akalin authored
Add space around operator '*'. Problem found using checkpatch.pl CHECK: spaces preferred around that '*' (ctx:VxV) Signed-off-by:
Daniel Baluta <daniel.baluta@intel.com> Signed-off-by:
-
Luis de Bethencourt authored
For loop is outside of the else branch of the above conditional statement. Fixing misleading indentation. Fix a smatch warning: drivers/staging/rtl8723au/core/rtw_wlan_util.c:528 WMMOnAssocRsp23a() warn: curly braces intended? Signed-off-by:
Luis de Bethencourt <luisbg@osg.samsung.com> Acked-by:
Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by:
-
Cristina Moraru authored
Replace kmalloc with specialized function kmalloc_array when the size is a multiplication of: number_of_elements * size_of_element Signed-off-by:
Cristina Moraru <cristina.moraru09@gmail.com> Signed-off-by:
-
Burcin Akalin authored
Remove multiple blank lines. Problem found using checkpatch.pl "CHECK: Please don't use multiple blank lines" Signed-off-by:
-
Ioana Ciornei authored
This patch adds spaces around binary operator '|'. Done with coccinelle semantic patch: @@ identifier x, y, z; @@ ( - x|y|z + x | y | z | - x|y + x | y ) Signed-off-by:
Ioana Ciornei <ciorneiioana@gmail.com> Signed-off-by:
-
Ioana Ciornei authored
This patch removes unnecessary brackets when dealing with unary operators like '&'. Done with coccinelle semantic patch: @@ expression e; @@ ( - &(e) + &e ) Signed-off-by:
-
Ioana Ciornei authored
This patch removes unnecessary spaces after the cast. Patch done with coccinelle semantic patch: @rule0@ type t; identifier e; constant c; expression expr; @@ ( - (t) e + (t)e | - (t) c + (t)c | - (t) expr + (t)expr ) Signed-off-by:
-
Ioana Ciornei authored
Indent parameters and arguments passed to function calls to match open paranthesis Signed-off-by:
-
Ioana Ciornei authored
This patch removes the unnecessary blank lines before a closed bracket and after an open bracket. Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Cristina Moraru authored
Remove comparisons to 0 or NULL Signed-off-by:
-
Anton Tatuev authored
Added spaces around binary operators. Signed-off-by:
Tatuev Anton <tatuev.a.a@yandex.ru> Signed-off-by:
-
Burcin Akalin authored
Add space around operator '%'. Problem found using checkpatch.pl CHECK: spaces preferred around that '%' (ctx:VxV) Signed-off-by:
-
Burcin Akalin authored
Remove multiple blank lines. Problem found using checkpatch.pl "CHECK: Please don't use multiple blank lines". Signed-off-by:
-
Cristina Moraru authored
Replace kmalloc with specialized function kmalloc_array when the size is a multiplication of: number_of_elements * size_of_element Signed-off-by:
-
Muhammad Falak R Wani authored
Remove the explicit NULL comparison and rewrite in a compact form. Signed-off-by:
Muhammad Falak R Wani <falakreyaz@gmail.com> Signed-off-by:
-
Muhammad Falak R Wani authored
Remove the explicit NULL comparison and rewrite in a compact form. Signed-off-by:
-
Luis de Bethencourt authored
Clean up all instances of checkpatch.pl checks: CHECK: spaces preferred around that '+' (and other operands) Signed-off-by:
-
Luis de Bethencourt authored
Fix all instances of the following checkpatch.pl check: CHECK: braces {} should be used on all arms of this statement Signed-off-by: -
Luis de Bethencourt authored
Boolean tests do not need explicit comparison to true or false. Signed-off-by:
-
Muhammad Falak R Wani authored
Rewrite explicit NULL comparison in its simpler form. <smpl> @NULL_REPLACE@ expression e; @@ -e == NULL + !e Signed-off-by:
-
Luis de Bethencourt authored
Code is correct, i needs to be moved back by 2 to correct for the last iteration of the while loop, since READ_NEXT_PAIR advances two. Fixing the misleading indentation. Fix a smatch warning: drivers/staging/rtl8188eu/hal/rf_cfg.c:217 rtl88e_phy_config_rf_with_headerfile() warn: curly braces intended? Signed-off-by:
-
Anish Bhatt authored
Use the BIT(x) macro directly instead using multiple BITX defines. Signed-off-by:
Anish Bhatt <anish@gatech.edu> Signed-off-by:
-
Mateusz Kulikowski authored
Drop ugly label as no resource cleanup is needed. Signed-off-by:
Mateusz Kulikowski <mateusz.kulikowski@gmail.com> Signed-off-by:
-
