I was just now trying to track down my other issue, whereby somewhere
along the tree kexec stops working properly. In the process of doing
that I realized I had initially made one change to the original 4.9
config beyond oldconfig: I'd turned off WX debugging.
I've now compiled a bunch of
> From: Alexandru Chirvasitu [mailto:achirva...@gmail.com]
> Sent: Friday, December 22, 2017 14:29
>
> The output of that precise command run just now on a freshly-compiled
> copy of that commit is attached.
>
> On Fri, Dec 22, 2017 at 09:31:28PM +, Dexuan Cui wrote:
> > > From: Alexandru
> From: Alexandru Chirvasitu [mailto:achirva...@gmail.com]
> Sent: Friday, December 22, 2017 06:21
>
> In the absence of logs, the best I can do at the moment is attach a
> picture of the screen I am presented with on the apic=debug boot
> attempt.
> Alex
The panic happens in
This patch fixes a coding style issue as noted by
checkpatch.pl related to unnecessary parentheses.
This patch fixes the following checkpatch.pl warning:
WARNING: Unnecessary parentheses around 'priv->basic_rates'.
Signed-off-by: Sumit Pundir
---
From: Harsh Shandilya
Several methods in the driver were hardcoding
the function name in their logging calls which
is a checkpatch violation. Utilise the __func__
macro to avoid needing to add the function name
as is to the string.
Test: logtags in dmesg remain unaffected before
From: Harsh Shandilya
checkpatch warns against the use of symbolic permissions,
this patch migrates all symbolic permissions in the binder
driver to octal permissions.
Test: debugfs nodes created by binder have the same unix
permissions prior to and after this patch was applied.
Fix the following errors found by checkpatch.pl:
CHECK: Prefer kernel type 'u32' over 'uint32_t'
Cc: Vaibhav Agarwal
Signed-off-by: Kamal Heib
---
drivers/staging/greybus/audio_codec.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
Fix "Alignment should match open parenthesis" checkpatch.pl errors.
Cc: Vaibhav Agarwal
Signed-off-by: Kamal Heib
---
drivers/staging/greybus/audio_codec.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Remove the blank lines.
Cc: Vaibhav Agarwal
Signed-off-by: Kamal Heib
---
drivers/staging/greybus/audio_codec.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/greybus/audio_codec.c
b/drivers/staging/greybus/audio_codec.c
index
Fix the following errors found by checkpatch.pl:
CHECK: Prefer kernel type 'u32' over 'uint32_t'
Cc: Vaibhav Agarwal
Signed-off-by: Kamal Heib
---
drivers/staging/greybus/audio_codec.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff
Space is required when using the question mark operator around ':'
Cc: Vaibhav Agarwal
Signed-off-by: Kamal Heib
---
drivers/staging/greybus/audio_codec.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Cleanup "Alignment should match open parenthesis" checkpatch.pl errors.
Cc: Vaibhav Agarwal
Signed-off-by: Kamal Heib
---
drivers/staging/greybus/audio_codec.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git
Fix the following error found by checkpatch.pl:
CHECK: Logical continuations should be on the previous line
Cc: Vaibhav Agarwal
Signed-off-by: Kamal Heib
---
drivers/staging/greybus/audio_codec.c | 4 ++--
1 file changed, 2 insertions(+), 2
Fix the following error found by checkpatch.pl:
CHECK: Alignment should match open parenthesis
+static int cap_ioctl(struct gb_cap *cap, unsigned int cmd,
+void __user *buf)
Cc: Viresh Kumar
Signed-off-by: Kamal Heib
---
This patch set include multiple fixes for the issues reported by
checkpatch.pl
Kamal Heib (8):
staging: greybus: authentication.c: Fix alignment should match open
parenthesis
staging: greybus: audio_codec.c: Fix alignment should match open
parenthesis
staging: greybus:
The misspelled variable name 'splitted' was changed into 'split'.
Signed-off-by: Julian Geus
Signed-off-by: Kilian Bender
---
drivers/staging/lustre/lustre/ldlm/ldlm_flock.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff
Hi Alexandru,
At 12/21/2017 10:23 AM, Alexandru Chirvasitu wrote:
This might be more helpful. I ran another bisect with the following
final log:
---
git bisect start
# bad: [d6ffc6ac83b1f9f12652d89b9cb5bcbfbea7796c] x86/vector: Respect affinity
mask in irq descriptor
git bisect bad
From: Ravi Eluri
fixed "function definition argument should have an identifier name",
with appropriate identifier names. Pointed out by checkpatch.
Signed-off-by: Ravi Eluri
Signed-off-by: Suniel Mahesh
---
Note:
-
4.14-stable review patch. If anyone has any objections, please let me know.
--
From: Juergen Gross
commit 03b2a320b19f1424e9ac9c21696be9c60b6d0d93 upstream.
The x86_hyper pointer is only used for checking whether a virtual
device is supporting the hypervisor
4.14-stable review patch. If anyone has any objections, please let me know.
--
From: Juergen Gross
commit f72e38e8ec8869ac0ba5a75d7d2f897d98a1454e upstream.
Instead of x86_hyper being either NULL on bare metal or a pointer to a
struct hypervisor_x86 in case
Fixes checkpatch warnings:
CHECK: Avoid CamelCase:
CHECK: Avoid CamelCase:
CHECK: Avoid CamelCase:
CHECK: Avoid CamelCase:
CHECK: Avoid CamelCase:
Signed-off-by: Valentin Vidic
---
drivers/staging/pi433/Documentation/pi433.txt | 10 +-
From: Ravi Eluri
Fixes checkpatch warning:
macros should not use a trailing semicolon.
Signed-off-by: Ravi Eluri
---
Note:
- Patch was compile tested and built(ARCH=arm) on linux-next
(latest).
- No build issues reported.
---
22 matches
Mail list logo