Re: [PATCH] clk: test: Forward-declare struct of_phandle_args in kunit/clk.h

2025-05-06 Thread Stephen Boyd
Quoting Richard Fitzgerald (2025-03-27 05:52:14) > Add a forward-declare of struct of_phandle_args to prevent the compiler > warning: > > ../include/kunit/clk.h:29:63: warning: ‘struct of_phandle_args’ declared > inside parameter list will not be visible outside of this definition or > declaration

Re: [PATCH RESEND v13 01/12] clk: mmp: Switch to use struct u32_fract instead of custom one

2024-11-14 Thread Stephen Boyd
> Reviewed-by: Linus Walleij > Reviewed-by: Stephen Boyd > Signed-off-by: Duje Mihanović > --- Applied to clk-next

Re: [PATCH RESEND v13 08/12] clk: mmp: Add Marvell PXA1908 MPMU driver

2024-11-14 Thread Stephen Boyd
Quoting Duje Mihanović via B4 Relay (2024-11-04 08:37:10) > From: Duje Mihanović > > Add driver for the MPMU controller block on Marvell's PXA1908 SoC. The > driver is incomplete, currently only supporting the fixed PLL1; dynamic > PLLs 2-4 and CPU/DDR/AXI clock support is missing. > > Signed-of

Re: [PATCH RESEND v13 07/12] clk: mmp: Add Marvell PXA1908 APMU driver

2024-11-14 Thread Stephen Boyd
Quoting Duje Mihanović via B4 Relay (2024-11-04 08:37:09) > From: Duje Mihanović > > Add driver for the APMU controller block found on Marvell's PXA1908 SoC. > This driver is incomplete, lacking support for (at least) GPU, VPU, DSI > and CCIC (camera related) clocks. > > Signed-off-by: Duje Miha

Re: [PATCH RESEND v13 06/12] clk: mmp: Add Marvell PXA1908 APBCP driver

2024-11-14 Thread Stephen Boyd
Quoting Duje Mihanović via B4 Relay (2024-11-04 08:37:08) > From: Duje Mihanović > > Add driver for the APBCP controller block found on Marvell's PXA1908 > SoC. > > Signed-off-by: Duje Mihanović > --- Applied to clk-next

Re: [PATCH RESEND v13 05/12] clk: mmp: Add Marvell PXA1908 APBC driver

2024-11-14 Thread Stephen Boyd
Quoting Duje Mihanović via B4 Relay (2024-11-04 08:37:07) > From: Duje Mihanović > > Add driver for the APBC controller block found on Marvell's PXA1908 SoC. > > Signed-off-by: Duje Mihanović > --- Applied to clk-next

Re: [PATCH RESEND v13 04/12] dt-bindings: clock: Add Marvell PXA1908 clock bindings

2024-11-14 Thread Stephen Boyd
Quoting Duje Mihanović via B4 Relay (2024-11-04 08:37:06) > From: Duje Mihanović > > Add dt bindings and documentation for the Marvell PXA1908 clock > controller. > > Reviewed-by: Conor Dooley > Reviewed-by: Stephen Boyd > Signed-off-by: Duje Mihanović > --- Applied to clk-next

Re: [PATCH v8 8/8] clk: Add KUnit tests for clks registered with struct clk_parent_data

2024-10-09 Thread Stephen Boyd
Quoting Guenter Roeck (2024-10-08 16:27:37) > On 10/8/24 16:12, Stephen Boyd wrote: > > > > The best I can come up with then is to test for a NULL of_root when > > CONFIG_ARM64 and CONFIG_ACPI are enabled, because the tests > > intentionally don't work when both t

Re: [PATCH v8 8/8] clk: Add KUnit tests for clks registered with struct clk_parent_data

2024-10-08 Thread Stephen Boyd
Quoting Guenter Roeck (2024-10-03 21:52:09) > On 10/3/24 17:42, Stephen Boyd wrote: > > > > Can you please describe how you run the kunit test? And provide the qemu > > command you run to boot arm64 with acpi? > > > > Example command line: >

Re: [PATCH v8 8/8] clk: Add KUnit tests for clks registered with struct clk_parent_data

2024-10-03 Thread Stephen Boyd
Quoting Guenter Roeck (2024-10-03 17:25:37) > On 10/3/24 16:46, Stephen Boyd wrote: > [ ... ] > > That DT test has been there for a few releases. Is this the first time > > those tests have been run on arm64+acpi? I didn't try after sending the > > patches and forg

Re: [PATCH v8 8/8] clk: Add KUnit tests for clks registered with struct clk_parent_data

2024-10-03 Thread Stephen Boyd
rote: > >>>>> On 9/27/24 10:19, Guenter Roeck wrote: > >>>>>> Copying devicetree maintainers. > >>>>>> > >>>>>> On Thu, Sep 26, 2024 at 09:39:38PM -0700, Guenter Roeck wrote: > >>>>>>> On Thu, Sep

