Re: [PATCH] riscv: Fix early ftrace nop patching

2024-05-23 Thread Conor Dooley
since > the booting core could have half-patched instructions in its icache > which would trigger an illegal instruction trap: fix this by emitting a > local flush icache when early patching nops. > > Fixes: c97bf629963e ("riscv: Fix text patching when IPI are used") > Signed-o

Re: [PATCH v8 1/4] dt-bindings: remoteproc: k3-m4f: Add K3 AM64x SoCs

2024-04-26 Thread Conor Dooley
On Thu, Apr 25, 2024 at 02:03:36PM -0500, Andrew Davis wrote: > On 4/25/24 12:15 PM, Conor Dooley wrote: > > On Wed, Apr 24, 2024 at 03:36:39PM -0500, Rob Herring wrote: > > > > > > On Wed, 24 Apr 2024 14:06:09 -0500, Andrew Davis wrote: > > > > From: Hari

Re: [PATCH v8 1/4] dt-bindings: remoteproc: k3-m4f: Add K3 AM64x SoCs

2024-04-25 Thread Conor Dooley
Welch > Signed-off-by: Hari Nagalla > Signed-off-by: Andrew Davis > + mboxes: > +description: | > + memory-region: > +description: | Not sure that either chomping operator is needed here, but that's a nit. With the incorrect link fixed up, Reviewed-by: Conor Dooley Cheers, Conor. signature.asc Description: PGP signature

Re: [PATCH v8 1/4] dt-bindings: remoteproc: k3-m4f: Add K3 AM64x SoCs

2024-04-25 Thread Conor Dooley
On Wed, Apr 24, 2024 at 03:36:39PM -0500, Rob Herring wrote: > > On Wed, 24 Apr 2024 14:06:09 -0500, Andrew Davis wrote: > > From: Hari Nagalla > > > > K3 AM64x SoC has a Cortex M4F subsystem in the MCU voltage domain. > > The remote processor's life cycle management and IPC mechanisms are > >

Re: [PATCH v7 2/2] arch/riscv: Enable kprobes when CONFIG_MODULES=n

2024-03-26 Thread Conor Dooley
On Tue, Mar 26, 2024 at 03:46:16PM +0200, Jarkko Sakkinen wrote: > Tacing with kprobes while running a monolithic kernel is currently > impossible due the kernel module allocator dependency. > > Address the issue by implementing textmem API for RISC-V. This doesn't compile for nommu:

Re: [PATCH 1/5] dt-bindings: regulator: qcom,usb-vbus-regulator: Add PM7250B compatible

2024-03-22 Thread Conor Dooley
On Fri, Mar 22, 2024 at 09:01:32AM +0100, Luca Weiss wrote: > The VBUS register block on the PM6150 PMIC shares the design with the > PM8150B one. Define corresponding compatible string, having the > qcom,pm8150b-vbus-reg as a fallback. > > Signed-off-by: Luca Weiss Acked-b

Re: [PATCH 2/5] dt-bindings: usb: qcom,pmic-typec: Add support for the PM7250B PMIC

2024-03-22 Thread Conor Dooley
On Fri, Mar 22, 2024 at 09:01:33AM +0100, Luca Weiss wrote: > The PM6150 PMIC has the same Type-C register block as the PM8150B. > Define corresponding compatible string, having the qcom,pm8150b-vbus-reg > as a fallback. > > Signed-off-by: Luca Weiss Acked-by: Conor Dooley

Re: [PATCH 1/3] dt-bindings: remoteproc: add Versal platform support

2024-03-17 Thread Conor Dooley
On Sun, Mar 17, 2024 at 02:50:27PM +, Conor Dooley wrote: > On Fri, Mar 15, 2024 at 02:15:31PM -0700, Tanmay Shah wrote: > > AMD-Xilinx Versal platform is successor of ZynqMP platform. Real-time > > Processor Unit R5 cluster IP on Versal is same as of ZynqMP Platform. > &

Re: [PATCH 1/3] dt-bindings: remoteproc: add Versal platform support

2024-03-17 Thread Conor Dooley
On Fri, Mar 15, 2024 at 02:15:31PM -0700, Tanmay Shah wrote: > AMD-Xilinx Versal platform is successor of ZynqMP platform. Real-time > Processor Unit R5 cluster IP on Versal is same as of ZynqMP Platform. > Only difference is power-domains ID needed by power management firmware. > Hence, keeping

Re: [PATCH v3 2/2] riscv: Fix text patching when IPI are used

2024-03-05 Thread Conor Dooley
On Tue, Mar 05, 2024 at 08:33:30AM +0530, Anup Patel wrote: > On Tue, Mar 5, 2024 at 1:54 AM Björn Töpel wrote: > > > > Conor Dooley writes: > > > > > On Thu, Feb 29, 2024 at 01:10:56PM +0100, Alexandre Ghiti wrote: > > >> For now, we use stop_machin

Re: [PATCH v3 2/2] riscv: Fix text patching when IPI are used

2024-03-04 Thread Conor Dooley
On Thu, Feb 29, 2024 at 01:10:56PM +0100, Alexandre Ghiti wrote: > For now, we use stop_machine() to patch the text and when we use IPIs for > remote icache flushes (which is emitted in patch_text_nosync()), the system > hangs. > > So instead, make sure every CPU executes the stop_machine()

Re: [PATCH v3 1/2] riscv: Remove superfluous smp_mb()

