From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
To: Eric Chanudet <echanude@redhat.com>
Cc: Bartosz Golaszewski <brgl@bgdev.pl>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <andersson@kernel.org>,
Konrad Dybcio <konrad.dybcio@linaro.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Manivannan Sadhasivam <mani@kernel.org>,
Jassi Brar <jassisinghbrar@gmail.com>,
Wim Van Sebroeck <wim@linux-watchdog.org>,
Guenter Roeck <linux@roeck-us.net>,
linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-watchdog@vger.kernel.org
Subject: Re: [PATCH v3 5/5] arm64: dts: qcom: add initial support for qcom sa8775p-ride
Date: Thu, 9 Feb 2023 20:52:11 +0100 [thread overview]
Message-ID: <CACMJSevxZ0_40MwqVSm401=us6dwpKYcuOzFW+Ypks3Ri4eReg@mail.gmail.com> (raw)
In-Reply-To: <20230209180456.eahx5fbcd2kax2au@echanude>
On Thu, 9 Feb 2023 at 19:05, Eric Chanudet <echanude@redhat.com> wrote:
>
> On Wed, Feb 01, 2023 at 04:20:38PM +0100, Bartosz Golaszewski wrote:
> > From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> >
> > This adds basic support for the Qualcomm sa8775p platform and the
> > reference board: sa8775p-ride. The dt files describe the basics of the
> > SoC and enable booting to shell.
> >
> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> > Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> > ---
[...]
> > +
> > + watchdog@17c10000 {
> > + compatible = "qcom,apss-wdt-sa8775p", "qcom,kpss-wdt";
> > + reg = <0x0 0x17c10000 0x0 0x1000>;
> > + clocks = <&sleep_clk>;
> > + interrupts = <GIC_SPI 0 IRQ_TYPE_LEVEL_HIGH>;
> > + };
>
> Trying this DTS on sa8775p-ride with next-20230209, I get this splat
> with the qcom_wdt:
>
> [ 2.667479] Internal error: synchronous external abort: 0000000096000010 [#1] PREEMPT SMP
> [ 2.675911] Modules linked in: qcom_wdt(+) crct10dif_ce socinfo fuse ipv6
> [ 2.675931] CPU: 5 PID: 155 Comm: systemd-udevd Not tainted 6.2.0-rc7-next-20230209-00011-g897899f9de9e #80
> [ 2.675936] Hardware name: Qualcomm SA8775P Ride (DT)
> [ 2.675938] pstate: 60400005 (nZCv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--)
> [ 2.675942] pc : qcom_wdt_probe+0x20c/0x388 [qcom_wdt]
> [ 2.675964] lr : qcom_wdt_probe+0x308/0x388 [qcom_wdt]
> [ 2.675970] sp : ffff80000a4db8e0
> [ 2.675971] x29: ffff80000a4db8e0 x28: 0000000000000000 x27: ffff80000a4dbd50
> [ 2.675975] x26: 0000000000000000 x25: ffff80000a4dbcd0 x24: ffff060041723680
> [ 2.675978] x23: ffff060040c2b400 x22: ffffdc111597c560 x21: ffff060040c2b410
> [ 2.675981] x20: ffff060040959e80 x19: 0000000000000000 x18: ffffffffffffffff
> [ 2.675984] x17: 0000000000000000 x16: ffffdc116777c59c x15: ffff80000a4db670
> [ 2.675987] x14: ffff80008a4db7cd x13: 007473696c5f7974 x12: 0000000000000001
> [ 2.675990] x11: 0000000000000040 x10: ffffdc1169893790 x9 : 0000000000000000
> [ 2.675994] x8 : ffff0600414ffd80 x7 : 0000000000000000 x6 : 0000000000000000
> [ 2.675997] x5 : ffff0600414ffc88 x4 : ffff060040c2b610 x3 : 0000000000007ffc
> [ 2.676001] x2 : 0000000000000001 x1 : 000000000000000c x0 : ffff800008f5d00c
> [ 2.676004] Call trace:
> [ 2.676006] qcom_wdt_probe+0x20c/0x388 [qcom_wdt]
> [ 2.676013] platform_probe+0x68/0xc4
> [ 2.676026] really_probe+0x148/0x2ac
> [ 2.676038] __driver_probe_device+0x78/0xe0
> [ 2.676042] driver_probe_device+0x3c/0x15c
> [ 2.676045] __driver_attach+0x94/0x19c
> [ 2.676048] bus_for_each_dev+0x74/0xcc
> [ 2.676052] driver_attach+0x24/0x30
> [ 2.676055] bus_add_driver+0x104/0x204
> [ 2.676059] driver_register+0x68/0x12c
> [ 2.676063] __platform_driver_register+0x28/0x34
> [ 2.676065] qcom_watchdog_driver_init+0x20/0x1000 [qcom_wdt]
> [ 2.676072] do_one_initcall+0x80/0x1c4
> [ 2.676081] do_init_module+0x58/0x1e8
> [ 2.676088] load_module+0x19d4/0x1a18
> [ 2.676091] __do_sys_finit_module+0xa8/0xfc
> [ 2.676095] __arm64_sys_finit_module+0x20/0x2c
> [ 2.676098] invoke_syscall+0x48/0x114
> [ 2.676107] el0_svc_common.constprop.0+0xcc/0xec
> [ 2.676110] do_el0_svc+0x38/0x98
> [ 2.676113] el0_svc+0x2c/0x84
> [ 2.676121] el0t_64_sync_handler+0xf4/0x120
> [ 2.676123] el0t_64_sync+0x190/0x194
> [ 2.676129] Code: f94002c1 f9006a81 b9400821 8b010000 (b9400000)
> [ 2.676135] ---[ end trace 0000000000000000 ]---
>
> I most likely did not see it before as I either had qcom_wdt configured
> out or disabled on the kernel cmdline.
>
On Bjorn's board I can probe the watchdog and even trigger it from
user-space (although it doesn't seem to reset the board, it just
freezes - not sure if that's not Bjorn's setup as rebooting the board
from console doesn't work either).
Can this be the hypervisor's doing?
Bart
[...]
next prev parent reply other threads:[~2023-02-09 19:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-01 15:20 [PATCH v3 0/5] arm64: qcom: add initial support for qcom sa8775p-ride Bartosz Golaszewski
2023-02-01 15:20 ` [PATCH v3 1/5] dt-bindings: firmware: qcom,scm: add qcom,scm-sa8775p compatible Bartosz Golaszewski
2023-02-01 15:20 ` [PATCH v3 2/5] dt-bindings: mailbox: qcom-ipcc: document the sa8775p platform Bartosz Golaszewski
2023-02-01 18:00 ` Krzysztof Kozlowski
2023-02-01 15:20 ` [PATCH v3 3/5] dt-bindings: watchdog: qcom-wdt: add qcom,apss-wdt-sa8775p compatible Bartosz Golaszewski
2023-02-01 15:36 ` Guenter Roeck
2023-02-01 15:20 ` [PATCH v3 4/5] dt-bindings: arm: qcom: add the sa8775p-ride board Bartosz Golaszewski
2023-02-01 15:20 ` [PATCH v3 5/5] arm64: dts: qcom: add initial support for qcom sa8775p-ride Bartosz Golaszewski
2023-02-01 18:02 ` Krzysztof Kozlowski
2023-02-09 18:04 ` Eric Chanudet
2023-02-09 19:52 ` Bartosz Golaszewski [this message]
2023-02-14 9:28 ` Bartosz Golaszewski
2023-02-06 21:08 ` [PATCH v3 0/5] arm64: " Bartosz Golaszewski
2023-03-15 23:34 ` (subset) " Bjorn Andersson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CACMJSevxZ0_40MwqVSm401=us6dwpKYcuOzFW+Ypks3Ri4eReg@mail.gmail.com' \
--to=bartosz.golaszewski@linaro.org \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=brgl@bgdev.pl \
--cc=devicetree@vger.kernel.org \
--cc=echanude@redhat.com \
--cc=jassisinghbrar@gmail.com \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=mani@kernel.org \
--cc=robh+dt@kernel.org \
--cc=wim@linux-watchdog.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).