Re: [PATCH v2 2/5] dt-bindings: soc: qcom: smd-rpm: add generic compatibles

2024-07-31 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2024-07-29 12:52:15) > Add two generic compatibles to all smd-rpm devices, they follow the same > RPMSG protocol and are either accessed through the smd-edge or through > the glink-edge. > > Signed-off-by: Dmitry Baryshkov > --- Acked-by: Stephen Boyd

Re: [PATCH v9 5/9] clk: mmp: Add Marvell PXA1908 clock driver

2024-04-22 Thread Stephen Boyd
Quoting Duje Mihanović (2024-04-20 06:32:56) > On 4/20/24 00:24, Stephen Boyd wrote: > > Quoting Duje Mihanović (2024-04-19 07:31:14) > >> On Friday, April 12, 2024 4:57:09 AM GMT+2 Stephen Boyd wrote: > >>> Quoting Duje Mihanović (2024-04-11 03:15:34) > >>

Re: [PATCH v9 5/9] clk: mmp: Add Marvell PXA1908 clock driver

2024-04-19 Thread Stephen Boyd
Quoting Duje Mihanović (2024-04-19 07:31:14) > On Friday, April 12, 2024 4:57:09 AM GMT+2 Stephen Boyd wrote: > > Quoting Duje Mihanović (2024-04-11 03:15:34) > > > > > On 4/11/2024 10:00 AM, Stephen Boyd wrote: > > > > Is there a reason this file can't be

Re: [PATCH v9 5/9] clk: mmp: Add Marvell PXA1908 clock driver

2024-04-11 Thread Stephen Boyd
Quoting Duje Mihanović (2024-04-11 03:15:34) > On 4/11/2024 10:00 AM, Stephen Boyd wrote: > > > > Is there a reason this file can't be a platform driver? > > Not that I know of, I did it like this only because the other in-tree > MMP clk drivers do so. I guess t

Re: [PATCH v9 5/9] clk: mmp: Add Marvell PXA1908 clock driver

2024-04-11 Thread Stephen Boyd
Quoting Duje Mihanović (2024-04-02 13:55:41) > diff --git a/drivers/clk/mmp/clk-of-pxa1908.c > b/drivers/clk/mmp/clk-of-pxa1908.c > new file mode 100644 > index ..6f1f6e25a718 > --- /dev/null > +++ b/drivers/clk/mmp/clk-of-pxa1908.c > @@ -0,0 +1,328 @@ > +// SPDX-License-Identifier: GP

Re: [PATCH v9 4/9] dt-bindings: clock: Add Marvell PXA1908 clock bindings

2024-04-11 Thread Stephen Boyd
Quoting Duje Mihanović (2024-04-02 13:55:40) > Add dt bindings and documentation for the Marvell PXA1908 clock > controller. > > Reviewed-by: Conor Dooley > Signed-off-by: Duje Mihanović > --- Reviewed-by: Stephen Boyd

Re: [PATCH v9 1/9] clk: mmp: Switch to use struct u32_fract instead of custom one

2024-04-11 Thread Stephen Boyd
viewed-by: Linus Walleij > Signed-off-by: Duje Mihanović > --- Reviewed-by: Stephen Boyd > > diff --git a/drivers/clk/mmp/clk.h b/drivers/clk/mmp/clk.h > index 55ac05379781..c83cec169ddc 100644 > --- a/drivers/clk/mmp/clk.h > +++ b/drivers/clk/mmp/clk.h > @@ -3,6 +3,

Re: [PATCH 1/3] dt-bindings: clock: keystone: remove unstable remark

2024-02-28 Thread Stephen Boyd
ntrol clock driver") Almost eleven years is enough, so drop the > "unstable" remark and expect usual ABI rules. > > Signed-off-by: Krzysztof Kozlowski > --- Acked-by: Stephen Boyd

Re: [PATCH 2/3] dt-bindings: clock: ti: remove unstable remark

2024-02-28 Thread Stephen Boyd
> and expect usual ABI rules. > > Signed-off-by: Krzysztof Kozlowski > --- Acked-by: Stephen Boyd > Documentation/devicetree/bindings/clock/ti/adpll.txt| 2 -- > Documentation/devicetree/bindings/clock/ti/apll.txt | 2 -- > Documentation/d

Re: [PATCH] dt-bindings: correct white-spaces in examples

2023-11-27 Thread Stephen Boyd
Quoting Krzysztof Kozlowski (2023-11-24 01:21:21) > Use only one and exactly one space around '=' in DTS example. > > Signed-off-by: Krzysztof Kozlowski > > --- Acked-by: Stephen Boyd

Re: [PATCH v3 3/3] drm/msm/dp: check main link status before start aux read

2021-04-20 Thread Stephen Boyd
Quoting Kuogee Hsieh (2021-04-16 10:38:51) > Maybe when the cable is disconnected the DP phy should be shutdown and > some bit in the phy could effectively "cut off" the aux channel and then > NAKs would start coming through here in the DP controller I/O register > space. This patch have DP aux cha

