[PATCH net-next,v2] net: dsa: mt7530: clean up core and TRGMII clock setup

Three minor changes: - When disabling PLL, there is no need to call core_write_mmd_indirect directly, use the core_write wrapper instead like the rest of the code in the function does. This change helps with consistency and readability. Move the comment to the definition of

[PATCH v2 00/15] acpi: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl. Differences from v1 to v2: - Add subsystem and module name in the name of patch 05/15. - Change to use more proper module name for some patch names. Xiaofei Tan (15): ACPI: APD: fix a block comment align issue ACPI: processor: fix some

[PATCH v2 01/15] ACPI: APD: fix a block comment align issue

Fix the following coding style issue reported by checkpatch.pl. WARNING: Block comments should align the * on each line +/** +* Create platform device during acpi scan attach handle. Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_apd.c | 8 1 file changed, 4 insertions(+), 4

[PATCH v2 04/15] ACPI: table: replace __attribute__((packed)) by __packed

Replace __attribute__((packed)) by __packed following the advice of checkpatch.pl. Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_fpdt.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/acpi/acpi_fpdt.c b/drivers/acpi/acpi_fpdt.c index a89a806..690a88a 100644

[PATCH v2 03/15] ACPI: debug: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: space prohibited between function name and open parenthesis WARNING: else is not generally useful after a break or return Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_dbg.c | 40

[PATCH v2 05/15] ACPI: ipmi: remove useless return statement for void function

Remove useless return statement for void function, reported by checkpatch.pl. WARNING: void function return statements are not generally useful FILE: drivers/acpi/acpi_ipmi.c:482: + return; +} Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_ipmi.c | 1 - 1 file changed, 1 deletion(-)

[PATCH v2 12/15] ACPI: custom_method: fix a coding style issue

Fix the following coding style issue reported by checkpatch.pl ERROR: "foo * bar" should be "foo *bar" FILE: drivers/acpi/custom_method.c:22: +static ssize_t cm_write(struct file *file, const char __user * user_buf, Signed-off-by: Xiaofei Tan --- drivers/acpi/custom_method.c | 2 +- 1 file

[PATCH v2 07/15] ACPI: memhotplug: fix a coding style issue

Fix the following coding style issue reported by checkpatch.pl WARNING: __initdata should be placed after acpi_no_memhotplug FILE: drivers/acpi/acpi_memhotplug.c:326: +static bool __initdata acpi_no_memhotplug; Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_memhotplug.c | 2 +- 1 file

[PATCH v2 08/15] ACPI: acpi_pad: fix a coding style issue

Fix the following coding style issue reported by checkpatch.pl WARNING: Missing a blank line after declarations Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_pad.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/acpi/acpi_pad.c b/drivers/acpi/acpi_pad.c index

[PATCH v2 09/15] ACPI: battery: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Block comments use * on subsequent lines WARNING: Block comments use a trailing */ on a separate line ERROR: code indent should use tabs where possible WARNING: Missing a blank line after declarations

[PATCH v2 06/15] ACPI: LPSS: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: simple_strtol is obsolete, use kstrtol instead WARNING: Missing a blank line after declarations Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_lpss.c | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH v2 11/15] ACPI: CPPC: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Missing a blank line after declarations WARNING: unnecessary whitespace before a quoted newline ERROR: spaces required around that '>=' ERROR: switch and case should be at the same indent Signed-off-by:

[PATCH v2 13/15] ACPI: PM: fix some coding style issues

Fix the following coding style issue reported by checkpatch.pl WARNING: Missing a blank line after declarations Signed-off-by: Xiaofei Tan --- drivers/acpi/device_pm.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/acpi/device_pm.c b/drivers/acpi/device_pm.c index

[PATCH v2 02/15] ACPI: processor: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: ERROR: code indent should use tabs where possible WARNING: Block comments use a trailing */ on a separate line WARNING: Missing a blank line after declarations WARNING: labels should not be indented Signed-off-by:

[PATCH v2 10/15] ACPI: button: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Block comments use * on subsequent lines WARNING: Block comments use a trailing */ on a separate line ERROR: code indent should use tabs where possible Signed-off-by: Xiaofei Tan ---

[PATCH v2 15/15] ACPI: dock: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Missing a blank line after declarations ERROR: spaces required around that ':' WARNING: Statements should start on a tabstop Signed-off-by: Xiaofei Tan --- drivers/acpi/dock.c | 7 +-- 1 file

