[PATCH] lib/test_kmod: add missing MODULE_DESCRIPTION() macro

2024-05-31 Thread Jeff Johnson
make allmodconfig && make W=1 C=1 reports: WARNING: modpost: missing MODULE_DESCRIPTION() in lib/test_kmod.o Add the missing invocation of the MODULE_DESCRIPTION() macro. Signed-off-by: Jeff Johnson --- lib/test_kmod.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/test_kmod

[PATCH] nvdimm: add missing MODULE_DESCRIPTION() macros

2024-05-26 Thread Jeff Johnson
MODULE_DESCRIPTION() in drivers/nvdimm/nd_e820.o WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/nvdimm/of_pmem.o WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/nvdimm/nd_virtio.o Signed-off-by: Jeff Johnson --- drivers/nvdimm/btt.c | 1 + drivers/nvdimm/core.c | 1 + drivers

[PATCH RESEND] nvdimm: add missing MODULE_DESCRIPTION() macros

2024-05-26 Thread Jeff Johnson
MODULE_DESCRIPTION() in drivers/nvdimm/nd_e820.o WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/nvdimm/of_pmem.o WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/nvdimm/nd_virtio.o Signed-off-by: Jeff Johnson --- drivers/nvdimm/btt.c | 1 + drivers/nvdimm/core.c | 1 + drivers

[PATCH] testing: nvdimm: iomap: add MODULE_DESCRIPTION()

2024-05-26 Thread Jeff Johnson
Fix the 'make W=1' warning: WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/nvdimm/../../tools/testing/nvdimm/test/iomap.o Signed-off-by: Jeff Johnson --- tools/testing/nvdimm/test/iomap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/testing/nvdimm/test/iomap.c b/tools

Re: [PATCH 09/12] remoteproc: qcom_wcnss: make use of QCOM_FW_HELPER

2024-05-22 Thread Jeff Johnson
On 5/21/2024 2:45 AM, Dmitry Baryshkov wrote: > Make the driver use qcom_fw_helper to autodetect the path to the > calibration data file. > > Signed-off-by: Dmitry Baryshkov > --- > drivers/remoteproc/qcom_wcnss.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git

[PATCH] kernel: trace: preemptirq_delay_test: add MODULE_DESCRIPTION()

2024-05-18 Thread Jeff Johnson
Fix the 'make W=1' warning: WARNING: modpost: missing MODULE_DESCRIPTION() in kernel/trace/preemptirq_delay_test.o Signed-off-by: Jeff Johnson --- kernel/trace/preemptirq_delay_test.c | 1 + 1 file changed, 1 insertion(+) diff --git a/kernel/trace/preemptirq_delay_test.c b/kernel/trace

[PATCH] vringh: add MODULE_DESCRIPTION()

2024-05-16 Thread Jeff Johnson
Fix the allmodconfig 'make w=1' issue: WARNING: modpost: missing MODULE_DESCRIPTION() in drivers/vhost/vringh.o Signed-off-by: Jeff Johnson --- drivers/vhost/vringh.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/vhost/vringh.c b/drivers/vhost/vringh.c index 7b8fd977f71c

[PATCH] tracing: Fix trace_pid_list_free() kernel-doc

2024-05-06 Thread Jeff Johnson
make C=1 reports: kernel/trace/pid_list.c:458: warning: Function parameter or struct member 'pid_list' not described in 'trace_pid_list_free' Add the missing parameter to the trace_pid_list_free() kernel-doc. Signed-off-by: Jeff Johnson --- kernel/trace/pid_list.c | 1 + 1 file changed, 1

Re: [FYI][PATCH] tracing/treewide: Remove second parameter of __assign_str()