Re: [PATCH 1/2] drm/msm/dp: service only one irq_hpd if there are multiple irq_hpd pending

2021-04-20 Thread Stephen Boyd
Quoting Kuogee Hsieh (2021-04-16 13:27:57) > Some dongle may generate more than one irq_hpd events in a short period of > time. This patch will treat those irq_hpd events as single one and service > only one irq_hpd event. Why is it bad to get multiple irq_hpd events in a short period of time? Ple

[PATCH v5 13/13] kdump: Use vmlinux_build_id to simplify

2021-04-20 Thread Stephen Boyd
We can use the vmlinux_build_id array here now instead of open coding it. This mostly consolidates code. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Dave Young Cc: Baoquan He Cc: Vivek Goyal Cc: Signed-off-by: Stephen Boyd --- include/linux

[PATCH v5 09/13] scripts/decode_stacktrace.sh: Silence stderr messages from addr2line/nm

2021-04-20 Thread Stephen Boyd
cript. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Konstantin Khlebnikov Cc: Sasha Levin Signed-off-by: Stephen Boyd --- scripts/decode_stacktrace.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/decod

[PATCH v5 12/13] buildid: Fix kernel-doc notation

2021-04-20 Thread Stephen Boyd
Kernel doc should use "Return:" instead of "Returns" to properly reflect the return values. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Signed-off-by: Stephen Boyd --- lib/buildid.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH v5 10/13] scripts/decode_stacktrace.sh: Indicate 'auto' can be used for base path

2021-04-20 Thread Stephen Boyd
he variable argument and that it is optional so that we can differentiate from the literal "auto" that should be passed. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Konstantin Khlebnikov Cc: Sasha Levin Signed-off-by: Stephen Boyd ---

[PATCH v5 08/13] scripts/decode_stacktrace.sh: Support debuginfod

2021-04-20 Thread Stephen Boyd
stedt Cc: Andy Shevchenko Cc: Matthew Wilcox Signed-off-by: Stephen Boyd --- scripts/decode_stacktrace.sh | 81 +++- 1 file changed, 70 insertions(+), 11 deletions(-) diff --git a/scripts/decode_stacktrace.sh b/scripts/decode_stacktrace.sh index 9039834

[PATCH v5 11/13] buildid: Mark some arguments const

2021-04-20 Thread Stephen Boyd
These arguments are never modified so they can be marked const to indicate as such. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Signed-off-by: Stephen Boyd --- lib/buildid.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a

[PATCH v5 07/13] x86/dumpstack: Use %pSb/%pBb for backtrace printing

2021-04-20 Thread Stephen Boyd
ceware.org/elfutils/Debuginfod.html [2] Signed-off-by: Stephen Boyd --- arch/x86/kernel/dumpstack.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/dumpstack.c b/arch/x86/kernel/dumpstack.c index 299c20f0a38b..ea4fe192189d 100644 --- a/arch/x86/kernel/dumpstack.c

[PATCH v5 05/13] module: Add printk formats to add module build ID to stacktraces

2021-04-20 Thread Stephen Boyd
Petr Mladek Cc: Steven Rostedt Cc: Sergey Senozhatsky Cc: Andy Shevchenko Cc: Rasmus Villemoes Cc: Cc: Matthew Wilcox Link: https://fedoraproject.org/wiki/Releases/FeatureBuildId [1] Link: https://sourceware.org/elfutils/Debuginfod.html [2] Signed-off-by: Stephen Boyd --

[PATCH v5 06/13] arm64: stacktrace: Use %pSb for backtrace printing

2021-04-20 Thread Stephen Boyd
infod.html [2] Signed-off-by: Stephen Boyd --- arch/arm64/kernel/stacktrace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/kernel/stacktrace.c b/arch/arm64/kernel/stacktrace.c index ad20981dfda4..9d38da01ff98 100644 --- a/arch/arm64/kernel/stacktrace.c +++ b/

[PATCH v5 04/13] dump_stack: Add vmlinux build ID to stack traces

2021-04-20 Thread Stephen Boyd
ceware.org/elfutils/Debuginfod.html [2] Signed-off-by: Stephen Boyd --- include/linux/buildid.h | 4 lib/Kconfig.debug | 11 +++ lib/buildid.c | 2 ++ lib/dump_stack.c| 13 +++-- 4 files changed, 28 insertions(+), 2 deletions(-) diff --git a/inc

[PATCH v5 02/13] buildid: Add API to parse build ID out of buffer

2021-04-20 Thread Stephen Boyd
Add an API that can parse the build ID out of a buffer, instead of a vma, to support printing a kernel module's build ID for stack traces. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Signed-off-by: Stephen Boyd --- include/linux/buildid.h | 1 +

[PATCH v5 03/13] buildid: Stash away kernels build ID on init

2021-04-20 Thread Stephen Boyd
Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Dave Young Cc: Baoquan He Cc: Vivek Goyal Cc: Signed-off-by: Stephen Boyd --- include/linux/buildid.h | 3 +++ init/main.c | 1 + lib/buildid.c | 15 +++ 3 files c