[PATCH v2 14/15] ACPI: sysfs: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Missing a blank line after declarations WARNING: Block comments should align the * on each line ERROR: open brace '{' following function definitions go on the next line Signed-off-by: Xiaofei Tan ---

Re: nvme: disallow passthru cmd from targeting a nsid != nsid of the block dev

On 26.03.2021 19:48, Niklas Cassel wrote: From: Niklas Cassel When a passthru command targets a specific namespace, the ns parameter to nvme_user_cmd()/nvme_user_cmd64() is set. However, there is currently no validation that the nsid specified in the passthru command targets the namespace/nsid

[PATCH] crypto: hisilicon/hpre - delete redundant log

'hpre_cfg_by_dsm' has checked and printed error path, so it is not necessary at all. Signed-off-by: Hui Tang --- drivers/crypto/hisilicon/hpre/hpre_main.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/crypto/hisilicon/hpre/hpre_main.c

[PATCH] crypto: hisilicon - delete redundant '\n'

It has newline already by sysfs, so delete redundant '\n' Signed-off-by: Hui Tang --- drivers/crypto/hisilicon/hpre/hpre_main.c | 4 ++-- drivers/crypto/hisilicon/sec2/sec_main.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/crypto/hisilicon/hpre/hpre_main.c

[PATCH] crypto: hisilicon/hpre - fix a typo and delete redundant blank line

s/shoul/should/ Signed-off-by: Hui Tang --- drivers/crypto/hisilicon/hpre/hpre_main.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/crypto/hisilicon/hpre/hpre_main.c b/drivers/crypto/hisilicon/hpre/hpre_main.c index f2605c4..8aae921 100644 ---

[PATCH -next] fs: dlm: fix missing unlock on error in accept_from_sock()

Add the missing unlock before return from accept_from_sock() in the error handling case. Fixes: 6cde210a9758 ("fs: dlm: add helper for init connection") Reported-by: Hulk Robot Signed-off-by: Yang Yingliang --- fs/dlm/lowcomms.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH] sit: use min

From: kernel test robot Opportunity for min() Generated by: scripts/coccinelle/misc/minmax.cocci CC: Denis Efremov Reported-by: kernel test robot Signed-off-by: kernel test robot --- sit.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/net/ipv6/sit.c +++ b/net/ipv6/sit.c

[PATCH] powerpc: Fix HAVE_HARDLOCKUP_DETECTOR_ARCH build configuration

When compiling the powerpc with the SMP disabled, it shows the issue: arch/powerpc/kernel/watchdog.c: In function ‘watchdog_smp_panic’: arch/powerpc/kernel/watchdog.c:177:4: error: implicit declaration of function ‘smp_send_nmi_ipi’; did you mean ‘smp_send_stop’?

Re: [PATCH 4/4] erofs: add on-disk compression configurations

On 2021/3/27 11:49, Gao Xiang wrote: From: Gao Xiang Add a bitmap for available compression algorithms and a variable-sized on-disk table for compression options in preparation for upcoming big pcluster and LZMA algorithm, which follows the end of super block. To parse the compression

Re: [PATCH] MIPS: ralink: mt7621: add memory detection support

On Thu, Mar 25, 2021 at 07:45:23PM -0700, Ilya Lipnitskiy wrote: > On Thu, Mar 25, 2021 at 3:01 AM Thomas Bogendoerfer > wrote: > > > > On Tue, Mar 16, 2021 at 10:59:02PM -0700, Ilya Lipnitskiy wrote: > > > From: Chuanhong Guo > > > > > > mt7621 has the following memory map: > > >

Re: [PATCH net 1/4] virtchnl: Fix layout of RSS structures

Hi Samudrala, On Fri, Mar 26, 2021 at 11:45 PM Samudrala, Sridhar wrote: > On 3/26/2021 1:06 AM, Geert Uytterhoeven wrote: > > On Thu, Mar 25, 2021 at 11:29 PM Tony Nguyen > > wrote: > > From: Norbert Ciosek > > > > Remove padding from RSS structures. Previous layout > > could lead to

[PATCH -next] i2c: gpio: use DEFINE_SPINLOCK() for spinlock