2024-02-23 Thread Jeff Johnson
On 2/23/2024 9:56 AM, Steven Rostedt wrote: > From: "Steven Rostedt (Google)" > > [ >This is a treewide change. I will likely re-create this patch again in >the second week of the merge window of v6.9 and submit it then. Hoping >to keep the conflicts that it will cause to a minimum.

Re: [net-next PATCH v3 3/3] net: phy: add support for PHY package MMD read/write

2023-12-05 Thread Jeff Johnson
On 12/5/2023 10:14 AM, Russell King (Oracle) wrote: > On Tue, Dec 05, 2023 at 09:44:05AM -0800, Jeff Johnson wrote: >> So in my experience a function prototype IS the function definition, and >> the actual function is just the implementation of that definition. >> >>

Re: [net-next PATCH v3 3/3] net: phy: add support for PHY package MMD read/write

2023-12-05 Thread Jeff Johnson
On 12/5/2023 8:11 AM, Russell King (Oracle) wrote: > On Tue, Dec 05, 2023 at 07:29:12AM -0800, Jakub Kicinski wrote: >> On Tue, 5 Dec 2023 15:10:50 + Russell King (Oracle) wrote: >>> I've raised this before in other subsystems, and it's suggested that >>> it's better to have it in the .c file.

[PATCH v2 2/2] kbuild: handle excessively long argument lists

2021-01-14 Thread Jeff Johnson
. The usage is well documented at https://www.gnu.org/software/make/manual/html_node/File-Function.html Signed-off-by: Mahesh Kumar Kalikot Veetil Signed-off-by: Jeff Johnson --- Changes in v2: - Remove spurious endif scripts/Makefile.build | 5 - 1 file changed, 4 insertions(+), 1 deletion

[PATCH 2/2] kbuild: handle excessively long argument lists

2021-01-12 Thread Jeff Johnson
. The usage is well documented at https://www.gnu.org/software/make/manual/html_node/File-Function.html Signed-off-by: Mahesh Kumar Kalikot Veetil Signed-off-by: Jeff Johnson --- scripts/Makefile.build | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/scripts/Makefile.build b

[PATCH 1/2] kbuild: simplify cmd_mod

2021-01-12 Thread Jeff Johnson
mar Kalikot Veetil Signed-off-by: Jeff Johnson --- scripts/Makefile.build | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/scripts/Makefile.build b/scripts/Makefile.build index 4c058f1..252b7d2 100644 --- a/scripts/Makefile.build +++ b/scripts/Makefile.build @@ -279,10 +279

Re: [PATCH v2] net: enable wireless core features with LEGACY_WEXT_ALLCONFIG

2019-09-09 Thread Jeff Johnson
On 2019-09-09 08:44, Johannes Berg wrote: Also, you probably know this, but in this particular case you really should just get rid of your wext dependencies This. Particularly for one out-of-tree driver with which I'm intimately familiar there has been considerable recent work to make all

Re: [PATCH] wcn36xx: Add support for Factory Test Mode (FTM)

2018-05-17 Thread Jeff Johnson
On 2018-05-17 04:32, Ramon Fried wrote: From: Eyal Ilsar ... +int wcn36xx_smd_process_ptt_msg(struct wcn36xx *wcn, + struct ieee80211_vif *vif, void *ptt_msg, size_t len, + void **ptt_rsp_msg) +{ + struct

Re: [PATCH] wcn36xx: Add support for Factory Test Mode (FTM)

2018-05-17 Thread Jeff Johnson
On 2018-05-17 04:32, Ramon Fried wrote: From: Eyal Ilsar ... +int wcn36xx_smd_process_ptt_msg(struct wcn36xx *wcn, + struct ieee80211_vif *vif, void *ptt_msg, size_t len, + void **ptt_rsp_msg) +{ + struct wcn36xx_hal_process_ptt_msg_req_msg

Re: [PATCH] wcn36xx: Add support for Factory Test Mode (FTM)

2018-05-17 Thread Jeff Johnson
On 2018-05-17 04:32, Ramon Fried wrote: From: Eyal Ilsar ... +static int wcn36xx_smd_process_ptt_msg_rsp(void *buf, size_t len, + void **p_ptt_rsp_msg) +{ + struct wcn36xx_hal_process_ptt_msg_rsp_msg *rsp; + int ret =

Re: [PATCH] wcn36xx: Add support for Factory Test Mode (FTM)

2018-05-17 Thread Jeff Johnson
On 2018-05-17 04:32, Ramon Fried wrote: From: Eyal Ilsar ... +static int wcn36xx_smd_process_ptt_msg_rsp(void *buf, size_t len, + void **p_ptt_rsp_msg) +{ + struct wcn36xx_hal_process_ptt_msg_rsp_msg *rsp; + int ret = 0; why initialize

2.6 diskstats and 4096 sector block devices?

2016-05-13 Thread Jeff Johnson
would point to diskstats handling everything in terms of 512B operations. To multiply by 8 or not to multiply by 8, that is the question.. :-) Thanks, --Jeff -- Jeff Johnson Co-Founder Aeon Computing jeff.johnson "dot" aeoncomputing.com www.aeoncomputing

2.6 diskstats and 4096 sector block devices?

2016-05-13 Thread Jeff Johnson
would point to diskstats handling everything in terms of 512B operations. To multiply by 8 or not to multiply by 8, that is the question.. :-) Thanks, --Jeff -- Jeff Johnson Co-Founder Aeon Computing jeff.johnson "dot" aeoncomputing.com www.aeoncomputing