[PATCH v5 01/13] buildid: Only consider GNU notes for build ID parsing

2021-04-20 Thread Stephen Boyd
Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Fixes: bd7525dacd7e ("bpf: Move stack_map_get_build_id into lib") Tested-by: Petr Mladek Signed-off-by: Stephen Boyd --- lib/buildid.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/buildid.c b/lib/buildid.c index 6156997c3895..

[PATCH v5 00/13] Add build ID to stacktraces

2021-04-20 Thread Stephen Boyd
This series adds the kernel's build ID[1] to the stacktrace header printed in oops messages, warnings, etc. and the build ID for any module that appears in the stacktrace after the module name. The goal is to make the stacktrace more self-contained and descriptive by including the relevant build ID

Re: [PATCH v3] arm64: vdso32: drop -no-integrated-as flag

2021-04-20 Thread Stephen Boyd
ed-by: Nathan Chancellor > Reviewed-by: Vincenzo Frascino > --- It boots for me and compat vDSO seems to work properly per tools/testing/selftests/vDSO/, userspace programs aren't crashing right and left, must be good: Tested-by: Stephen Boyd

Re: [PATCH 12/12] dt-bindings: soc: qcom: aoss: Delete unused power-domain definitions

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:58) > Delete unused power-domain definitions exposed by AOSS QMP. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 10/12] arm64: dts: qcom: sm8250: Use QMP binding to control load state

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:56) > Use the Qualcomm Mailbox Protocol (QMP) binding to control the load > state resources on SM8250 SoCs and drop deprecated power-domains exposed > by AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 11/12] arm64: dts: qcom: sm8350: Use QMP binding to control load state

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:57) > Use the Qualcomm Mailbox Protocol (QMP) binding to control the load > state resources on SM8350 SoCs and drop deprecated power-domains exposed > by AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 09/12] arm64: dts: qcom: sm8150: Use QMP binding to control load state

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:55) > Use the Qualcomm Mailbox Protocol (QMP) binding to control the load > state resources on SM8150 SoCs and drop deprecated power-domains exposed > by AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 08/12] arm64: dts: qcom: sdm845: Use QMP binding to control load state

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:54) > Use the Qualcomm Mailbox Protocol (QMP) binding to control the load > state resources on SDM845 SoCs and drop deprecated power-domains exposed > by AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 07/12] arm64: dts: qcom: sc7280: Use QMP binding to control load state

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:53) > Use the Qualcomm Mailbox Protocol (QMP) binding to control the load > state resources on SC7280 SoCs and drop deprecated power-domains exposed > by AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 06/12] arm64: dts: qcom: sc7180: Use QMP binding to control load state

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:52) > Use the Qualcomm Mailbox Protocol (QMP) binding to control the load > state resources on SC7180 SoCs and drop deprecated power-domains exposed > by AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 02/12] soc: qcom: aoss: Drop power domain support

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:48) > The load state resources are expected to follow the life cycle of the > remote processor it tracks. However, modeling load state resources as > power-domains result in them getting turned off during system suspend > and thereby falling out of sync with th

Re: [PATCH 03/12] dt-bindings: remoteproc: qcom: pas: Add QMP bindings

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:49) > Add Qualcomm Mailbox Protocol (QMP) binding to replace the power domains > exposed by the AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 04/12] dt-bindings: remoteproc: qcom: Add QMP bindings

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:50) > Add Qualcomm Mailbox Protocol (QMP) binding to replace the power domains > exposed by the AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH 01/12] dt-bindings: soc: qcom: aoss: Drop power-domain bindings

2021-04-17 Thread Stephen Boyd
Quoting Sibi Sankar (2021-04-16 05:03:47) > Drop power-domain bindings exposed by AOSS QMP node. > > Signed-off-by: Sibi Sankar > --- Reviewed-by: Stephen Boyd

Re: [PATCH v4 05/13] module: Add printk formats to add module build ID to stacktraces

2021-04-17 Thread Stephen Boyd
Quoting Jessica Yu (2021-04-15 06:04:35) > +++ Stephen Boyd [09/04/21 18:52 -0700]: > >diff --git a/include/linux/module.h b/include/linux/module.h > >index 59f094fa6f74..4bf869f6c944 100644 > >--- a/include/linux/module.h > >+++ b/include/linux/module.h > >@

Re: [PATCH v2 1/7] dt-bindings: clock: Add MT8192 APU clock bindings

2021-04-16 Thread Stephen Boyd
Quoting Flora Fu (2021-04-14 22:52:34) > Add clock bindings for APU on MT8192. > > Signed-off-by: Flora Fu > Acked-by: Rob Herring > --- Acked-by: Stephen Boyd

Re: [PATCH v2 2/7] clk: mediatek: mt8192: Add APU clocks support