From: Lifu Chen spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than explicitly calling spin_lock_init(). Reported-by: Hulk Robot Signed-off-by: Lifu Chen --- arch/arm/mach-sa1100/simpad.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH -next] MIPS: Alchemy: Use DEFINE_SPINLOCK() for spinlock

From: Lifu Chen spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than explicitly calling spin_lock_init(). Reported-by: Hulk Robot Signed-off-by: Lifu Chen --- arch/mips/alchemy/common/clock.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH -next] ARM: davinci: dm646x-evm: use DEFINE_SPINLOCK() for spinlock

From: Lifu Chen spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than explicitly calling spin_lock_init(). Reported-by: Hulk Robot Signed-off-by: Lifu Chen --- arch/arm/mach-davinci/board-dm646x-evm.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH -next] ARM: OMAP: Use DEFINE_SPINLOCK() for spinlock

From: Lifu Chen spinlock can be initialized automatically with DEFINE_SPINLOCK() rather than explicitly calling spin_lock_init(). Reported-by: Hulk Robot Signed-off-by: Lifu Chen --- arch/arm/plat-omap/dma.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH net-next v2] net: dsa: hellcreek: Remove redundant dev_err call in hellcreek_probe()

From: Guobin Huang There is a error message within devm_ioremap_resource already, so remove the dev_err call to avoid redundant error message. Reported-by: Hulk Robot Signed-off-by: Guobin Huang --- drivers/net/dsa/hirschmann/hellcreek.c | 8 ++-- 1 file changed, 2 insertions(+), 6

Re: [PATCH v5 0/5] Add r8a77965 DRIF support

Hi Laurent, On 21/10/2020 23:43, Laurent Pinchart wrote: > Hi Fabrizio, > > On Wed, Oct 21, 2020 at 02:53:27PM +0100, Fabrizio Castro wrote: >> Dear All, >> >> this series is to add DRIF support for the r8a77965 >> (a.k.a. R-Car M3-N). Version 5 fixes a warning reported >> by 'make

Re: [PATCH] ia64: tools: add generic errno.h definition

On Fri, Mar 12, 2021 at 07:51:35AM +, Sergei Trofimovich wrote: > Noticed missing header when build bpfilter helper: > > CC [U] net/bpfilter/main.o > In file included from /usr/include/linux/errno.h:1, >from /usr/include/bits/errno.h:26, >from

[PATCH] crypto: hisilicon/sec - Fix a module parameter error

ctx_q_num is a module parameter set by the user to specify the number of qp queues required to create a ctx. When the number of qp queues allocated by PF or VF is less than the ctx_q_num, an error will be reported when ctx is initialized in kernel mode, which leads to the problem that the

[PATCH] crypto: hisilicon/sec - Fixes AES algorithm mode parameter problem

The input data of the ECB (AES) algorithm needs to be aligned with 16 bytes, and the input data of the XTS (AES) algorithm is at least 16 bytes. Otherwise the SEC hardware will go wrong. Signed-off-by: Longfang Liu --- drivers/crypto/hisilicon/sec2/sec_crypto.c | 37

[tip:master] BUILD SUCCESS 7c06dc5c466683ca68185511b75af21cc9c15f0f

i386 randconfig-a003-20210327 i386 randconfig-a004-20210327 i386 randconfig-a001-20210327 i386 randconfig-a002-20210327 i386 randconfig-a006-20210327 i386 randconfig-a005-20210327 x86_64

Re: [PATCH 4/4] PCI/sysfs: Allow userspace to query and set device reset mechanism

On Fri, Mar 26, 2021 at 08:20:07AM -0600, Alex Williamson wrote: > On Fri, 26 Mar 2021 09:40:30 +0300 > Leon Romanovsky wrote: <...> > > > > It supports by writing: echo "bus,pm" > reset_methods. > > Regarding comma, IMHO it is easiest pattern for the parsing. > > > > Anyway, The in-kernel

Re: [syzbot] KASAN: use-after-free Read in disk_part_iter_next (2)

On Sat, Mar 27, 2021 at 1:01 AM syzbot wrote: > > Hello, > > syzbot has tested the proposed patch but the reproducer is still triggering > an issue: > WARNING in kvm_wait > > [ cut here ] > raw_local_irq_restore() called with IRQs enabled > WARNING: CPU: 1 PID: 10753 at