2024-03-04 Thread Conor Dooley
On Thu, Feb 29, 2024 at 01:10:55PM +0100, Alexandre Ghiti wrote: > This memory barrier is not needed and not documented so simply remove > it. This looks like it should be patch 2 in the series, not patch 1, as it is cleanup rather than a fix that needs backporting. > > Suggested-by: Andrea

Re: [PATCH v2 1/2] dt-bindings: arm: qcom: Document samsung,milletwifi device

2024-02-15 Thread Conor Dooley
On Thu, Feb 15, 2024 at 06:19:48PM +0100, Bryant Mairs wrote: > Add binding documentation for Samsung Galaxy Tab 4 8.0 Wi-Fi > tablet which is based on Snapdragon 400 (apq8026) SoC. > > Signed-off-by: Bryant Mairs > Acked-by: Linus Walleij Acked-by: Conor Dooley

Re: [PATCH] media: dt-bindings: qcom,sc7280-venus: Allow one IOMMU entry

2024-01-29 Thread Conor Dooley
On Mon, Jan 29, 2024 at 08:48:54AM +0100, Luca Weiss wrote: > Some SC7280-based boards crash when providing the "secure_non_pixel" > context bank, so allow only one iommu in the bindings also. > > Signed-off-by: Luca Weiss Do we have any idea why this happens? How is someone supposed to know

Re: [PATCH] dt-bindings: remoteproc: do not override firmware-name $ref

2024-01-16 Thread Conor Dooley
Acked-by: Conor Dooley signature.asc Description: PGP signature

Re: [PATCH v3 2/5] dt-bindings: input/touchscreen: Add compatible for IST3038B

2023-12-09 Thread Conor Dooley
On Sat, Dec 09, 2023 at 10:05:27AM +0100, Karel Balej wrote: > On Mon Dec 4, 2023 at 1:52 PM CET, Conor Dooley wrote: > > On Mon, Dec 04, 2023 at 02:40:44PM +0200, Markuss Broks wrote: > > > On 12/3/23 13:20, Conor Dooley wrote: > > > > On Sat, Dec 02, 2023 at 01:48

Re: [PATCH v3 2/5] dt-bindings: input/touchscreen: Add compatible for IST3038B

2023-12-04 Thread Conor Dooley
On Mon, Dec 04, 2023 at 02:40:44PM +0200, Markuss Broks wrote: > On 12/3/23 13:20, Conor Dooley wrote: > > On Sat, Dec 02, 2023 at 01:48:33PM +0100, Karel Balej wrote: > > > From: Markuss Broks > > > > > > Imagis IST3038B is a variant (firmware?) of Imagis

Re: [PATCH v3 2/5] dt-bindings: input/touchscreen: Add compatible for IST3038B

2023-12-03 Thread Conor Dooley
On Sat, Dec 02, 2023 at 01:48:33PM +0100, Karel Balej wrote: > From: Markuss Broks > > Imagis IST3038B is a variant (firmware?) of Imagis IST3038 IC, > add the compatible for it to the IST3038C bindings. This one is better, but would be well served by mentioning what specifically is different

Re: [PATCH v3 4/5] dt-bindings: input/touchscreen: imagis: add compatible for IST3032C

2023-12-03 Thread Conor Dooley
On Sat, Dec 02, 2023 at 01:48:35PM +0100, Karel Balej wrote: > From: Karel Balej > > Document possible usage of the Imagis driver with the IST3032C > touchscreen. Please leave mention of the driver out of the binding patch (we deal only with the hardware here) and instead describe what is

Re: [PATCH 1/2] dt-bindings: arm: qcom: Add Huawei Honor 5X / GR5 (2016)

2023-10-22 Thread Conor Dooley
On Sun, Oct 22, 2023 at 01:40:41PM +0200, lukas walter wrote: > >> Acked-by: Krzysztof Kozlowski > > > >How come this v1 has an ack? > > This is supposed to be a v2, but I had problems sending it correctly, > sorry >

Re: [PATCH 1/2] dt-bindings: arm: qcom: Add Huawei Honor 5X / GR5 (2016)

2023-10-22 Thread Conor Dooley
On Sat, Oct 21, 2023 at 04:30:24PM +0200, Lukas Walter wrote: > Add a compatible for Huawei Honor 5X / GR5 (2016). > > Acked-by: Krzysztof Kozlowski How come this v1 has an ack? > Signed-off-by: Lukas Walter > --- > Documentation/devicetree/bindings/arm/qcom.yaml | 1 + > 1 file changed, 1

Re: [PATCH v2 1/2] dt-bindings: i2c: qcom-cci: Document SC7280 compatible

2023-10-02 Thread Conor Dooley
On Mon, Oct 02, 2023 at 08:55:30AM +0200, Luca Weiss wrote: > Document the compatible for the CCI block found on SC7280 SoC. > > Reviewed-by: Bryan O'Donoghue > Signed-off-by: Luca Weiss Acked-by: Conor Dooley Thanks, Conor. > --- > Documentation/devicetree/bindings/i2c

Re: [PATCH v2 1/2] dt-bindings: mfd: qcom,spmi-pmic: Drop unused labels from examples

2023-10-02 Thread Conor Dooley
;) updating all dts files to > use the plural _gpios label instead of the singular _gpio as label but > this example wasn't updated. But since we should just drop the label > alltogether, do that. > > Signed-off-by: Luca Weiss Acked-by: Conor Dooley Thanks, Conor. > --- > D

Re: [PATCH] dt-bindings: remoteproc: mtk,scp: Add missing additionalProperties on child node schemas

2023-09-27 Thread Conor Dooley
for any node. > > Signed-off-by: Rob Herring > --- Acked-by: Conor Dooley > Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml > b/Documentatio