2021-04-16 Thread Stephen Boyd
Quoting Flora Fu (2021-04-14 22:52:35) > Add APU clocks support on MT8192. > > Signed-off-by: Flora Fu > --- Acked-by: Stephen Boyd

Re: [PATCH 2/4] dt-bindings: clock: Convert ti,sci-clk to json schema

2021-04-16 Thread Stephen Boyd
Quoting Nishanth Menon (2021-04-15 23:37:19) > diff --git a/Documentation/devicetree/bindings/clock/ti,sci-clk.yaml > b/Documentation/devicetree/bindings/clock/ti,sci-clk.yaml > new file mode 100644 > index ..72633651f0c7 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/

Re: [PATCH] mmc: core: Don't allocate IDA for OF aliases

2021-04-16 Thread Stephen Boyd
Quoting Ulf Hansson (2021-04-16 00:17:10) > On Thu, 15 Apr 2021 at 21:29, Stephen Boyd wrote: > > > > > > Don't think so. The device (with the kobject inside) is removed, and > > thus the mmc1 device will be removed, but the kobject's release function > &g

Re: [PATCH v2 3/3] drm/msm/dp: check main link status before start aux read

2021-04-15 Thread Stephen Boyd
Quoting khs...@codeaurora.org (2021-04-15 15:02:40) > On 2021-04-15 13:06, Stephen Boyd wrote: > > > > Is it really necessary to have this patch at all? I think there are > > bigger problems with suspend/resume of the DP driver in relation to the > > kthread stopping.

Re: [PATCH v2 3/3] drm/msm/dp: check main link status before start aux read

2021-04-15 Thread Stephen Boyd
Quoting khs...@codeaurora.org (2021-04-15 10:37:29) > On 2021-04-14 14:09, Stephen Boyd wrote: > > Quoting Kuogee Hsieh (2021-04-13 16:11:44) > >> Make sure main link is in connection state before start aux > >> read/write operation to avoid unnecessary long delay due

Re: [PATCH] mmc: core: Don't allocate IDA for OF aliases