Re: [PATCH 05/15] remove useless return statement for void function

Oh, sorry, the title doesn't contain subsystem and module name. I'll send v2 later. Please ignore this one. thanks. On 2021/3/27 14:40, Xiaofei Tan wrote: Remove useless return statement for void function, reported by checkpatch.pl. WARNING: void function return statements are not generally

[PATCH -next 1/2] scsi: myrb: Make symbols DAC960_{LA/PG/PD/P}_privdata static

This symbol is not used outside of myrb.c, so we can marks it static. Signed-off-by: Shixin Liu --- drivers/scsi/myrb.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/myrb.c b/drivers/scsi/myrb.c index d469a4889777..56767f8610d4 100644 ---

[PATCH -next 2/2] scsi: myrs: Make symbols DAC960_{GEM/BA/LP}_privdata static

This symbol is not used outside of myrs.c, so we can marks it static. Signed-off-by: Shixin Liu --- drivers/scsi/myrs.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/scsi/myrs.c b/drivers/scsi/myrs.c index 9ebff2449a54..d5ec1cdea0e1 100644 ---

[PATCH] scripts: checkdeclares for checking duplicate struct declares

checkdeclares: find struct declared more than once. Inspired by checkincludes.pl This script checks for duplicate struct declares. Note that this will not take into consideration macros so you should run this only if you know you do have real dups and do not have them under #ifdef's. You could

[PATCH v2 2/4] crypto: hisilicon/zip - add comments for 'hisi_zip_sqe'

Some fields of 'hisi_zip_sqe' are unused, and some fields have misc utilities. So add comments for used fields and make others unnamed. Signed-off-by: Yang Shen --- drivers/crypto/hisilicon/zip/zip.h| 45 ++- drivers/crypto/hisilicon/zip/zip_crypto.c | 4 +--

[PATCH v2 3/4] crypto: hisilicon/zip - initialize operations about 'sqe' in 'acomp_alg.init'

The operations about 'sqe' are different on some hardwares. Add a struct 'hisi_zip_sqe_ops' to describe the operations in a hardware. And choose the 'ops' in 'hisi_zip_acomp_init' according to the hardware. Signed-off-by: Yang Shen --- drivers/crypto/hisilicon/zip/zip_crypto.c | 141

[PATCH v2 4/4] crypto: hisilicon/zip - support new 'sqe' type in Kunpeng930

The Kunpeng930 changes some field meanings in 'sqe'. So add a new 'hisi_zip_sqe_ops' to describe the 'sqe' operations. Signed-off-by: Yang Shen --- drivers/crypto/hisilicon/zip/zip.h| 1 + drivers/crypto/hisilicon/zip/zip_crypto.c | 25 + 2 files changed, 26

[PATCH v2 0/4] crypto: hisilicon/zip - support new 'sqe' type in Kunpeng930

In Kunpeng930, some field meanings in 'sqe' are changed, so driver need to distinguish the type on different platform. To avoid checking the platform everytime when driver fills the 'sqe', add a struct 'hisi_zip_sqe_ops' to describe the 'sqe' operations. The driver only need to choose the 'ops'

[PATCH v2 1/4] crypto: hisilicon/zip - adjust functions location

This patch changes nothing about functions except location in order to make code logic clearly. This adjustment follows three principles: 1.The called functions are listed in order above the calling functions. 2.The paired functions are next to each other. 3.Logically similar functions are placed

Re: [PATCH 0/4] PCI: Correct function names in the header

Hi, Krzysztof On 2021/3/26 9:07, Krzysztof Wilczyński wrote: > Hi, > > [...] >> PCI: acpi_pcihp: Correct acpi_pci_check_ejectable() function name in >> the header >> PCI/AER: Correct function names in the header >> PCI/PME: Correct pcie_pme_init() function name in the header >>

[PATCH] crypto: hisilicon/hpre - fix PASID setting on kunpeng 920

We must confirm the PASID is disabled before using no-sva mode. Signed-off-by: Hui Tang --- drivers/crypto/hisilicon/hpre/hpre_main.c | 43 +-- 1 file changed, 18 insertions(+), 25 deletions(-) diff --git a/drivers/crypto/hisilicon/hpre/hpre_main.c

Re: [PATCH 06/15] ACPI: acpi_lpss: fix some coding style issues

' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Xiaofei-Tan/acpi-fix-some-coding-style-issues/20210327-144714 base: https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git linux-next config: x86_64-randconfig-a005-20210326

Re: Candidate Linux ABI for Intel AMX and hypothetical new related features

On Fri, Mar 26, 2021 at 11:39:18PM -0400, Len Brown wrote: > Say a mainline links with a math library that uses AMX without the > knowledge of the mainline. What is a "mainline"? -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette

Re: [PATCH 2/4] erofs: support adjust lz4 history window size

On 2021/3/27 11:49, Gao Xiang wrote: From: Huang Jianan lz4 uses LZ4_DISTANCE_MAX to record history preservation. When using rolling decompression, a block with a higher compression ratio will cause a larger memory allocation (up to 64k). It may cause a large resource burden in extreme cases

Re: [PATCH 1/4] erofs: introduce erofs_sb_has_xxx() helpers

On 2021/3/27 11:49, Gao Xiang wrote: From: Gao Xiang Introduce erofs_sb_has_xxx() to make long checks short, especially for later big pcluster & LZMA features. Signed-off-by: Gao Xiang Reviewed-by: Chao Yu Thanks,

Re: [PATCH v2] media: sq905.c: fix uninitialized variable

Hi Alaa, FYI: this patch has already been applied to the media_tree master git repo: https://patchwork.linuxtv.org/project/linux-media/patch/2c46832a-99a8-73bf-ec85-085052f8b...@xs4all.nl/ That's good enough for this issue so I am marking this patch as Obsolete in our patchwork. On 26/03/2021

Re: [PATCH v2 2/4] leds: simatic-ipc-leds: add new driver for Siemens Industial PCs

Am Thu, 18 Mar 2021 12:38:53 +0100 schrieb "Enrico Weigelt, metux IT consult" : > On 15.03.21 12:19, Pavel Machek wrote: > > > But I still don't like the naming. simantic-ipc: prefix is > > useless. Having 6 status leds is not good, either. > > Do we have some standard naming policy those

Re: [PATCH 00/15] acpi: fix some coding style issues

OK. thanks for reviewing this patch set. On 2021/3/27 16:21, Andy Shevchenko wrote: On Saturday, March 27, 2021, Xiaofei Tan mailto:tanxiao...@huawei.com>> wrote: Fix some coding style issues reported by checkpatch.pl . NAK until it’s proven that you have

[PATCH] mt76: Convert to DEFINE_SHOW_ATTRIBUTE

From: Junlin Yang Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code. Signed-off-by: Junlin Yang --- .../net/wireless/mediatek/mt76/mt7915/debugfs.c| 36 -- .../net/wireless/mediatek/mt76/mt7921/debugfs.c| 18 ++- 2 files changed, 9 insertions(+), 45

Re: [PATCH] crypto: hisilicon/sec - Fix a module parameter error

On 2021/3/27 17:29, Longfang Liu Wrote: > ctx_q_num is a module parameter set by the user to specify the > number of qp queues required to create a ctx. > > When the number of qp queues allocated by PF or VF is less than > the ctx_q_num, an error will be reported when ctx is initialized > in

Re: [PATCH v3 6/9] debugfs: Implement debugfs_create_str()

On Fri, Mar 26, 2021 at 03:50:00PM +0100, Peter Zijlstra wrote: > Subject: debugfs: Implement debugfs_create_str() > From: Peter Zijlstra > Date: Thu Mar 25 10:53:55 CET 2021 > > Implement debugfs_create_str() to easily display names and such in > debugfs. > > Signed-off-by: Peter Zijlstra

Re: [PATCH v2 0/3] scsi: add runtime PM workaround for SD cardreaders

Am Dienstag, dem 12.01.2021 um 10:33 +0100 schrieb Martin Kepplinger: > revision history > > v2: >  * move module parameter to sd >  * add Documentation > > v1: > https://lore.kernel.org/linux-scsi/2021052029.28426-1-martin.kepplin...@puri.sm/T/#t > > > > hi, > > In

[PATCH 07/15] ACPI: acpi_memhotplug: fix a coding style issue

Fix the following coding style issue reported by checkpatch.pl WARNING: __initdata should be placed after acpi_no_memhotplug FILE: drivers/acpi/acpi_memhotplug.c:326: +static bool __initdata acpi_no_memhotplug; Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_memhotplug.c | 2 +- 1 file

