From: Markus Elfring
Date: Wed, 21 Sep 2016 08:58:41 +0200
Use another space character behind the keyword "if" according to
the Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/nouveau/nvkm/subdev/fb/ramgt215.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(
From: Markus Elfring
Date: Tue, 20 Sep 2016 22:22:14 +0200
* A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle s
From: Markus Elfring
Date: Tue, 20 Sep 2016 22:32:14 +0200
* A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle s
From: Markus Elfring
Date: Wed, 21 Sep 2016 08:28:08 +0200
Do not use curly brackets at four source code places
where a single statement should be sufficient.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/nouveau/nvkm/subdev/fb/ramgt215.c | 14 +-
1 file changed, 5 insertions(+
From: Markus Elfring
Date: Wed, 21 Sep 2016 08:44:38 +0200
The script "checkpatch.pl" can point out that assignments should usually
not be performed within condition checks.
Thus move the assignment for one local variable to a separate statement
in this function.
Signed-off-by: Markus Elfring
-
From: Markus Elfring
Date: Wed, 21 Sep 2016 09:09:09 +0200
A few update suggestions were taken into account
from static source code analysis.
Markus Elfring (5):
Use kmalloc_array() in nvbios_iccsense_parse()
Use kmalloc_array() in gt215_link_train()
Delete unnecessary braces
Adjust a kz
Comments in-line. Thanks.
Roy
Op 21-09-16 om 08:26 schreef SF Markus Elfring:
From: Markus Elfring
Date: Wed, 21 Sep 2016 08:28:08 +0200
Do not use curly brackets at four source code places
where a single statement should be sufficient.
Signed-off-by: Markus Elfring
---
drivers/gpu/drm/no
Thanks for these style fixes. Patches 1 and 2 seem good at first sight.
Not acking because I haven't tested the patches and the implementation
of kmalloc_array() appears to slightly differ from kmalloc(). This
difference *should* only affect allocations larger than the ones in
patch 1 and 2, bu
https://bugs.freedesktop.org/show_bug.cgi?id=97614
--- Comment #9 from Elmar Stellnberger ---
Created attachment 126697
--> https://bugs.freedesktop.org/attachment.cgi?id=126697&action=edit
newer dmesg with 4.8-rc7+
The original page_not_present errors seem to have been resolved upstreams.
N
> The original style was correct, the new style is wrong.
I find your feedback interesting for further clarifications.
> Multi-line indents get curly braces for readability.
How do you think about to transform such an information
into an official specification for the the document "CodingStyle"
Hi, Distribution: openSUSE 42.2 Beta, Kernel: 4.4. The box is a Dell
Precision T7500 with an nVidia Quadro 4000 card and a displayport
connected iiyama B2783QSU 2560x1440 monitor identified as DP-1.
Problem: The monitor is detected as 1920x1080 and not 2560x1440. See Xorg
log file at http://
I think you want the reduced blanking modeline:
$ cvt -r 2560 1440
# 2560x1440 59.95 Hz (CVT 3.69M9-R) hsync: 88.79 kHz; pclk: 241.50 MHz
Modeline "2560x1440R" 241.50 2560 2608 2640 2720 1440 1443 1448
1481 +hsync -vsync
This matches your monitor's detailed timings:
[55.220] (II) NOUVEAU(
12 matches
Mail list logo