2021-04-15 Thread Stephen Boyd
Quoting Ulf Hansson (2021-04-15 01:56:12) > On Tue, 13 Apr 2021 at 02:36, Stephen Boyd wrote: > > > > - err = ida_simple_get(&mmc_host_ida, min_idx, max_idx, GFP_KERNEL); > > - if (err < 0) { > > - kfree(host); > > -

Re: [PATCH v2 2/3] drm/msm/dp: initialize audio_comp when audio starts

2021-04-14 Thread Stephen Boyd
Quoting Kuogee Hsieh (2021-04-14 14:02:50) > Initialize audio_comp when audio starts and wait for audio_comp at > dp_display_disable(). This will take care of both dongle unplugged > and display off (suspend) cases. > > Changes in v2: > -- add dp_display_start_audio() > > Signed-off-by: Kuogee Hs

Re: [PATCH v2 1/3] drm/msm/dp: check sink_count before update is_connected status

2021-04-14 Thread Stephen Boyd
only at boot up and > pm_resume") > Reported-by: Stephen Boyd > Reviewed-by: Stephen Boyd > Tested-by: Stephen Boyd > Signed-off-by: Kuogee Hsieh > --- Can you please thread your emailed patches? I see them all as toplevel messages in my inbox :( > drivers/gpu/drm/msm

Re: [PATCH v2 3/3] drm/msm/dp: check main link status before start aux read

2021-04-14 Thread Stephen Boyd
Quoting Kuogee Hsieh (2021-04-13 16:11:44) > Make sure main link is in connection state before start aux > read/write operation to avoid unnecessary long delay due to > main link had been unplugged. > > Signed-off-by: Kuogee Hsieh > --- > drivers/gpu/drm/msm/dp/dp_aux.c | 5 + > drivers/gp

Re: [PATCH v2] usb: dwc3: core: Add shutdown callback for dwc3

2021-04-14 Thread Stephen Boyd
(NoC/interconnect errors). > > Signed-off-by: Sandeep Maheswaram > --- Reviewed-by: Stephen Boyd

Re: [PATCH v2 2/3] drm/msm/dp: do not re initialize of audio_comp at display_disable()

2021-04-13 Thread Stephen Boyd
Quoting Kuogee Hsieh (2021-04-13 16:11:30) > At dongle unplug, dp initializes audio_comp followed by sending disconnect > event notification to audio and to make sure audio had shutdown completely > by wait for audio completion notification at display_disable(). This patch Is this dp_display_disab

Re: [PATCH v2 1/3] drm/msm/dp: check sink_count before update is_connected status

2021-04-13 Thread Stephen Boyd
only at boot up and > pm_resume") > Reported-by: Stephen Boyd > Signed-off-by: Kuogee Hsieh > --- Reviewed-by: Stephen Boyd Tested-by: Stephen Boyd

Re: [PATCH v4 05/13] module: Add printk formats to add module build ID to stacktraces

2021-04-13 Thread Stephen Boyd
Quoting Petr Mladek (2021-04-13 08:01:14) > On Fri 2021-04-09 18:52:52, Stephen Boyd wrote: > > Let's make kernel stacktraces easier to identify by including the build > > ID[1] of a module if the stacktrace is printing a symbol from a module. > > This makes it simpler

Re: [PATCH v4 05/13] module: Add printk formats to add module build ID to stacktraces

2021-04-13 Thread Stephen Boyd
Quoting Stephen Boyd (2021-04-13 13:10:05) > Quoting Petr Mladek (2021-04-13 08:16:20) > > On Tue 2021-04-13 13:56:31, Andy Shevchenko wrote: > > > On Mon, Apr 12, 2021 at 12:29:05PM -0700, Stephen Boyd wrote: > > > > Quoting Andy Shevchenko (2021-04-12 04:58:02) >

Re: [PATCH v4 04/13] dump_stack: Add vmlinux build ID to stack traces

2021-04-13 Thread Stephen Boyd
Quoting Petr Mladek (2021-04-13 07:41:11) > > diff --git a/lib/dump_stack.c b/lib/dump_stack.c > > index f5a33b6f773f..d685331b065f 100644 > > --- a/lib/dump_stack.c > > +++ b/lib/dump_stack.c > > @@ -5,6 +5,7 @@ > > */ > > > > #include > > +#include > > #include > > #include > > #inclu

Re: [PATCH v4 05/13] module: Add printk formats to add module build ID to stacktraces

2021-04-13 Thread Stephen Boyd
Quoting Petr Mladek (2021-04-13 08:16:20) > On Tue 2021-04-13 13:56:31, Andy Shevchenko wrote: > > On Mon, Apr 12, 2021 at 12:29:05PM -0700, Stephen Boyd wrote: > > > Quoting Andy Shevchenko (2021-04-12 04:58:02) > > > > > > > > First of all, why not

Re: [PATCH v13 4/4] MAINTAINERS: add MT7621 CLOCK maintainer

2021-04-13 Thread Stephen Boyd
Quoting Sergio Paracuellos (2021-04-09 22:50:59) > Adding myself as maintainer for mt7621 clock driver. > > Signed-off-by: Sergio Paracuellos > --- Applied to clk-next

Re: [PATCH v13 2/4] staging: mt7621-dts: make use of new 'mt7621-clk'

2021-04-13 Thread Stephen Boyd
Quoting Sergio Paracuellos (2021-04-09 22:50:57) > Clocks for SoC mt7621 have been properly integrated so there is > no need to declare fixed clocks at all in the device tree. Remove > all of them, add new device tree nodes for mt7621-clk and update > the rest of the nodes to use them. > > Acked-b

Re: [PATCH v13 3/4] staging: mt7621-dts: use valid vendor 'mediatek' instead of invalid 'mtk'

2021-04-13 Thread Stephen Boyd
Quoting Sergio Paracuellos (2021-04-09 22:50:58) > Vendor listed for mediatek in kernel vendor file 'vendor-prefixes.yaml' > contains 'mediatek' as a valid vendor string. Some nodes in the device > tree are using an invalid vendor string vfor 'mtk' instead. Fix all of > them in dts file. Update als

Re: [PATCH v13 1/4] clk: ralink: add clock driver for mt7621 SoC

2021-04-13 Thread Stephen Boyd
Quoting Sergio Paracuellos (2021-04-09 22:50:56) > The documentation for this SOC only talks about two > registers regarding to the clocks: > * SYSC_REG_CPLL_CLKCFG0 - provides some information about > boostrapped refclock. PLL and dividers used for CPU and some > sort of BUS. > * SYSC_REG_CPLL_CLK

Re: [PATCH 2/2] drm/msm/dp: do not re initialize of audio_comp

2021-04-12 Thread Stephen Boyd
Quoting Kuogee Hsieh (2021-04-12 10:03:23) > At dp_display_disable(), do not re initialize audio_comp if > hdp_state == ST_DISCONNECT_PENDING (unplug event) to avoid > race condition which cause 5 second timeout expired. More details please. > Also > add abort mechanism to reduce time spinning at

Re: [PATCH 1/2] drm/msm/dp: check sink_count before update is_connected status

2021-04-12 Thread Stephen Boyd
ned-off-by: Kuogee Hsieh > --- Also please include Reported-by: Stephen Boyd in the next post.

Re: [PATCH 1/2] drm/msm/dp: check sink_count before update is_connected status

2021-04-12 Thread Stephen Boyd
Quoting Kuogee Hsieh (2021-04-12 10:02:51) > At pm_resume check link sisnk_count before update is_connected status s/sisnk_count/sink_count/ > base on HPD real time link status. Also print out error message only > when either EV_CONNECT_PENDING_TIMEOUT or EV_DISCONNECT_PENDING_TIMEOUT > happen.

Re: [PATCH][V2] clk: uniphier: Fix potential infinite loop

2021-04-12 Thread Stephen Boyd
Quoting Colin King (2021-04-09 02:01:04) > From: Colin Ian King > > The for-loop iterates with a u8 loop counter i and compares this > with the loop upper limit of num_parents that is an int type. > There is a potential infinite loop if num_parents is larger than > the u8 loop counter. Fix this b

Re: [PATCH] clk: qcom: rpmh: add support for SDX55 rpmh IPA clock

2021-04-12 Thread Stephen Boyd
Quoting Alex Elder (2021-04-09 06:44:07) > The IPA core clock is required for SDX55. Define it. > > Signed-off-by: Alex Elder > --- Applied to clk-next

[PATCH] mmc: core: Don't allocate IDA for OF aliases

2021-04-12 Thread Stephen Boyd
t;mmc: core: Allow setting slot index via device tree alias") Signed-off-by: Stephen Boyd --- drivers/mmc/core/host.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c index 9b89a91b6b47..137b4a769f62 1

Re: [PATCH 3/5] ASoC: rt5682: clock driver must use the clock provider API

2021-04-12 Thread Stephen Boyd
Quoting Jerome Brunet (2021-04-10 04:13:54) > Clock drivers ops should not the clk API but the clock provider (clk_hw) > instead. > > Signed-off-by: Jerome Brunet > --- > sound/soc/codecs/rt5682.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/sound/soc/codecs/rt

Re: [PATCH 5/5] ASoC: da7219: properly get clk from the provider

2021-04-12 Thread Stephen Boyd
> --- Reviewed-by: Stephen Boyd

Re: [PATCH 2/5] ASoC: wcd934x: use the clock provider API

2021-04-12 Thread Stephen Boyd
Quoting Jerome Brunet (2021-04-10 04:13:53) > Clock providers should use the clk_hw API It sort of already is :) > > Signed-off-by: Jerome Brunet > --- Reviewed-by: Stephen Boyd