[PATCH 09/15] ACPI: battery: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Block comments use * on subsequent lines WARNING: Block comments use a trailing */ on a separate line ERROR: code indent should use tabs where possible WARNING: Missing a blank line after declarations

[PATCH 03/15] ACPI: acpi_dbg: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: space prohibited between function name and open parenthesis WARNING: else is not generally useful after a break or return Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_dbg.c | 40

[PATCH 11/15] ACPI: cppc_acpi: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Missing a blank line after declarations WARNING: unnecessary whitespace before a quoted newline ERROR: spaces required around that '>=' ERROR: switch and case should be at the same indent Signed-off-by:

[PATCH 06/15] ACPI: acpi_lpss: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: simple_strtol is obsolete, use kstrtol instead WARNING: Missing a blank line after declarations Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_lpss.c | 4 +++- 1 file changed, 3 insertions(+), 1

[PATCH 13/15] ACPI: device_pm: fix some coding style issues

Fix the following coding style issue reported by checkpatch.pl WARNING: Missing a blank line after declarations Signed-off-by: Xiaofei Tan --- drivers/acpi/device_pm.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/acpi/device_pm.c b/drivers/acpi/device_pm.c index

[PATCH 02/15] ACPI: processor: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: ERROR: code indent should use tabs where possible WARNING: Block comments use a trailing */ on a separate line WARNING: Missing a blank line after declarations WARNING: labels should not be indented Signed-off-by:

[PATCH 15/15] ACPI: dock: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Missing a blank line after declarations ERROR: spaces required around that ':' WARNING: Statements should start on a tabstop Signed-off-by: Xiaofei Tan --- drivers/acpi/dock.c | 7 +-- 1 file

[PATCH 14/15] ACPI: device_sysfs: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Missing a blank line after declarations WARNING: Block comments should align the * on each line ERROR: open brace '{' following function definitions go on the next line Signed-off-by: Xiaofei Tan ---

[PATCH 00/15] acpi: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl. Xiaofei Tan (15): ACPI: APD: fix a block comment align issue ACPI: processor: fix some coding style issues ACPI: acpi_dbg: fix some coding style issues ACPI: acpi_fpdt: replace __attribute__((packed)) by __packed remove useless

[PATCH 05/15] remove useless return statement for void function

Remove useless return statement for void function, reported by checkpatch.pl. WARNING: void function return statements are not generally useful FILE: drivers/acpi/acpi_ipmi.c:482: + return; +} Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_ipmi.c | 1 - 1 file changed, 1 deletion(-)

[PATCH 10/15] ACPI: button: fix some coding style issues

Fix some coding style issues reported by checkpatch.pl, including following types: WARNING: Block comments use * on subsequent lines WARNING: Block comments use a trailing */ on a separate line ERROR: code indent should use tabs where possible Signed-off-by: Xiaofei Tan ---

[PATCH 12/15] ACPI: custom_method: fix a coding style issue