Re: [PATCH 1/5] ASoC: stm32: properly get clk from the provider

2021-04-12 Thread Stephen Boyd
> --- Reviewed-by: Stephen Boyd

Re: [PATCH] [v2] clk: renesas: rcar-usb2-clock-sel: Fix error handling in rcar_usb2_clock_sel_probe

2021-04-12 Thread Stephen Boyd
Quoting Dinghao Liu (2021-04-12 04:26:01) > diff --git a/drivers/clk/renesas/rcar-usb2-clock-sel.c > b/drivers/clk/renesas/rcar-usb2-clock-sel.c > index 3abafd78f7c8..3b0e33e0bf7b 100644 > --- a/drivers/clk/renesas/rcar-usb2-clock-sel.c > +++ b/drivers/clk/renesas/rcar-usb2-clock-sel.c > @@ -193,1

Re: [PATCH v4 05/13] module: Add printk formats to add module build ID to stacktraces

2021-04-12 Thread Stephen Boyd
Quoting Andy Shevchenko (2021-04-12 04:58:02) > On Fri, Apr 09, 2021 at 06:52:52PM -0700, Stephen Boyd wrote: > > Let's make kernel stacktraces easier to identify by including the build > > ID[1] of a module if the stacktrace is printing a symbol from a module. > >

Re: [PATCH v2] arm64: dts: qcom: Update iommu property for simultaneous playback

2021-04-12 Thread Stephen Boyd
Quoting Srinivasa Rao Mandadapu (2021-04-09 22:17:07) > Hi Stephen. > > Thanks for your time!!! > > > On 4/9/2021 10:31 PM, Stephen Boyd wrote: > > Quoting Srinivasa Rao Mandadapu (2021-04-06 09:33:30) > >> From: V Sujith Kumar Reddy > >> > &

Re: [PATCH v2] arm64: dts: qcom: Update iommu property for simultaneous playback

2021-04-12 Thread Stephen Boyd
u > --- Reviewed-by: Stephen Boyd

Re: [PATCH v1 1/2] arm64: dts: qcom: sc7280: Add cpufreq hw node

2021-04-09 Thread Stephen Boyd
Quoting Taniya Das (2021-04-09 19:04:39) > @@ -1116,6 +1124,17 @@ > #clock-cells = <1>; > }; > }; > + > + cpufreq_hw: cpufreq@18591000 { > + compatible = "qcom,cpufreq-epss"; > +

Re: [PATCH v2] platform/chrome: Don't populate lightbar device if it isn't there

2021-04-09 Thread Stephen Boyd
Quoting Stephen Boyd (2020-09-10 17:53:07) > Quoting Enric Balletbo i Serra (2020-09-10 08:49:42) > > On 10/9/20 16:52, Guenter Roeck wrote: > > > On Thu, Sep 10, 2020 at 7:32 AM Enric Balletbo i Serra > > > wrote: > > >> On 10/9/20 16:18, Guenter Roeck wro

[PATCH v4 11/13] buildid: Mark some arguments const

2021-04-09 Thread Stephen Boyd
These arguments are never modified so they can be marked const to indicate as such. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Signed-off-by: Stephen Boyd --- lib/buildid.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a

[PATCH v4 10/13] scripts/decode_stacktrace.sh: Indicate 'auto' can be used for base path

2021-04-09 Thread Stephen Boyd
he variable argument and that it is optional so that we can differentiate from the literal "auto" that should be passed. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Konstantin Khlebnikov Cc: Sasha Levin Signed-off-by: Stephen Boyd ---

[PATCH v4 13/13] kdump: Use vmlinux_build_id to simplify

2021-04-09 Thread Stephen Boyd
We can use the vmlinux_build_id array here now instead of open coding it. This mostly consolidates code. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Dave Young Cc: Baoquan He Cc: Vivek Goyal Cc: Signed-off-by: Stephen Boyd --- include/linux

[PATCH v4 09/13] scripts/decode_stacktrace.sh: Silence stderr messages from addr2line/nm

2021-04-09 Thread Stephen Boyd
cript. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Konstantin Khlebnikov Cc: Sasha Levin Signed-off-by: Stephen Boyd --- scripts/decode_stacktrace.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/scripts/decod

[PATCH v4 12/13] buildid: Fix kernel-doc notation

2021-04-09 Thread Stephen Boyd
Kernel doc should use "Return:" instead of "Returns" to properly reflect the return values. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Signed-off-by: Stephen Boyd --- lib/buildid.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

[PATCH v4 08/13] scripts/decode_stacktrace.sh: Support debuginfod

2021-04-09 Thread Stephen Boyd
stedt Cc: Andy Shevchenko Cc: Matthew Wilcox Signed-off-by: Stephen Boyd --- scripts/decode_stacktrace.sh | 81 +++- 1 file changed, 70 insertions(+), 11 deletions(-) diff --git a/scripts/decode_stacktrace.sh b/scripts/decode_stacktrace.sh index 9039834

[PATCH v4 07/13] x86/dumpstack: Use %pSb/%pBb for backtrace printing

2021-04-09 Thread Stephen Boyd
ceware.org/elfutils/Debuginfod.html [2] Signed-off-by: Stephen Boyd --- arch/x86/kernel/dumpstack.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/x86/kernel/dumpstack.c b/arch/x86/kernel/dumpstack.c index 299c20f0a38b..be2de39bf16f 100644 --- a/arch/x86/kernel/d

[PATCH v4 05/13] module: Add printk formats to add module build ID to stacktraces

2021-04-09 Thread Stephen Boyd
Petr Mladek Cc: Steven Rostedt Cc: Sergey Senozhatsky Cc: Andy Shevchenko Cc: Rasmus Villemoes Cc: Cc: Matthew Wilcox Link: https://fedoraproject.org/wiki/Releases/FeatureBuildId [1] Link: https://sourceware.org/elfutils/Debuginfod.html [2] Signed-off-by: Stephen Boyd --- Documentation/core

[PATCH v4 06/13] arm64: stacktrace: Use %pSb for backtrace printing

2021-04-09 Thread Stephen Boyd
infod.html [2] Signed-off-by: Stephen Boyd --- arch/arm64/kernel/stacktrace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/kernel/stacktrace.c b/arch/arm64/kernel/stacktrace.c index ad20981dfda4..9d38da01ff98 100644 --- a/arch/arm64/kernel/stacktrace.c +++ b/

[PATCH v4 03/13] buildid: Stash away kernels build ID on init

2021-04-09 Thread Stephen Boyd
Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Cc: Dave Young Cc: Baoquan He Cc: Vivek Goyal Cc: Signed-off-by: Stephen Boyd --- include/linux/buildid.h | 3 +++ init/main.c | 1 + lib/buildid.c | 15 +++ 3 files c

[PATCH v4 04/13] dump_stack: Add vmlinux build ID to stack traces

2021-04-09 Thread Stephen Boyd
ceware.org/elfutils/Debuginfod.html [2] Signed-off-by: Stephen Boyd --- lib/Kconfig.debug | 11 +++ lib/dump_stack.c | 13 +++-- 2 files changed, 22 insertions(+), 2 deletions(-) diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index 2779c29d9981..5f883e50f406 100644

[PATCH v4 02/13] buildid: Add API to parse build ID out of buffer

2021-04-09 Thread Stephen Boyd
Add an API that can parse the build ID out of a buffer, instead of a vma, to support printing a kernel module's build ID for stack traces. Cc: Jiri Olsa Cc: Alexei Starovoitov Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Signed-off-by: Stephen Boyd --- include/linux/buildid.h | 1 +

[PATCH v4 01/13] buildid: Only consider GNU notes for build ID parsing

2021-04-09 Thread Stephen Boyd
Cc: Jessica Yu Cc: Evan Green Cc: Hsin-Yi Wang Fixes: bd7525dacd7e ("bpf: Move stack_map_get_build_id into lib") Signed-off-by: Stephen Boyd --- lib/buildid.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/buildid.c b/lib/buildid.c index 6156997c3895..e014636ec3eb 100644 ---

  1   2   3   4   5   6   7   8   9   10   >