Fix the following coding style issue reported by checkpatch.pl ERROR: "foo * bar" should be "foo *bar" FILE: drivers/acpi/custom_method.c:22: +static ssize_t cm_write(struct file *file, const char __user * user_buf, Signed-off-by: Xiaofei Tan --- drivers/acpi/custom_method.c | 2 +- 1 file

[PATCH 08/15] ACPI: acpi_pad: fix a coding style issue

Fix the following coding style issue reported by checkpatch.pl WARNING: Missing a blank line after declarations Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_pad.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/acpi/acpi_pad.c b/drivers/acpi/acpi_pad.c index

[PATCH 04/15] ACPI: acpi_fpdt: replace __attribute__((packed)) by __packed

Replace __attribute__((packed)) by __packed following the advice of checkpatch.pl. Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_fpdt.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/acpi/acpi_fpdt.c b/drivers/acpi/acpi_fpdt.c index a89a806..690a88a 100644

[PATCH 01/15] ACPI: APD: fix a block comment align issue

Fix the following coding style issue reported by checkpatch.pl. WARNING: Block comments should align the * on each line +/** +* Create platform device during acpi scan attach handle. Signed-off-by: Xiaofei Tan --- drivers/acpi/acpi_apd.c | 8 1 file changed, 4 insertions(+), 4

[PATCH 0/2] kernel-doc and MAINTAINERS clean-up

Roughly 900 warnings of about 21.000 kernel-doc warnings in the kernel tree warn with 'cannot understand function prototype:', i.e., the kernel-doc parser cannot parse the function's signature. The majority, about 600 cases of those, are just struct definitions following the kernel-doc

[PATCH 2/2] soc: qcom: address kernel-doc warnings

The command: find ./include/linux/soc/qcom/ | xargs ./scripts/kernel-doc -none reports: ./include/linux/soc/qcom/qmi.h:26: warning: cannot understand function prototype: 'struct qmi_header ' ./include/linux/soc/qcom/qmi.h:101: warning: cannot understand function prototype: 'struct

[PATCH 1/2] MAINTAINERS: add another entry for ARM/QUALCOMM SUPPORT

The files in ./include/linux/soc/qcom/ are headers to the corresponding files in ./drivers/soc/qcom/, which are assigned to ARM/QUALCOMM SUPPORT. Possibly, the file pattern include/linux/*/qcom* intended to match this directory and its containing files, but unfortunately, it does not. Hence, add

Re: [External] [PATCH 5/8] hugetlb: call update_and_free_page without hugetlb_lock

On Thu, Mar 25, 2021 at 8:29 AM Mike Kravetz wrote: > > With the introduction of remove_hugetlb_page(), there is no need for > update_and_free_page to hold the hugetlb lock. Change all callers to > drop the lock before calling. > > With additional code modifications, this will allow loops which

[PATCH v3 1/2] usb: gadget: s3c: Fix incorrect resources releasing

Since commit 188db4435ac6 ("usb: gadget: s3c: use platform resources"), 'request_mem_region()' and 'ioremap()' are no more used, so they don't need to be undone in the error handling path of the probe and in the remove function. Remove these calls and the unneeded 'rsrc_start' and 'rsrc_len'

[PATCH v3 2/2] usb: gadget: s3c: Fix the error handling path in 's3c2410_udc_probe()'

Some 'clk_prepare_enable()' and 'clk_get()' must be undone in the error handling path of the probe function, as already done in the remove function. Fixes: 3fc154b6b813 ("USB Gadget driver for Samsung s3c2410 ARM SoC") Signed-off-by: Christophe JAILLET Reviewed-by: Krzysztof Kozlowski --- v2:

Re: [External] [PATCH 8/8] hugetlb: add lockdep_assert_held() calls for hugetlb_lock

On Thu, Mar 25, 2021 at 8:29 AM Mike Kravetz wrote: > > After making hugetlb lock irq safe and separating some functionality > done under the lock, add some lockdep_assert_held to help verify > locking. > > Signed-off-by: Mike Kravetz Reviewed-by: Muchun Song Thanks. > --- > mm/hugetlb.c |

Re: [PATCH 3/4] erofs: introduce on-disk lz4 fs configurations

On 2021/3/27 11:49, Gao Xiang wrote: From: Gao Xiang Introduce z_erofs_lz4_cfgs to store all lz4 configurations. Currently it's only max_distance, but will be used for new features later. Signed-off-by: Gao Xiang Reviewed-by: Chao Yu Thanks,

Re: [PATCH v8 1/3] lib: zstd: Add kernel-specific API

to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Nick-Terrell/Update-to-zstd-1-4-10/20210327-031827 base: https://git.kernel.org/pub/scm

[PATCH] f2fs: introduce gc_merge mount option

In this patch, we will add two new mount options: "gc_merge" and "nogc_merge", when background_gc is on, "gc_merge" option can be set to let background GC thread to handle foreground GC requests, it can eliminate the sluggish issue caused by slow foreground GC operation when GC is triggered from a

Re: [PATCH v2 06/15] ACPI: LPSS: fix some coding style issues

Hi Andy, On 2021/3/27 16:19, Andy Shevchenko wrote: On Saturday, March 27, 2021, Xiaofei Tan mailto:tanxiao...@huawei.com>> wrote: Fix some coding style issues reported by checkpatch.pl , including following types: WARNING: simple_strtol is obsolete,

Re: Candidate Linux ABI for Intel AMX and hypothetical new related features

On Fri, Mar 26, 2021 at 11:39:18PM -0400, Len Brown wrote: > Hi Andy, > > Say a mainline links with a math library that uses AMX without the > knowledge of the mainline. What does this mean? What happened to the context here? > Say the mainline is also linked with a userspace threading library

[PATCH] crypto: hisilicon/hpre - Add processing of src_data in 'CURVE25519'

HPRE can only deal with src_data smaller than 'p' in 'CURVE25519' algorithm, but 'rfc7748' says: 'Implementations MUST accept non-canonical values and process them as if they had been reduced modulo the field prime' So we get its modulus to p, and then deal it with HPRE. Signed-off-by: Meng Yu

Re: [PATCH] arm: 9016/2: Make symbol 'tmp_pmd_table' static

Why do you have 9016/2 in the subject line? That's an identifier from the patch system which shouldn't be in the subject line. If you want to refer to something already committed, please do so via the sha1 git hash and quote the first line of the commit description within ("...") in the body of

Re: [PATCH 4/4] erofs: add on-disk compression configurations

Hi Chao, On Sat, Mar 27, 2021 at 05:46:44PM +0800, Chao Yu wrote: > On 2021/3/27 11:49, Gao Xiang wrote: > > From: Gao Xiang > > > > Add a bitmap for available compression algorithms and a variable-sized > > on-disk table for compression options in preparation for upcoming big > > pcluster and

Re: [PATCHv3] Introduced new tracing mode KCOV_MODE_UNIQUE.

On Fri, Mar 26, 2021 at 09:51:28PM +0100, Alexander Lochmann wrote: > It simply stores the executed PCs. > The execution order is discarded. > Each bit in the shared buffer represents every fourth > byte of the text segment. > Since a call instruction on every supported > architecture is at least

Re: [PATCH] PCI: Allow drivers to claim exclusive access to config regions

On Fri, Mar 26, 2021 at 11:12:47AM -0500, Bjorn Helgaas wrote: > [+cc Christoph] > > On Wed, Mar 24, 2021 at 06:23:54PM -0700, Dan Williams wrote: > > The PCIE Data Object Exchange (DOE) mailbox is a protocol run over > > configuration cycles. It assumes one initiator at a time is > >

[PATCH v2 2/2] fuse: Remove unused parameter

Since we move the fuse_wait_on_page_writeback() to fuse_fill_write_pages(), thus remove the unused 'inode' parameter of fuse_send_write_pages(). Signed-off-by: Baolin Wang --- Changes from v1: - New patch. --- fs/fuse/file.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff

[PATCH v2 1/2] fuse: Fix possible deadlock when writing back dirty pages

We can meet below deadlock scenario when writing back dirty pages, and writing files at the same time. The deadlock scenario can be reproduced by: - A writeback worker thread A is trying to write a bunch of dirty pages by fuse_writepages(), and the fuse_writepages() will lock one page (named page

Re: [External] [PATCH 4/8] hugetlb: create remove_hugetlb_page() to separate functionality

On Thu, Mar 25, 2021 at 8:29 AM Mike Kravetz wrote: > > The new remove_hugetlb_page() routine is designed to remove a hugetlb > page from hugetlbfs processing. It will remove the page from the active > or free list, update global counters and set the compound page > destructor to NULL so that

arch/mips/kernel/setup.c:47:39: error: conflicting types for '__appended_dtb'

Hi Thomas, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 0f4498cef9f5cd18d7c6639a2a902ec1edc5be4e commit: b83ba0b9df56f8404ccc6ebcc7050fb8294f0f20 MIPS: of: Introduce helper function to get DTB date: 7 weeks

[PATCH] drm: tegra: hub: struct tegra_dc is declared twice

struct tegra_dc has been declared at 13rd line. Remove the duplicate. Move struct tegra_plane declaration forward. Signed-off-by: Wan Jiabing --- drivers/gpu/drm/tegra/hub.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/tegra/hub.h

Re: [PATCH next 2/2] misc: Add Renesas Synchronization Management Unit (SMU) support

Hi, Thank you for the patch! Yet something to improve: [auto build test ERROR on next-20210326] url: https://github.com/0day-ci/linux/commits/min-li-xe-renesas-com/mfd-Add-Renesas-Synchronization-Management-Unit-SMU-support/20210327-150316 base:931294922e65a23e1aad6398b9ae02df74044679

  1   2   3   4   5   >