This series fixes the last few remaining issues reported when fw_devlink=on
by default.
Patch 1 is just [6] pulled in without changes into this series. It reduces
some unnecessary probe reordering caused by a combination of fw_devlink and
existing device link code. This fixes some issue caused by fw_devlink=on
with respect to DMAs and IOMMUs [1].
Patch 2 fixes a warning [2] present in code unrelated to fw_devlink. It was
just exposed by fw_devlink.
Jon,
Patch 2 should address the issues you reported[2] even without [3]. Could
you test this series please?
Michael,
I think Patch 1 should fix [4] without [5]. Can you test the series please?
Geert/Marek,
As far as I know, there shouldn't have any more issues you reported that
are still left unfixed after this series. Please correct me if I'm wrong or
if you find new issues.
[1] - https://lore.kernel.org/lkml/CAMuHMdUVVr8jES51_8_yPoicr-nwad_2nKLYUKweY8mbxx9GJw@mail.gmail.com/
[2] - https://lore.kernel.org/lkml/[email protected]/
[3] - https://lore.kernel.org/lkml/[email protected]/
[4] - https://lore.kernel.org/lkml/[email protected]/
[5] - https://lore.kernel.org/lkml/[email protected]/
[6] - https://lore.kernel.org/lkml/[email protected]/
Cc: Michael Walle <[email protected]>
Cc: Jon Hunter <[email protected]>
Cc: Marek Szyprowski <[email protected]>
Cc: Geert Uytterhoeven <[email protected]>
Cc: Guenter Roeck <[email protected]>
Thanks,
Saravana
Saravana Kannan (3):
driver core: Avoid pointless deferred probe attempts
driver core: Update device link status properly for
device_bind_driver()
Revert "Revert "driver core: Set fw_devlink=on by default""
drivers/base/base.h | 1 +
drivers/base/core.c | 37 ++++++++++++++++++++++++++++++++++++-
drivers/base/dd.c | 10 +++++++++-
include/linux/device.h | 4 ++++
4 files changed, 50 insertions(+), 2 deletions(-)
--
2.30.1.766.gb4fecdf3b7-goog
Device link status was not getting updated correctly when
device_bind_driver() is called on a device. This causes a warning[1].
Fix this by updating device links that can be updated and dropping
device links that can't be updated to a sensible state.
[1] - https://lore.kernel.org/lkml/[email protected]/
Signed-off-by: Saravana Kannan <[email protected]>
---
drivers/base/base.h | 1 +
drivers/base/core.c | 35 +++++++++++++++++++++++++++++++++++
drivers/base/dd.c | 4 +++-
3 files changed, 39 insertions(+), 1 deletion(-)
diff --git a/drivers/base/base.h b/drivers/base/base.h
index 52b3d7b75c27..1b44ed588f66 100644
--- a/drivers/base/base.h
+++ b/drivers/base/base.h
@@ -185,6 +185,7 @@ extern int device_links_read_lock(void);
extern void device_links_read_unlock(int idx);
extern int device_links_read_lock_held(void);
extern int device_links_check_suppliers(struct device *dev);
+extern void device_links_force_bind(struct device *dev);
extern void device_links_driver_bound(struct device *dev);
extern void device_links_driver_cleanup(struct device *dev);
extern void device_links_no_driver(struct device *dev);
diff --git a/drivers/base/core.c b/drivers/base/core.c
index f29839382f81..45c75cc96fdc 100644
--- a/drivers/base/core.c
+++ b/drivers/base/core.c
@@ -1153,6 +1153,41 @@ static ssize_t waiting_for_supplier_show(struct device *dev,
}
static DEVICE_ATTR_RO(waiting_for_supplier);
+/**
+ * device_links_force_bind - Prepares device to be force bound
+ * @dev: Consumer device.
+ *
+ * device_bind_driver() force binds a device to a driver without calling any
+ * driver probe functions. So the consumer really isn't going to wait for any
+ * supplier before it's bound to the driver. We still want the device link
+ * states to be sensible when this happens.
+ *
+ * In preparation for device_bind_driver(), this function goes through each
+ * supplier device links and checks if the supplier is bound. If it is, then
+ * the device link status is set to CONSUMER_PROBE. Otherwise, the device link
+ * is dropped. Links without the DL_FLAG_MANAGED flag set are ignored.
+ */
+void device_links_force_bind(struct device *dev)
+{
+ struct device_link *link, *ln;
+
+ device_links_write_lock();
+
+ list_for_each_entry_safe(link, ln, &dev->links.suppliers, c_node) {
+ if (!(link->flags & DL_FLAG_MANAGED))
+ continue;
+
+ if (link->status != DL_STATE_AVAILABLE) {
+ device_link_drop_managed(link);
+ continue;
+ }
+ WRITE_ONCE(link->status, DL_STATE_CONSUMER_PROBE);
+ }
+ dev->links.status = DL_DEV_PROBING;
+
+ device_links_write_unlock();
+}
+
/**
* device_links_driver_bound - Update device links after probing its driver.
* @dev: Device to update the links for.
diff --git a/drivers/base/dd.c b/drivers/base/dd.c
index f18963f42e21..eb201c6d5a6a 100644
--- a/drivers/base/dd.c
+++ b/drivers/base/dd.c
@@ -460,8 +460,10 @@ int device_bind_driver(struct device *dev)
int ret;
ret = driver_sysfs_add(dev);
- if (!ret)
+ if (!ret) {
+ device_links_force_bind(dev);
driver_bound(dev);
+ }
else if (dev->bus)
blocking_notifier_call_chain(&dev->bus->p->bus_notifier,
BUS_NOTIFY_DRIVER_NOT_BOUND, dev);
--
2.30.1.766.gb4fecdf3b7-goog
There's no point in adding a device to the deferred probe list if we
know for sure that it doesn't have a matching driver. So, check if a
device can match with a driver before adding it to the deferred probe
list.
Signed-off-by: Saravana Kannan <[email protected]>
---
drivers/base/dd.c | 6 ++++++
include/linux/device.h | 4 ++++
2 files changed, 10 insertions(+)
diff --git a/drivers/base/dd.c b/drivers/base/dd.c
index 9179825ff646..f18963f42e21 100644
--- a/drivers/base/dd.c
+++ b/drivers/base/dd.c
@@ -123,6 +123,9 @@ static DECLARE_WORK(deferred_probe_work, deferred_probe_work_func);
void driver_deferred_probe_add(struct device *dev)
{
+ if (!dev->can_match)
+ return;
+
mutex_lock(&deferred_probe_mutex);
if (list_empty(&dev->p->deferred_probe)) {
dev_dbg(dev, "Added to deferred list\n");
@@ -726,6 +729,7 @@ static int driver_probe_device(struct device_driver *drv, struct device *dev)
if (!device_is_registered(dev))
return -ENODEV;
+ dev->can_match = true;
pr_debug("bus: '%s': %s: matched device %s with driver %s\n",
drv->bus->name, __func__, dev_name(dev), drv->name);
@@ -829,6 +833,7 @@ static int __device_attach_driver(struct device_driver *drv, void *_data)
return 0;
} else if (ret == -EPROBE_DEFER) {
dev_dbg(dev, "Device match requests probe deferral\n");
+ dev->can_match = true;
driver_deferred_probe_add(dev);
} else if (ret < 0) {
dev_dbg(dev, "Bus failed to match device: %d\n", ret);
@@ -1064,6 +1069,7 @@ static int __driver_attach(struct device *dev, void *data)
return 0;
} else if (ret == -EPROBE_DEFER) {
dev_dbg(dev, "Device match requests probe deferral\n");
+ dev->can_match = true;
driver_deferred_probe_add(dev);
} else if (ret < 0) {
dev_dbg(dev, "Bus failed to match device: %d\n", ret);
diff --git a/include/linux/device.h b/include/linux/device.h
index ba660731bd25..569932d282c0 100644
--- a/include/linux/device.h
+++ b/include/linux/device.h
@@ -439,6 +439,9 @@ struct dev_links_info {
* @state_synced: The hardware state of this device has been synced to match
* the software state of this device by calling the driver/bus
* sync_state() callback.
+ * @can_match: The device has matched with a driver at least once or it is in
+ * a bus (like AMBA) which can't check for matching drivers until
+ * other devices probe successfully.
* @dma_coherent: this particular device is dma coherent, even if the
* architecture supports non-coherent devices.
* @dma_ops_bypass: If set to %true then the dma_ops are bypassed for the
@@ -545,6 +548,7 @@ struct device {
bool offline:1;
bool of_node_reused:1;
bool state_synced:1;
+ bool can_match:1;
#if defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_DEVICE) || \
defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_CPU) || \
defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_CPU_ALL)
--
2.30.1.766.gb4fecdf3b7-goog
This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
Since all reported issues due to fw_devlink=on should be addressed by
this series, revert the revert. fw_devlink=on Take II.
Signed-off-by: Saravana Kannan <[email protected]>
---
drivers/base/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/base/core.c b/drivers/base/core.c
index 45c75cc96fdc..de518178ac36 100644
--- a/drivers/base/core.c
+++ b/drivers/base/core.c
@@ -1538,7 +1538,7 @@ static void device_links_purge(struct device *dev)
#define FW_DEVLINK_FLAGS_RPM (FW_DEVLINK_FLAGS_ON | \
DL_FLAG_PM_RUNTIME)
-static u32 fw_devlink_flags = FW_DEVLINK_FLAGS_PERMISSIVE;
+static u32 fw_devlink_flags = FW_DEVLINK_FLAGS_ON;
static int __init fw_devlink_setup(char *arg)
{
if (!arg)
--
2.30.1.766.gb4fecdf3b7-goog
Am 2021-03-02 22:11, schrieb Saravana Kannan:
> I think Patch 1 should fix [4] without [5]. Can you test the series
> please?
Mh, I'm on latest linux-next (next-20210302) and I've applied patch 3/3
and
reverted commit 7007b745a508 ("PCI: layerscape: Convert to
builtin_platform_driver()"). I'd assumed that PCIe shouldn't be working,
right? But it is. Did I miss something?
Anyway, I've also applied Patch 1/3 and 2/3 and it still works. But I
guess that doesn't say much.
-michael
On Tue, Mar 2, 2021 at 2:24 PM Michael Walle <[email protected]> wrote:
>
> Am 2021-03-02 22:11, schrieb Saravana Kannan:
> > I think Patch 1 should fix [4] without [5]. Can you test the series
> > please?
>
> Mh, I'm on latest linux-next (next-20210302) and I've applied patch 3/3
> and
> reverted commit 7007b745a508 ("PCI: layerscape: Convert to
> builtin_platform_driver()"). I'd assumed that PCIe shouldn't be working,
> right? But it is. Did I miss something?
You need to revert [5].
-Saravana
>
> Anyway, I've also applied Patch 1/3 and 2/3 and it still works. But I
> guess that doesn't say much.
>
> -michael
On Tue, Mar 2, 2021 at 2:42 PM Saravana Kannan <[email protected]> wrote:
>
> On Tue, Mar 2, 2021 at 2:24 PM Michael Walle <[email protected]> wrote:
> >
> > Am 2021-03-02 22:11, schrieb Saravana Kannan:
> > > I think Patch 1 should fix [4] without [5]. Can you test the series
> > > please?
> >
> > Mh, I'm on latest linux-next (next-20210302) and I've applied patch 3/3
> > and
> > reverted commit 7007b745a508 ("PCI: layerscape: Convert to
> > builtin_platform_driver()"). I'd assumed that PCIe shouldn't be working,
> > right? But it is. Did I miss something?
>
> You need to revert [5].
My bad. You did revert it. Ah... I wonder if it was due to
fw_devlink.strict that I added. To break PCI again, also set
fw_devlink.strict=1 in the kernel command line.
-Saravana
On Wed, Mar 3, 2021 at 12:59 AM Michael Walle <[email protected]> wrote:
>
> Am 2021-03-02 23:47, schrieb Saravana Kannan:
> > On Tue, Mar 2, 2021 at 2:42 PM Saravana Kannan <[email protected]>
> > wrote:
> >>
> >> On Tue, Mar 2, 2021 at 2:24 PM Michael Walle <[email protected]> wrote:
> >> >
> >> > Am 2021-03-02 22:11, schrieb Saravana Kannan:
> >> > > I think Patch 1 should fix [4] without [5]. Can you test the series
> >> > > please?
> >> >
> >> > Mh, I'm on latest linux-next (next-20210302) and I've applied patch 3/3
> >> > and
> >> > reverted commit 7007b745a508 ("PCI: layerscape: Convert to
> >> > builtin_platform_driver()"). I'd assumed that PCIe shouldn't be working,
> >> > right? But it is. Did I miss something?
> >>
> >> You need to revert [5].
> >
> > My bad. You did revert it. Ah... I wonder if it was due to
> > fw_devlink.strict that I added. To break PCI again, also set
> > fw_devlink.strict=1 in the kernel command line.
>
> Indeed, adding fw_devlink.strict=1 will break PCI again. But if
> I then apply 1/3 and 2/3 again, PCI is still broken. Just to be clear:
> I'm keeping the fw_devlink.strict=1 parameter.
Thanks for your testing! I assume you are also setting fw_devlink=on?
Hmmm... ok. In the working case, does your PCI probe before IOMMU? If
yes, then your results make sense.
If your PCI does probe after IOMMU and uses IOMMU, then I'm not sure
what else could be changing the order of the device probing. In any
case, glad that the default case works and we have a fix merged even
for .strict=1.
-Saravana
On Wed, Mar 3, 2021 at 1:22 AM Geert Uytterhoeven <[email protected]> wrote:
>
> Hi Saravana,
>
> On Tue, Mar 2, 2021 at 10:11 PM Saravana Kannan <[email protected]> wrote:
> > This series fixes the last few remaining issues reported when fw_devlink=on
> > by default.
>
> [...]
>
> Thanks for your series!
>
> > Geert/Marek,
> >
> > As far as I know, there shouldn't have any more issues you reported that
> > are still left unfixed after this series. Please correct me if I'm wrong or
> > if you find new issues.
>
> While this fixes the core support, there may still be driver fixes left
> that were not developed in time for the v5.12-rc1 merge window.
> Personally, I'm aware of "soc: renesas: rmobile-sysc: Mark fwnode
> when PM domain is added", which I have queued for v5.13[1].
> There may be other fixes for other platforms.
Right, I intended this series for 5.13. Is that what you are trying to say too?
-Saravana
>
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-devel.git/commit/?h=renesas-drivers-for-v5.13&id=fb13bbd6c90ee4fb983c0e9a341bd2832a3857cf
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
Am 2021-03-03 10:28, schrieb Saravana Kannan:
> On Wed, Mar 3, 2021 at 12:59 AM Michael Walle <[email protected]> wrote:
>>
>> Am 2021-03-02 23:47, schrieb Saravana Kannan:
>> > On Tue, Mar 2, 2021 at 2:42 PM Saravana Kannan <[email protected]>
>> > wrote:
>> >>
>> >> On Tue, Mar 2, 2021 at 2:24 PM Michael Walle <[email protected]> wrote:
>> >> >
>> >> > Am 2021-03-02 22:11, schrieb Saravana Kannan:
>> >> > > I think Patch 1 should fix [4] without [5]. Can you test the series
>> >> > > please?
>> >> >
>> >> > Mh, I'm on latest linux-next (next-20210302) and I've applied patch 3/3
>> >> > and
>> >> > reverted commit 7007b745a508 ("PCI: layerscape: Convert to
>> >> > builtin_platform_driver()"). I'd assumed that PCIe shouldn't be working,
>> >> > right? But it is. Did I miss something?
>> >>
>> >> You need to revert [5].
>> >
>> > My bad. You did revert it. Ah... I wonder if it was due to
>> > fw_devlink.strict that I added. To break PCI again, also set
>> > fw_devlink.strict=1 in the kernel command line.
>>
>> Indeed, adding fw_devlink.strict=1 will break PCI again. But if
>> I then apply 1/3 and 2/3 again, PCI is still broken. Just to be clear:
>> I'm keeping the fw_devlink.strict=1 parameter.
>
> Thanks for your testing! I assume you are also setting fw_devlink=on?
I've applied patch 3/3 and added nothing to the commandline, so yes.
> Hmmm... ok. In the working case, does your PCI probe before IOMMU? If
> yes, then your results make sense.
Yes that was the conclusion last time. That the probe is deferred and
the __init section is already discarded when there might a second
try of the probe.
So I guess, Patch 1/3 and Patch 2/3 doesn't fix that and the drivers
still need to be converted to builtin_platform_driver(), right?
> If your PCI does probe after IOMMU and uses IOMMU, then I'm not sure
> what else could be changing the order of the device probing. In any
> case, glad that the default case works and we have a fix merged even
> for .strict=1.
>
> -Saravana
-michael
On Wed, Mar 3, 2021 at 2:03 AM Geert Uytterhoeven <[email protected]> wrote:
>
> Hi Saravana,
>
> On Wed, Mar 3, 2021 at 10:24 AM Saravana Kannan <[email protected]> wrote:
> > On Wed, Mar 3, 2021 at 1:22 AM Geert Uytterhoeven <[email protected]> wrote:
> > > On Tue, Mar 2, 2021 at 10:11 PM Saravana Kannan <[email protected]> wrote:
> > > > This series fixes the last few remaining issues reported when fw_devlink=on
> > > > by default.
> > >
> > > [...]
> > >
> > > Thanks for your series!
> > >
> > > > Geert/Marek,
> > > >
> > > > As far as I know, there shouldn't have any more issues you reported that
> > > > are still left unfixed after this series. Please correct me if I'm wrong or
> > > > if you find new issues.
> > >
> > > While this fixes the core support, there may still be driver fixes left
> > > that were not developed in time for the v5.12-rc1 merge window.
> > > Personally, I'm aware of "soc: renesas: rmobile-sysc: Mark fwnode
> > > when PM domain is added", which I have queued for v5.13[1].
> > > There may be other fixes for other platforms.
> >
> > Right, I intended this series for 5.13. Is that what you are trying to say too?
>
> OK, v5.13 is fine for me.
> It wasn't clear to me if you intended (the last patch of) this series to
> be merged for v5.12-rcX or v5.13.
The entire series is meant for 5.13.
I don't want to land the Patch 1/3 in 5.12 in case it causes some
regression. And 2/3 isn't urgent.
-Saravana
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
Am 2021-03-02 23:47, schrieb Saravana Kannan:
> On Tue, Mar 2, 2021 at 2:42 PM Saravana Kannan <[email protected]>
> wrote:
>>
>> On Tue, Mar 2, 2021 at 2:24 PM Michael Walle <[email protected]> wrote:
>> >
>> > Am 2021-03-02 22:11, schrieb Saravana Kannan:
>> > > I think Patch 1 should fix [4] without [5]. Can you test the series
>> > > please?
>> >
>> > Mh, I'm on latest linux-next (next-20210302) and I've applied patch 3/3
>> > and
>> > reverted commit 7007b745a508 ("PCI: layerscape: Convert to
>> > builtin_platform_driver()"). I'd assumed that PCIe shouldn't be working,
>> > right? But it is. Did I miss something?
>>
>> You need to revert [5].
>
> My bad. You did revert it. Ah... I wonder if it was due to
> fw_devlink.strict that I added. To break PCI again, also set
> fw_devlink.strict=1 in the kernel command line.
Indeed, adding fw_devlink.strict=1 will break PCI again. But if
I then apply 1/3 and 2/3 again, PCI is still broken. Just to be clear:
I'm keeping the fw_devlink.strict=1 parameter.
-michael
Hi Saravana,
On Tue, Mar 2, 2021 at 10:11 PM Saravana Kannan <[email protected]> wrote:
> This series fixes the last few remaining issues reported when fw_devlink=on
> by default.
[...]
Thanks for your series!
> Geert/Marek,
>
> As far as I know, there shouldn't have any more issues you reported that
> are still left unfixed after this series. Please correct me if I'm wrong or
> if you find new issues.
While this fixes the core support, there may still be driver fixes left
that were not developed in time for the v5.12-rc1 merge window.
Personally, I'm aware of "soc: renesas: rmobile-sysc: Mark fwnode
when PM domain is added", which I have queued for v5.13[1].
There may be other fixes for other platforms.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-devel.git/commit/?h=renesas-drivers-for-v5.13&id=fb13bbd6c90ee4fb983c0e9a341bd2832a3857cf
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Hi Saravana,
On Wed, Mar 3, 2021 at 10:24 AM Saravana Kannan <[email protected]> wrote:
> On Wed, Mar 3, 2021 at 1:22 AM Geert Uytterhoeven <[email protected]> wrote:
> > On Tue, Mar 2, 2021 at 10:11 PM Saravana Kannan <[email protected]> wrote:
> > > This series fixes the last few remaining issues reported when fw_devlink=on
> > > by default.
> >
> > [...]
> >
> > Thanks for your series!
> >
> > > Geert/Marek,
> > >
> > > As far as I know, there shouldn't have any more issues you reported that
> > > are still left unfixed after this series. Please correct me if I'm wrong or
> > > if you find new issues.
> >
> > While this fixes the core support, there may still be driver fixes left
> > that were not developed in time for the v5.12-rc1 merge window.
> > Personally, I'm aware of "soc: renesas: rmobile-sysc: Mark fwnode
> > when PM domain is added", which I have queued for v5.13[1].
> > There may be other fixes for other platforms.
>
> Right, I intended this series for 5.13. Is that what you are trying to say too?
OK, v5.13 is fine for me.
It wasn't clear to me if you intended (the last patch of) this series to
be merged for v5.12-rcX or v5.13.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
On Wed, Mar 3, 2021 at 2:21 AM Michael Walle <[email protected]> wrote:
>
> Am 2021-03-03 10:28, schrieb Saravana Kannan:
> > On Wed, Mar 3, 2021 at 12:59 AM Michael Walle <[email protected]> wrote:
> >>
> >> Am 2021-03-02 23:47, schrieb Saravana Kannan:
> >> > On Tue, Mar 2, 2021 at 2:42 PM Saravana Kannan <[email protected]>
> >> > wrote:
> >> >>
> >> >> On Tue, Mar 2, 2021 at 2:24 PM Michael Walle <[email protected]> wrote:
> >> >> >
> >> >> > Am 2021-03-02 22:11, schrieb Saravana Kannan:
> >> >> > > I think Patch 1 should fix [4] without [5]. Can you test the series
> >> >> > > please?
> >> >> >
> >> >> > Mh, I'm on latest linux-next (next-20210302) and I've applied patch 3/3
> >> >> > and
> >> >> > reverted commit 7007b745a508 ("PCI: layerscape: Convert to
> >> >> > builtin_platform_driver()"). I'd assumed that PCIe shouldn't be working,
> >> >> > right? But it is. Did I miss something?
> >> >>
> >> >> You need to revert [5].
> >> >
> >> > My bad. You did revert it. Ah... I wonder if it was due to
> >> > fw_devlink.strict that I added. To break PCI again, also set
> >> > fw_devlink.strict=1 in the kernel command line.
> >>
> >> Indeed, adding fw_devlink.strict=1 will break PCI again. But if
> >> I then apply 1/3 and 2/3 again, PCI is still broken. Just to be clear:
> >> I'm keeping the fw_devlink.strict=1 parameter.
> >
> > Thanks for your testing! I assume you are also setting fw_devlink=on?
>
> I've applied patch 3/3 and added nothing to the commandline, so yes.
>
> > Hmmm... ok. In the working case, does your PCI probe before IOMMU? If
> > yes, then your results make sense.
>
> Yes that was the conclusion last time. That the probe is deferred and
> the __init section is already discarded when there might a second
> try of the probe.
Long response below, but the TL;DR is:
The real fix for your case was the implementation of fw_devlink.strict
and NOT Patch 1 of this series. So, sorry for wasting your test
effort.
During the earlier debugging (for take I), this is what I thought:
With fw_devlink=permissive, your boot sequence was (Case 1):
1. IOMMU probe
2. PCI builtin_platform_driver_probe() attempt
- Driver core sets up PCI with IOMMU
- PCI probe succeeds.
- PCI works with IOMMU. <---- Remember this point.
And with fw_devlink=on, I thought the IOMMU probe order was
unnecessarily changed and caused this (Case 2):
1. IOMMU probe reordered for some reason to be attempted before its
suppliers. Gets deferred.
2. PCI probe attempt
- fw_devlink + device links defers the probe because IOMMU isn't ready.
- builtin_platform_driver_probe() replaces drv->probe with
platform_probe_fail()
3. IOMMU deferred probe succeeds eventually.
4. PCI deferred probe is attempted
- platform_probe_fail() which is a stub just returns -ENXIO
And if this was the case, patch 1 in this series would have fixed it
by removing unnecessary reordering of probes.
But what was really happening was (after I went through your logs
again and looked at the code):
With fw_devlink=permissive, your boot sequence was really (Case 3):
1. PCI builtin_platform_driver_probe() attempt
- Driver core does NOT set up PCI with IOMMU
- PCI probe succeeds.
- PCI works without IOMMU. <---- Remember this point.
2. IOMMU probes
And with fw_devlink=on what was happening was (Case 4):
1. PCI builtin_platform_driver_probe() attempt
- fw_devlink + device links defers the probe because it thinks
IOMMU is mandatory and isn't ready.
- builtin_platform_driver_probe() replaces drv->probe with
platform_probe_fail()
2. IOMMU probes.
3. PCI deferred probe is attempted
- platform_probe_fail() which is a stub just returns -ENXIO
4. PCI is broken now.
In your case IOMMU is not mandatory and PCI works without IOMMU even
when fw_devlink=off/permissive. So the real fix for your case is the
addition of fw_devlink.strict and defaulting it to 0. Because of my
misunderstanding of your case, I didn't realize I already fixed your
case and I thought Patch 1 in this series would fix your case.
Patch 1 in this series is still important for other reasons, just not for you.
> So I guess, Patch 1/3 and Patch 2/3 doesn't fix that and the drivers
> still need to be converted to builtin_platform_driver(), right?
So there is no real issue between fw_devlink=on and
builtin_platform_driver_probe() anymore. At least none that I know of
or has been reported.
If you really want your PCI to work _with_ IOMMU, then
builtin_platform_driver_probe() is wrong even with fw_devlink=off. And
if you wanted PCI to work with IOMMU support and fw_devlink wasn't
available, you'll have to play initcall chicken with the IOMMU driver
or implement some IOMMU check + deferred probing in your PCI probe
function.
However, with fw_devlink=on, all you have to do is set fw_devlink=on
and fw_devlink.strict=1 and use builtin_platform_driver() and not have
to care about initcall orders or figure out how to defer when IOMMU
isn't ready yet.
-Saravana
On Tue, Mar 2, 2021 at 1:11 PM Saravana Kannan <[email protected]> wrote:
>
> There's no point in adding a device to the deferred probe list if we
> know for sure that it doesn't have a matching driver. So, check if a
> device can match with a driver before adding it to the deferred probe
> list.
>
> Signed-off-by: Saravana Kannan <[email protected]>
Rafael/Greg,
Any concerns with this specific patch? Do you see any bugs? I'm asking
because some of the other improvements I'm working on depend on this
flag. So I want to make sure this can land before I take my work in
progress too far.
-Saravana
> ---
> drivers/base/dd.c | 6 ++++++
> include/linux/device.h | 4 ++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/drivers/base/dd.c b/drivers/base/dd.c
> index 9179825ff646..f18963f42e21 100644
> --- a/drivers/base/dd.c
> +++ b/drivers/base/dd.c
> @@ -123,6 +123,9 @@ static DECLARE_WORK(deferred_probe_work, deferred_probe_work_func);
>
> void driver_deferred_probe_add(struct device *dev)
> {
> + if (!dev->can_match)
> + return;
> +
> mutex_lock(&deferred_probe_mutex);
> if (list_empty(&dev->p->deferred_probe)) {
> dev_dbg(dev, "Added to deferred list\n");
> @@ -726,6 +729,7 @@ static int driver_probe_device(struct device_driver *drv, struct device *dev)
> if (!device_is_registered(dev))
> return -ENODEV;
>
> + dev->can_match = true;
> pr_debug("bus: '%s': %s: matched device %s with driver %s\n",
> drv->bus->name, __func__, dev_name(dev), drv->name);
>
> @@ -829,6 +833,7 @@ static int __device_attach_driver(struct device_driver *drv, void *_data)
> return 0;
> } else if (ret == -EPROBE_DEFER) {
> dev_dbg(dev, "Device match requests probe deferral\n");
> + dev->can_match = true;
> driver_deferred_probe_add(dev);
> } else if (ret < 0) {
> dev_dbg(dev, "Bus failed to match device: %d\n", ret);
> @@ -1064,6 +1069,7 @@ static int __driver_attach(struct device *dev, void *data)
> return 0;
> } else if (ret == -EPROBE_DEFER) {
> dev_dbg(dev, "Device match requests probe deferral\n");
> + dev->can_match = true;
> driver_deferred_probe_add(dev);
> } else if (ret < 0) {
> dev_dbg(dev, "Bus failed to match device: %d\n", ret);
> diff --git a/include/linux/device.h b/include/linux/device.h
> index ba660731bd25..569932d282c0 100644
> --- a/include/linux/device.h
> +++ b/include/linux/device.h
> @@ -439,6 +439,9 @@ struct dev_links_info {
> * @state_synced: The hardware state of this device has been synced to match
> * the software state of this device by calling the driver/bus
> * sync_state() callback.
> + * @can_match: The device has matched with a driver at least once or it is in
> + * a bus (like AMBA) which can't check for matching drivers until
> + * other devices probe successfully.
> * @dma_coherent: this particular device is dma coherent, even if the
> * architecture supports non-coherent devices.
> * @dma_ops_bypass: If set to %true then the dma_ops are bypassed for the
> @@ -545,6 +548,7 @@ struct device {
> bool offline:1;
> bool of_node_reused:1;
> bool state_synced:1;
> + bool can_match:1;
> #if defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_DEVICE) || \
> defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_CPU) || \
> defined(CONFIG_ARCH_HAS_SYNC_DMA_FOR_CPU_ALL)
> --
> 2.30.1.766.gb4fecdf3b7-goog
>
On 02/03/2021 21:11, Saravana Kannan wrote:
> Device link status was not getting updated correctly when
> device_bind_driver() is called on a device. This causes a warning[1].
> Fix this by updating device links that can be updated and dropping
> device links that can't be updated to a sensible state.
>
> [1] - https://lore.kernel.org/lkml/[email protected]/
> Signed-off-by: Saravana Kannan <[email protected]>
> ---
> drivers/base/base.h | 1 +
> drivers/base/core.c | 35 +++++++++++++++++++++++++++++++++++
> drivers/base/dd.c | 4 +++-
> 3 files changed, 39 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/base/base.h b/drivers/base/base.h
> index 52b3d7b75c27..1b44ed588f66 100644
> --- a/drivers/base/base.h
> +++ b/drivers/base/base.h
> @@ -185,6 +185,7 @@ extern int device_links_read_lock(void);
> extern void device_links_read_unlock(int idx);
> extern int device_links_read_lock_held(void);
> extern int device_links_check_suppliers(struct device *dev);
> +extern void device_links_force_bind(struct device *dev);
> extern void device_links_driver_bound(struct device *dev);
> extern void device_links_driver_cleanup(struct device *dev);
> extern void device_links_no_driver(struct device *dev);
> diff --git a/drivers/base/core.c b/drivers/base/core.c
> index f29839382f81..45c75cc96fdc 100644
> --- a/drivers/base/core.c
> +++ b/drivers/base/core.c
> @@ -1153,6 +1153,41 @@ static ssize_t waiting_for_supplier_show(struct device *dev,
> }
> static DEVICE_ATTR_RO(waiting_for_supplier);
>
> +/**
> + * device_links_force_bind - Prepares device to be force bound
> + * @dev: Consumer device.
> + *
> + * device_bind_driver() force binds a device to a driver without calling any
> + * driver probe functions. So the consumer really isn't going to wait for any
> + * supplier before it's bound to the driver. We still want the device link
> + * states to be sensible when this happens.
> + *
> + * In preparation for device_bind_driver(), this function goes through each
> + * supplier device links and checks if the supplier is bound. If it is, then
> + * the device link status is set to CONSUMER_PROBE. Otherwise, the device link
> + * is dropped. Links without the DL_FLAG_MANAGED flag set are ignored.
> + */
> +void device_links_force_bind(struct device *dev)
> +{
> + struct device_link *link, *ln;
> +
> + device_links_write_lock();
> +
> + list_for_each_entry_safe(link, ln, &dev->links.suppliers, c_node) {
> + if (!(link->flags & DL_FLAG_MANAGED))
> + continue;
> +
> + if (link->status != DL_STATE_AVAILABLE) {
> + device_link_drop_managed(link);
> + continue;
> + }
> + WRITE_ONCE(link->status, DL_STATE_CONSUMER_PROBE);
> + }
> + dev->links.status = DL_DEV_PROBING;
> +
> + device_links_write_unlock();
> +}
> +
> /**
> * device_links_driver_bound - Update device links after probing its driver.
> * @dev: Device to update the links for.
> diff --git a/drivers/base/dd.c b/drivers/base/dd.c
> index f18963f42e21..eb201c6d5a6a 100644
> --- a/drivers/base/dd.c
> +++ b/drivers/base/dd.c
> @@ -460,8 +460,10 @@ int device_bind_driver(struct device *dev)
> int ret;
>
> ret = driver_sysfs_add(dev);
> - if (!ret)
> + if (!ret) {
> + device_links_force_bind(dev);
> driver_bound(dev);
> + }
> else if (dev->bus)
> blocking_notifier_call_chain(&dev->bus->p->bus_notifier,
> BUS_NOTIFY_DRIVER_NOT_BOUND, dev);
>
Thanks, this fixes the problem I had observed.
Tested-by: Jon Hunter <[email protected]>
Cheers!
Jon
--
nvpublic
On Tue, Mar 09, 2021 at 03:26:05PM -0800, Saravana Kannan wrote:
> On Tue, Mar 2, 2021 at 1:11 PM Saravana Kannan <[email protected]> wrote:
> >
> > There's no point in adding a device to the deferred probe list if we
> > know for sure that it doesn't have a matching driver. So, check if a
> > device can match with a driver before adding it to the deferred probe
> > list.
> >
> > Signed-off-by: Saravana Kannan <[email protected]>
>
> Rafael/Greg,
>
> Any concerns with this specific patch? Do you see any bugs? I'm asking
> because some of the other improvements I'm working on depend on this
> flag. So I want to make sure this can land before I take my work in
> progress too far.
Looks fine, now queuing up, thanks.
greg k-h
On Thu, Mar 25, 2021 at 2:19 PM Stephen Boyd <[email protected]> wrote:
>
> Quoting Saravana Kannan (2021-03-02 13:11:32)
> > This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
> >
> > Since all reported issues due to fw_devlink=on should be addressed by
> > this series, revert the revert. fw_devlink=on Take II.
> >
> > Signed-off-by: Saravana Kannan <[email protected]>
> > ---
>
> This seems to break the display on lazor (see
> arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi) on linux next today
> (next-20210325). I tried booting with fw_devlink=permissive on the
> commandline and the display came up again. Looking at the drivers that
> are in the deferred state there are three:
>
> localhost ~ # cat /sys/kernel/debug/devices_deferred
> ae94000.dsi
> panel
> 2-002d
>
> and the panel has these suppliers:
>
> localhost ~ # ls /sys/devices/platform/panel/
> driver_override power supplier:platform:pp3300-dx-edp-regulator
> modalias subsystem uevent
> of_node supplier:i2c:2-002d waiting_for_supplier
>
> Is there some sort of circular dependency going on that is preventing
> either driver from probing? My understanding is 2-002d is the dsi bridge
> (compatible is ti,sn65dsi86) and that is waiting for the panel to come
> up, and the panel does a circular dependency where it requests the hpd
> gpio from the dsi bridge at probe but then ignores it and tries to get
> the hpd gpio later when powering on the panel. If it didn't do this it
> would probe defer forever because the bridge supplies the hpd gpio to
> the panel and the panel provides the panel to the bridge driver.
I had a side chat with Stephen. The problem is due to a cycle of
dependency between panel and bridge (supplier:i2c:2-002d). panel needs
GPIO from bridge, and bridge has panel as a remote-endpoint. But
fw_delink isn't able to break the cycle because it doesn't parse
"remote-endpoint" yet. So for now, only permissive will work for this
case.
I'll look into adding remote-endpoint support. But that's a bit more
complicated.
-Saravana
>
> > drivers/base/core.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/base/core.c b/drivers/base/core.c
> > index 45c75cc96fdc..de518178ac36 100644
> > --- a/drivers/base/core.c
> > +++ b/drivers/base/core.c
> > @@ -1538,7 +1538,7 @@ static void device_links_purge(struct device *dev)
> > #define FW_DEVLINK_FLAGS_RPM (FW_DEVLINK_FLAGS_ON | \
> > DL_FLAG_PM_RUNTIME)
> >
> > -static u32 fw_devlink_flags = FW_DEVLINK_FLAGS_PERMISSIVE;
> > +static u32 fw_devlink_flags = FW_DEVLINK_FLAGS_ON;
> > static int __init fw_devlink_setup(char *arg)
> > {
> > if (!arg)
Hi Saravana,
Adding Sudeep and Christian, Al and Jim.
On 3/2/21 1:11 PM, Saravana Kannan wrote:
> This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
>
> Since all reported issues due to fw_devlink=on should be addressed by
> this series, revert the revert. fw_devlink=on Take II.
>
> Signed-off-by: Saravana Kannan <[email protected]>
This change breaks booting on SCMI-based platforms such as ARCH_BRCMSTB.
If I revert this change or boot with fw_devlink=permissive, then our
systems boot again. From a quick look, the SCMI clock provider was never
probed which means that our UART driver never got a chance to get its
clock and we have no console -> timeout.
Al, AFAICT you had started to analyze this before in the context of
SCMI, do you mind sharing what you had found?
Saravana, is there any debugging output that we can help provide?
Thank you!
--
Florian
On Mon, Apr 26, 2021 at 1:51 PM Florian Fainelli <[email protected]> wrote:
>
> Hi Saravana,
>
> Adding Sudeep and Christian, Al and Jim.
>
> On 3/2/21 1:11 PM, Saravana Kannan wrote:
> > This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
> >
> > Since all reported issues due to fw_devlink=on should be addressed by
> > this series, revert the revert. fw_devlink=on Take II.
> >
> > Signed-off-by: Saravana Kannan <[email protected]>
>
> This change breaks booting on SCMI-based platforms such as ARCH_BRCMSTB.
> If I revert this change or boot with fw_devlink=permissive, then our
> systems boot again. From a quick look, the SCMI clock provider was never
> probed which means that our UART driver never got a chance to get its
> clock and we have no console -> timeout.
We explicitly landed changes to handle this condition. So we'll see if
this is what is happening.
> Al, AFAICT you had started to analyze this before in the context of
> SCMI, do you mind sharing what you had found?
>
> Saravana, is there any debugging output that we can help provide?
Thanks for the report. Couple of things that can help:
1. Example DTS file (the final board file so that I can get the full DT view).
2. Point out the UART device node and the SCMI device node that you
suspect is causing the issue.
3. Boot logs with dev_dbg changed to dev_info in device_link_add() and
device_links_check_suppliers()
Thanks,
Saravana
On 4/26/21 2:33 PM, Saravana Kannan wrote:
> On Mon, Apr 26, 2021 at 1:51 PM Florian Fainelli <[email protected]> wrote:
>>
>> Hi Saravana,
>>
>> Adding Sudeep and Christian, Al and Jim.
>>
>> On 3/2/21 1:11 PM, Saravana Kannan wrote:
>>> This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
>>>
>>> Since all reported issues due to fw_devlink=on should be addressed by
>>> this series, revert the revert. fw_devlink=on Take II.
>>>
>>> Signed-off-by: Saravana Kannan <[email protected]>
>>
>> This change breaks booting on SCMI-based platforms such as ARCH_BRCMSTB.
>> If I revert this change or boot with fw_devlink=permissive, then our
>> systems boot again. From a quick look, the SCMI clock provider was never
>> probed which means that our UART driver never got a chance to get its
>> clock and we have no console -> timeout.
>
> We explicitly landed changes to handle this condition. So we'll see if
> this is what is happening.
>
>> Al, AFAICT you had started to analyze this before in the context of
>> SCMI, do you mind sharing what you had found?
>>
>> Saravana, is there any debugging output that we can help provide?
>
> Thanks for the report. Couple of things that can help:
> 1. Example DTS file (the final board file so that I can get the full DT view).
Attached BCX972160DV.dts which is one such system.
> 2. Point out the UART device node and the SCMI device node that you
> suspect is causing the issue.
The SCMI provider node is brcm_scmi@0 and its sub-node protocol@14 is
the clock provider. The UART node is /rdb/serial@840c000.
> 3. Boot logs with dev_dbg changed to dev_info in device_link_add() and
> device_links_check_suppliers()
OK, I did not want to modify the kernel so I just enabled dynamic debug
prints from the command line, if this is too verbose let me know.
Thank you for your prompt response.
--
Florian
Hi Florian,
On Mon, Apr 26, 2021 at 11:48 PM Florian Fainelli <[email protected]> wrote:
> On 4/26/21 2:33 PM, Saravana Kannan wrote:
> > On Mon, Apr 26, 2021 at 1:51 PM Florian Fainelli <[email protected]> wrote:
> >> On 3/2/21 1:11 PM, Saravana Kannan wrote:
> >>> This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
> >>>
> >>> Since all reported issues due to fw_devlink=on should be addressed by
> >>> this series, revert the revert. fw_devlink=on Take II.
> >>>
> >>> Signed-off-by: Saravana Kannan <[email protected]>
> >>
> >> This change breaks booting on SCMI-based platforms such as ARCH_BRCMSTB.
> >> If I revert this change or boot with fw_devlink=permissive, then our
> >> systems boot again. From a quick look, the SCMI clock provider was never
> >> probed which means that our UART driver never got a chance to get its
> >> clock and we have no console -> timeout.
> >
> > We explicitly landed changes to handle this condition. So we'll see if
> > this is what is happening.
> >
> >> Al, AFAICT you had started to analyze this before in the context of
> >> SCMI, do you mind sharing what you had found?
> >>
> >> Saravana, is there any debugging output that we can help provide?
> >
> > Thanks for the report. Couple of things that can help:
> > 1. Example DTS file (the final board file so that I can get the full DT view).
>
> Attached BCX972160DV.dts which is one such system.
>
> > 2. Point out the UART device node and the SCMI device node that you
> > suspect is causing the issue.
>
> The SCMI provider node is brcm_scmi@0 and its sub-node protocol@14 is
> the clock provider. The UART node is /rdb/serial@840c000.
I guess dependencies on SCMI sub-nodes are not handled correctly?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Hi Florian,
On Mon, Apr 26, 2021 at 02:47:52PM -0700, Florian Fainelli wrote:
> On 4/26/21 2:33 PM, Saravana Kannan wrote:
> > On Mon, Apr 26, 2021 at 1:51 PM Florian Fainelli <[email protected]> wrote:
> >>
> >> Hi Saravana,
> >>
> >> Adding Sudeep and Christian, Al and Jim.
> >>
> >> On 3/2/21 1:11 PM, Saravana Kannan wrote:
> >>> This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
> >>>
> >>> Since all reported issues due to fw_devlink=on should be addressed by
> >>> this series, revert the revert. fw_devlink=on Take II.
> >>>
> >>> Signed-off-by: Saravana Kannan <[email protected]>
> >>
> >> This change breaks booting on SCMI-based platforms such as ARCH_BRCMSTB.
> >> If I revert this change or boot with fw_devlink=permissive, then our
> >> systems boot again. From a quick look, the SCMI clock provider was never
> >> probed which means that our UART driver never got a chance to get its
> >> clock and we have no console -> timeout.
> >
> > We explicitly landed changes to handle this condition. So we'll see if
> > this is what is happening.
> >
> >> Al, AFAICT you had started to analyze this before in the context of
> >> SCMI, do you mind sharing what you had found?
> >>
> >> Saravana, is there any debugging output that we can help provide?
> >
> > Thanks for the report. Couple of things that can help:
> > 1. Example DTS file (the final board file so that I can get the full DT view).
>
> Attached BCX972160DV.dts which is one such system.
>
> > 2. Point out the UART device node and the SCMI device node that you
> > suspect is causing the issue.
>
> The SCMI provider node is brcm_scmi@0 and its sub-node protocol@14 is
> the clock provider. The UART node is /rdb/serial@840c000.
>
> > 3. Boot logs with dev_dbg changed to dev_info in device_link_add() and
> > device_links_check_suppliers()
>
> OK, I did not want to modify the kernel so I just enabled dynamic debug
> prints from the command line, if this is too verbose let me know.
>
> Thank you for your prompt response.
> --
I may say something obvious but, looking at the kernel log, it seems to me
that it is the SCMI subsystem as a whole that is never initialized and goes
through a lot of deferrals because its only transport, the brcmstb-mbox, is
in turn never found to be ready, then as a consequence your serial, relying
on scmi-clock, is never initialized too.
In JUNO in fact I can see the same pattern of deferrals during boot (if I
enable dev_info in dev_link) until the ARM MHU mailboxes are found and
initialized and then firmware:scmi stops being deferred and it is
initialized with all the stack and its consumers.
I cannot see the brcm_scmi_mailbox@0 being found here instead.
Thanks
Cristian
> Florian
> [ 0.000000] Linux version 5.12.0-gef1244124349 (fainelli@fainelli-desktop) (arm-linux-gcc (GCC) 8.3.0, GNU ld (GNU Binutils) 2.32) #32 S
> MP Mon Apr 26 14:29:52 PDT 2021
> [ 0.000000] CPU: ARMv7 Processor [420f1000] revision 0 (ARMv7), cr=30c5383d
> [ 0.000000] CPU: div instructions available: patching division code
> [ 0.000000] CPU: PIPT / VIPT nonaliasing data cache, VIPT aliasing instruction cache
> [ 0.000000] OF: fdt: Machine model: BCX972160DV
> [ 0.000000] earlycon: ns16550a0 at MMIO32 0x000000000840c000 (options '115200')
> [ 0.000000] printk: bootconsole [ns16550a0] enabled
> [ 0.000000] printk: debug: skip boot console de-registration.
> [ 0.000000] Memory policy: Data cache writealloc
> [ 0.000000] cma: Reserved 16 MiB at 0x000000006f000000
> [ 0.000000] Zone ranges:
> [ 0.000000] DMA [mem 0x0000000040000000-0x000000006fffffff]
> [ 0.000000] Normal empty
> [ 0.000000] HighMem [mem 0x0000000070000000-0x000000013fffffff]
> [ 0.000000] Movable zone start for each node
> [ 0.000000] Early memory node ranges
> [ 0.000000] node 0: [mem 0x0000000040000000-0x00000000fdffefff]
> [ 0.000000] node 0: [mem 0x00000000fdfff000-0x00000000ffffffff]
> [ 0.000000] node 0: [mem 0x0000000100000000-0x000000013fffffff]
> [ 0.000000] Initmem setup node 0 [mem 0x0000000040000000-0x000000013fffffff]
> [ 0.000000] On node 0 totalpages: 1048576
> [ 0.000000] DMA zone: 1920 pages used for memmap
> [ 0.000000] DMA zone: 0 pages reserved
> [ 0.000000] DMA zone: 196608 pages, LIFO batch:63
> [ 0.000000] HighMem zone: 851968 pages, LIFO batch:63
> [ 0.000000] psci: probing for conduit method from DT.
> [ 0.000000] psci: PSCIv1.1 detected in firmware.
> [ 0.000000] psci: Using standard PSCI v0.2 function IDs
> [ 0.000000] psci: Trusted OS resident on physical CPU 0x0
> [ 0.000000] psci: SMC Calling Convention v1.1
> [ 0.000000] percpu: Embedded 20 pages/cpu s49804 r8192 d23924 u81920
> [ 0.000000] pcpu-alloc: s49804 r8192 d23924 u81920 alloc=20*4096
> [ 0.000000] pcpu-alloc: [0] 0 [0] 1 [0] 2 [0] 3
> [ 0.000000] Built 1 zonelists, mobility grouping on. Total pages: 1046656
> [ 0.000000] Kernel command line: fw_devlink=on dyndbg="file drivers/base/core.c +p" debug earlycon keep_bootcon
> [ 0.000000] Dentry cache hash table entries: 131072 (order: 7, 524288 bytes, linear)
> [ 0.000000] Inode-cache hash table entries: 65536 (order: 6, 262144 bytes, linear)
> [ 0.000000] mem auto-init: stack:off, heap alloc:off, heap free:off
> [ 0.000000] software IO TLB: mapped [mem 0x0000000068400000-0x000000006c400000] (64MB)
> [ 0.000000] Memory: 3986968K/4194304K available (12288K kernel code, 1210K rwdata, 2648K rodata, 14336K init, 246K bss, 190952K reserved
> , 16384K cma-reserved, 3358716K highmem)
> [ 0.000000] SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=4, Nodes=1
> [ 0.000000] ftrace: allocating 37847 entries in 74 pages
> [ 0.000000] ftrace: allocated 74 pages with 3 groups
> [ 0.000000] rcu: Hierarchical RCU implementation.
> [ 0.000000] rcu: RCU event tracing is enabled.
> [ 0.000000] Rude variant of Tasks RCU enabled.
> [ 0.000000] rcu: RCU calculated value of scheduler-enlistment delay is 100 jiffies.
> [ 0.000000] NR_IRQS: 16, nr_irqs: 16, preallocated irqs: 16
> [ 0.000000] GIC: Using split EOI/Deactivate mode
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@8400000, parent irq: 24)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@84a1000, parent irq: 25)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@84a1800, parent irq: 26)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@8410640, parent irq: 27)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@841a880, parent irq: 28)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@840a400, parent irq: 29)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@841a380, parent irq: 30)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@840a440, parent irq: 31)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@841a3c0, parent irq: 32)
> [ 0.000000] irq_brcmstb_l2: registered L2 intc (/rdb/interrupt-controller@8419000, parent irq: 33)
> [ 0.000000] random: get_random_bytes called from start_kernel+0x42c/0x60c with crng_init=0
> [ 0.000000] arch_timer: cp15 timer(s) running at 27.00MHz (phys).
> [ 0.000000] clocksource: arch_sys_counter: mask: 0xffffffffffffff max_cycles: 0x63a1e71a3, max_idle_ns: 440795203123 ns
> [ 0.000001] sched_clock: 56 bits at 27MHz, resolution 37ns, wraps every 4398046511093ns
> [ 0.008134] Switching to timer-based delay loop, resolution 37ns
> [ 0.014453] Console: colour dummy device 80x30
> [ 0.018988] printk: console [tty0] enabled
> [ 0.023186] Calibrating delay loop (skipped), value calculated using timer frequency.. 54.00 BogoMIPS (lpj=27000)
> [ 0.033654] pid_max: default: 32768 minimum: 301
> [ 0.038432] Mount-cache hash table entries: 2048 (order: 1, 8192 bytes, linear)
> [ 0.045855] Mountpoint-cache hash table entries: 2048 (order: 1, 8192 bytes, linear)
> [ 0.054210] CPU: Testing write buffer coherency: ok
> [ 0.059444] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000
> [ 0.065526] Setting up static identity map for 0x40200000 - 0x40200060
> [ 0.072250] rcu: Hierarchical SRCU implementation.
> [ 0.078389] brcmstb: biuctrl: MCP: Enabling write pairing
> [ 0.084034] smp: Bringing up secondary CPUs ...
> [ 0.089532] CPU1: thread -1, cpu 1, socket 0, mpidr 80000001
> [ 0.090446] CPU2: thread -1, cpu 2, socket 0, mpidr 80000002
> [ 0.091316] CPU3: thread -1, cpu 3, socket 0, mpidr 80000003
> [ 0.091378] smp: Brought up 1 node, 4 CPUs
> [ 0.112826] SMP: Total of 4 processors activated (216.00 BogoMIPS).
> [ 0.119209] CPU: All CPU(s) started in HYP mode.
> [ 0.123902] CPU: Virtualization extensions available.
> [ 0.130052] devtmpfs: initialized
> [ 0.135401] device: 'platform': device_add
> [ 0.139664] device: 'cpu': device_add
> [ 0.143419] device: 'container': device_add
> [ 0.148274] VFP support v0.3: implementor 42 architecture 3 part 10 variant 0 rev 0
> [ 0.156107] device: 'workqueue': device_add
> [ 0.160469] clocksource: jiffies: mask: 0xffffffff max_cycles: 0xffffffff, max_idle_ns: 1911260446275000 ns
> [ 0.170396] futex hash table entries: 1024 (order: 4, 65536 bytes, linear)
> [ 0.178019] pinctrl core: initialized pinctrl subsystem
> [ 0.183502] device: 'reg-dummy': device_add
> [ 0.187874] device: 'regulator.0': device_add
> [ 0.192941] NET: Registered protocol family 16
> [ 0.198853] DMA: preallocated 256 KiB pool for atomic coherent allocations
> [ 0.206556] device: 'vtcon0': device_add
> [ 0.210923] thermal_sys: Registered thermal governor 'step_wise'
> [ 0.210942] thermal_sys: Registered thermal governor 'user_space'
> [ 0.217182] device: 'thermal_zone0': device_add
> [ 0.228089] device: 'thermal_zone1': device_add
> [ 0.232819] cpuidle: using governor menu
> [ 0.236916] No ATAGs?
> [ 0.239331] hw-breakpoint: found 5 (+1 reserved) breakpoint and 4 watchpoint registers.
> [ 0.247505] hw-breakpoint: maximum watchpoint size is 8 bytes.
> [ 0.253693] /rdb/syscon@84a2400: Broadcom Brahma-B15 readahead cache
> [ 0.260356] device: 'soc0': device_add
> [ 0.264207] Serial: AMBA PL011 UART driver
> [ 0.268567] device: 'psci': device_add
> [ 0.272585] device: 'timer': device_add
> [ 0.276708] device: 'brcm_scmi_mailbox@0': device_add
> [ 0.281972] device: 'brcm_scmi@0': device_add
> [ 0.286506] device: 'platform:brcm_scmi_mailbox@0--platform:brcm_scmi@0': device_add
> [ 0.294403] devices_kset: Moving brcm_scmi@0 to end of list
> [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
> [ 0.307400] device: 'rdb': device_add
> [ 0.311163] OF: /rdb/brcmstb-clks/cpu_mdiv_ch0@8161010: could not get #clock-cells for /rdb/brcmstb-clks/cpu_ndiv_int@8161018
> [ 0.324194] device: 'platform:brcm_scmi@0--platform:rdb': device_add
> [ 0.330675] platform rdb: Linked as a sync state only consumer to brcm_scmi@0
> [ 0.338229] device: '840c000.serial': device_add
> [ 0.342957] device: 'platform:brcm_scmi@0--platform:840c000.serial': device_add
> [ 0.350440] devices_kset: Moving 840c000.serial to end of list
> [ 0.356358] platform 840c000.serial: Linked as a consumer to brcm_scmi@0
> [ 0.363315] device: '840d000.serial': device_add
> [ 0.368047] device: 'platform:brcm_scmi@0--platform:840d000.serial': device_add
> [ 0.375512] devices_kset: Moving 840d000.serial to end of list
> [ 0.381446] platform 840d000.serial: Linked as a consumer to brcm_scmi@0
> [ 0.388373] device: '840e000.serial': device_add
> [ 0.393116] device: 'platform:brcm_scmi@0--platform:840e000.serial': device_add
> [ 0.400573] devices_kset: Moving 840e000.serial to end of list
> [ 0.406506] platform 840e000.serial: Linked as a consumer to brcm_scmi@0
> [ 0.413451] device: '8b2c800.rescal': device_add
> [ 0.418349] device: '8d0f200.usb-phy': device_add
> [ 0.423175] device: 'platform:brcm_scmi@0--platform:8d0f200.usb-phy': device_add
> [ 0.430729] devices_kset: Moving 8d0f200.usb-phy to end of list
> [ 0.436794] platform 8d0f200.usb-phy: Linked as a consumer to brcm_scmi@0
> [ 0.443804] device: '8d00000.xhci_v2': device_add
> [ 0.448650] device: 'platform:brcm_scmi@0--platform:8d00000.xhci_v2': device_add
> [ 0.456190] devices_kset: Moving 8d00000.xhci_v2 to end of list
> [ 0.462212] platform 8d00000.xhci_v2: Linked as a consumer to brcm_scmi@0
> [ 0.469118] device: 'platform:8d0f200.usb-phy--platform:8d00000.xhci_v2': device_add
> [ 0.477025] devices_kset: Moving 8d00000.xhci_v2 to end of list
> [ 0.483053] platform 8d00000.xhci_v2: Linked as a consumer to 8d0f200.usb-phy
> [ 0.490510] device: '8f00000.ethernet': device_add
> [ 0.495435] device: 'platform:brcm_scmi@0--platform:8f00000.ethernet': device_add
> [ 0.503067] devices_kset: Moving 8f00000.ethernet to end of list
> [ 0.509174] platform 8f00000.ethernet: Linked as a consumer to brcm_scmi@0
> [ 0.516310] device: '840a500.gpio': device_add
> [ 0.520962] device: '841a400.gpio': device_add
> [ 0.525641] device: '8b0a000.sata': device_add
> [ 0.530223] device: 'platform:brcm_scmi@0--platform:8b0a000.sata': device_add
> [ 0.537529] devices_kset: Moving 8b0a000.sata to end of list
> [ 0.543275] platform 8b0a000.sata: Linked as a consumer to brcm_scmi@0
> [ 0.549992] device: '8b08100.sata_phy': device_add
> [ 0.554899] device: 'platform:8b08100.sata_phy--platform:8b0a000.sata': device_add
> [ 0.562609] platform 8b0a000.sata: Linked as a sync state only consumer to 8b08100.sata_phy
> [ 0.571259] device: '84b0000.sdhci': device_add
> [ 0.575894] device: 'platform:brcm_scmi@0--platform:84b0000.sdhci': device_add
> [ 0.583294] devices_kset: Moving 84b0000.sdhci to end of list
> [ 0.589133] platform 84b0000.sdhci: Linked as a consumer to brcm_scmi@0
> [ 0.596079] device: '83fc000.gisb-arb': device_add
> [ 0.601132] device: '841a840.waketimer': device_add
> [ 0.606194] device: 'a581500.thermal': device_add
> [ 0.611102] device: '8410000.aon-ctrl': device_add
> [ 0.616025] device: 'rdb:memory_controllers': device_add
> [ 0.621502] device: 'rdb:memory_controllers:memc@0': device_add
> [ 0.627652] device: '9900000.memc-gen': device_add
> [ 0.632658] device: '9903000.memc-arb': device_add
> [ 0.637649] device: '9920000.ddr-phy': device_add
> [ 0.642559] device: '9908000.shimphy': device_add
> [ 0.647470] device: '9902000.memc-ddr': device_add
> [ 0.652578] device: '84b3400.spi': device_add
> [ 0.657422] device: '84b2120.qspi': device_add
> [ 0.662032] device: 'platform:brcm_scmi@0--platform:84b2120.qspi': device_add
> [ 0.669346] devices_kset: Moving 84b2120.qspi to end of list
> [ 0.675099] platform 84b2120.qspi: Linked as a consumer to brcm_scmi@0
> [ 0.681851] device: '8404000.syscon': device_add
> [ 0.686640] device: '8404534.syscon': device_add
> [ 0.691437] device: '8404100.syscon': device_add
> [ 0.696226] device: '8404124.syscon': device_add
> [ 0.700992] device: '8410700.syscon': device_add
> [ 0.705773] device: '841070c.syscon': device_add
> [ 0.710565] device: '9903004.syscon': device_add
> [ 0.715348] device: '8404084.syscon': device_add
> [ 0.720141] device: '84040a4.syscon': device_add
> [ 0.724924] device: '84b0344.syscon': device_add
> [ 0.729715] device: '84b1344.syscon': device_add
> [ 0.734500] device: '84b1500.syscon': device_add
> [ 0.739283] device: '84b037c.syscon': device_add
> [ 0.744090] device: '84b137c.syscon': device_add
> [ 0.748875] device: '84040ac.syscon': device_add
> [ 0.753711] device: '840a628.watchdog': device_add
> [ 0.758676] device: '83b0000.bsp': device_add
> [ 0.763261] device: '8418000.mspi': device_add
> [ 0.767901] device: '8408000.pwm': device_add
> [ 0.772519] device: '9700000.gpu': device_add
> [ 0.777022] device: 'rdb:gpu-mmu@9705000': device_add
> [ 0.782254] device: '9910000.syscon': device_add
> [ 0.787033] device: '9909000.syscon': device_add
> [ 0.791869] device: '9932000.dpfe-cpu': device_add
> [ 0.796819] device: '8402000.rng': device_add
> [ 0.801358] device: '8404318.reset-controller': device_add
> [ 0.807013] device: '84a2400.syscon': device_add
> [ 0.811815] device: '8452000.syscon': device_add
> [ 0.816581] device: 'reboot': device_add
> [ 0.820800] device: 'pmu': device_add
> [ 0.824740] device: '8b20000.pcie': device_add
> [ 0.829402] device: 'platform:brcm_scmi@0--platform:8b20000.pcie': device_add
> [ 0.836711] devices_kset: Moving 8b20000.pcie to end of list
> [ 0.842472] platform 8b20000.pcie: Linked as a consumer to brcm_scmi@0
> [ 0.849151] device: 'smpboot': device_add
> [ 0.853324] device: '100000.sram': device_add
> [ 0.857885] device: 'cpu0': device_add
> [ 0.861770] device: 'cpu1': device_add
> [ 0.865667] device: 'cpu2': device_add
> [ 0.869530] device: 'cpu3': device_add
> [ 0.883847] device: 'writeback': device_add
> [ 0.889865] SCSI subsystem initialized
> [ 0.893864] libata version 3.00 loaded.
> [ 0.897999] usbcore: registered new interface driver usbfs
> [ 0.903747] usbcore: registered new interface driver hub
> [ 0.909224] usbcore: registered new device driver usb
> [ 0.914426] mc: Linux media interface: v0.10
> [ 0.918809] videodev: Linux video capture interface: v2.00
> [ 0.924438] pps_core: LinuxPPS API ver. 1 registered
> [ 0.929498] pps_core: Software ver. 5.3.6 - Copyright 2005-2007 Rodolfo Giometti <[email protected]>
> [ 0.938794] PTP clock support registered
> [ 0.942998] platform brcm_scmi@0: probe deferral - supplier brcm_scmi_mailbox@0 not ready
> [ 0.951637] Advanced Linux Sound Architecture Driver Initialized.
> [ 0.957934] device: 'lo': device_add
> [ 0.961904] device: 'rfkill': device_add
> [ 0.966762] clocksource: Switched to clocksource arch_sys_counter
> [ 1.003627] device: 'mem': device_add
> [ 1.007505] device: 'null': device_add
> [ 1.011388] device: 'port': device_add
> [ 1.015245] device: 'zero': device_add
> [ 1.019096] device: 'full': device_add
> [ 1.022942] device: 'random': device_add
> [ 1.026950] device: 'urandom': device_add
> [ 1.031079] device: 'kmsg': device_add
> [ 1.035020] device: 'tty': device_add
> [ 1.038831] device: 'console': device_add
> [ 1.042974] device: 'tty0': device_add
> [ 1.046862] device: 'vcs': device_add
> [ 1.050700] device: 'vcsu': device_add
> [ 1.054551] device: 'vcsa': device_add
> [ 1.058402] device: 'vcs1': device_add
> [ 1.062280] device: 'vcsu1': device_add
> [ 1.066224] device: 'vcsa1': device_add
> [ 1.070199] device: 'tty1': device_add
> [ 1.074081] device: 'tty2': device_add
> [ 1.077945] device: 'tty3': device_add
> [ 1.081798] device: 'tty4': device_add
> [ 1.085686] device: 'tty5': device_add
> [ 1.089517] device: 'tty6': device_add
> [ 1.093411] device: 'tty7': device_add
> [ 1.097295] device: 'tty8': device_add
> [ 1.101167] device: 'tty9': device_add
> [ 1.105025] device: 'tty10': device_add
> [ 1.109001] device: 'tty11': device_add
> [ 1.112966] device: 'tty12': device_add
> [ 1.116899] device: 'tty13': device_add
> [ 1.120850] device: 'tty14': device_add
> [ 1.124795] device: 'tty15': device_add
> [ 1.128739] device: 'tty16': device_add
> [ 1.132684] device: 'tty17': device_add
> [ 1.136623] device: 'tty18': device_add
> [ 1.140617] device: 'tty19': device_add
> [ 1.144553] device: 'tty20': device_add
> [ 1.148485] device: 'tty21': device_add
> [ 1.152405] device: 'tty22': device_add
> [ 1.156351] device: 'tty23': device_add
> [ 1.160287] device: 'tty24': device_add
> [ 1.164224] device: 'tty25': device_add
> [ 1.168171] device: 'tty26': device_add
> [ 1.172123] device: 'tty27': device_add
> [ 1.176078] device: 'tty28': device_add
> [ 1.180033] device: 'tty29': device_add
> [ 1.183996] device: 'tty30': device_add
> [ 1.187955] device: 'tty31': device_add
> [ 1.191913] device: 'tty32': device_add
> [ 1.195867] device: 'tty33': device_add
> [ 1.199802] device: 'tty34': device_add
> [ 1.203742] device: 'tty35': device_add
> [ 1.207702] device: 'tty36': device_add
> [ 1.211651] device: 'tty37': device_add
> [ 1.215628] device: 'tty38': device_add
> [ 1.219599] device: 'tty39': device_add
> [ 1.223560] device: 'tty40': device_add
> [ 1.227530] device: 'tty41': device_add
> [ 1.231491] device: 'tty42': device_add
> [ 1.235429] device: 'tty43': device_add
> [ 1.239398] device: 'tty44': device_add
> [ 1.243334] device: 'tty45': device_add
> [ 1.247261] device: 'tty46': device_add
> [ 1.251269] device: 'tty47': device_add
> [ 1.255217] device: 'tty48': device_add
> [ 1.259169] device: 'tty49': device_add
> [ 1.263136] device: 'tty50': device_add
> [ 1.267075] device: 'tty51': device_add
> [ 1.271072] device: 'tty52': device_add
> [ 1.275026] device: 'tty53': device_add
> [ 1.278965] device: 'tty54': device_add
> [ 1.282958] device: 'tty55': device_add
> [ 1.286905] device: 'tty56': device_add
> [ 1.290840] device: 'tty57': device_add
> [ 1.294797] device: 'tty58': device_add
> [ 1.298736] device: 'tty59': device_add
> [ 1.302709] device: 'tty60': device_add
> [ 1.306673] device: 'tty61': device_add
> [ 1.310628] device: 'tty62': device_add
> [ 1.314609] device: 'tty63': device_add
> [ 1.318700] NET: Registered protocol family 2
> [ 1.323375] tcp_listen_portaddr_hash hash table entries: 512 (order: 0, 6144 bytes, linear)
> [ 1.331890] TCP established hash table entries: 8192 (order: 3, 32768 bytes, linear)
> [ 1.339829] TCP bind hash table entries: 8192 (order: 4, 65536 bytes, linear)
> [ 1.347106] TCP: Hash tables configured (established 8192 bind 8192)
> [ 1.353670] UDP hash table entries: 512 (order: 2, 16384 bytes, linear)
> [ 1.360436] UDP-Lite hash table entries: 512 (order: 2, 16384 bytes, linear)
> [ 1.367702] NET: Registered protocol family 1
> [ 1.375093] RPC: Registered named UNIX socket transport module.
> [ 1.381137] RPC: Registered udp transport module.
> [ 1.385948] RPC: Registered tcp transport module.
> [ 1.390729] RPC: Registered tcp NFSv4.1 backchannel transport module.
> [ 1.397603] device: 'regulatory.0': device_add
> [ 1.402302] PCI: CLS 0 bytes, default 64
> [ 3.410785] device: 'clocksource': device_add
> [ 3.415283] device: 'clocksource0': device_add
> [ 3.420011] device: 'clockevents': device_add
> [ 3.424466] device: 'clockevent0': device_add
> [ 3.428918] device: 'clockevent1': device_add
> [ 3.433358] device: 'clockevent2': device_add
> [ 3.437820] device: 'clockevent3': device_add
> [ 3.442277] device: 'broadcast': device_add
> [ 3.446657] device: 'software': device_add
> [ 3.450861] device: 'tracepoint': device_add
> [ 3.455213] device: 'uprobe': device_add
> [ 3.459228] device: 'breakpoint': device_add
> [ 3.463589] Initialise system trusted keyrings
> [ 3.468304] workingset: timestamp_bits=30 max_order=20 bucket_order=0
> [ 3.477332] squashfs: version 4.0 (2009/01/31) Phillip Lougher
> [ 3.483696] NFS: Registering the id_resolver key type
> [ 3.488880] Key type id_resolver registered
> [ 3.493160] Key type id_legacy registered
> [ 3.497284] nfs4filelayout_init: NFSv4 File Layout Driver Registering...
> [ 3.504137] nfs4flexfilelayout_init: NFSv4 Flexfile Layout Driver Registering...
> [ 3.511656] jffs2: version 2.2. (NAND) ? 2001-2006 Red Hat, Inc.
> [ 3.518014] fuse: init (API version 7.33)
> [ 3.522124] device: 'fuse': device_add
> [ 3.526097] device: 'cuse': device_add
> [ 3.530132] Key type asymmetric registered
> [ 3.534299] Asymmetric key parser 'x509' registered
> [ 3.539303] bounce: pool size: 64 pages
> [ 3.543245] Block layer SCSI generic (bsg) driver version 0.4 loaded (major 247)
> [ 3.550780] io scheduler mq-deadline registered
> [ 3.555372] io scheduler kyber registered
> [ 3.559815] brcm-gisb-arb 83fc000.gisb-arb: registered irqs: 53, 54
> [ 3.566619] device: 'phy-8b08100.sata_phy.0': device_add
> [ 3.572114] brcm-sata-phy 8b08100.sata_phy: registered 1 port(s)
> [ 3.578762] device: 'gpiochip0': device_add
> [ 3.583122] device: 'gpiochip0': device_add
> [ 3.587461] device: 'gpiochip1': device_add
> [ 3.591811] device: 'gpiochip32': device_add
> [ 3.596239] device: 'wakeup0': device_add
> [ 3.600547] device: 'gpiochip2': device_add
> [ 3.604914] device: 'gpiochip64': device_add
> [ 3.609343] device: 'gpiochip3': device_add
> [ 3.613662] device: 'gpiochip96': device_add
> [ 3.618077] device: 'wakeup1': device_add
> [ 3.622741] platform 8b20000.pcie: probe deferral - supplier brcm_scmi@0 not ready
> [ 3.631146] brcmstb-pm: error mapping DDR PHY
> [ 3.635609] brcmstb-pm: probe of 8410000.aon-ctrl failed with error -22
> [ 3.642854] device: 'ptyp0': device_add
> [ 3.646907] device: 'ptyp1': device_add
> [ 3.650874] device: 'ptyp2': device_add
> [ 3.654826] device: 'ptyp3': device_add
> [ 3.658785] device: 'ptyp4': device_add
> [ 3.662719] device: 'ptyp5': device_add
> [ 3.666673] device: 'ptyp6': device_add
> [ 3.670626] device: 'ptyp7': device_add
> [ 3.674571] device: 'ptyp8': device_add
> [ 3.678508] device: 'ptyp9': device_add
> [ 3.682469] device: 'ptypa': device_add
> [ 3.686417] device: 'ptypb': device_add
> [ 3.690368] device: 'ptypc': device_add
> [ 3.694310] device: 'ptypd': device_add
> [ 3.698243] device: 'ptype': device_add
> [ 3.702181] device: 'ptypf': device_add
> [ 3.706121] device: 'ptyq0': device_add
> [ 3.710059] device: 'ptyq1': device_add
> [ 3.714037] device: 'ptyq2': device_add
> [ 3.717993] device: 'ptyq3': device_add
> [ 3.721935] device: 'ptyq4': device_add
> [ 3.725896] device: 'ptyq5': device_add
> [ 3.729849] device: 'ptyq6': device_add
> [ 3.733799] device: 'ptyq7': device_add
> [ 3.737773] device: 'ptyq8': device_add
> [ 3.741709] device: 'ptyq9': device_add
> [ 3.745700] device: 'ptyqa': device_add
> [ 3.749652] device: 'ptyqb': device_add
> [ 3.753596] device: 'ptyqc': device_add
> [ 3.757547] device: 'ptyqd': device_add
> [ 3.761493] device: 'ptyqe': device_add
> [ 3.765428] device: 'ptyqf': device_add
> [ 3.769388] device: 'ptyr0': device_add
> [ 3.773339] device: 'ptyr1': device_add
> [ 3.777295] device: 'ptyr2': device_add
> [ 3.781233] device: 'ptyr3': device_add
> [ 3.785182] device: 'ptyr4': device_add
> [ 3.789136] device: 'ptyr5': device_add
> [ 3.793066] device: 'ptyr6': device_add
> [ 3.796995] device: 'ptyr7': device_add
> [ 3.800928] device: 'ptyr8': device_add
> [ 3.804864] device: 'ptyr9': device_add
> [ 3.808812] device: 'ptyra': device_add
> [ 3.812777] device: 'ptyrb': device_add
> [ 3.816722] device: 'ptyrc': device_add
> [ 3.820706] device: 'ptyrd': device_add
> [ 3.824652] device: 'ptyre': device_add
> [ 3.828611] device: 'ptyrf': device_add
> [ 3.832535] device: 'ptys0': device_add
> [ 3.836481] device: 'ptys1': device_add
> [ 3.840415] device: 'ptys2': device_add
> [ 3.844333] device: 'ptys3': device_add
> [ 3.848259] device: 'ptys4': device_add
> [ 3.852204] device: 'ptys5': device_add
> [ 3.856163] device: 'ptys6': device_add
> [ 3.860135] device: 'ptys7': device_add
> [ 3.864072] device: 'ptys8': device_add
> [ 3.868010] device: 'ptys9': device_add
> [ 3.871952] device: 'ptysa': device_add
> [ 3.875902] device: 'ptysb': device_add
> [ 3.879847] device: 'ptysc': device_add
> [ 3.883786] device: 'ptysd': device_add
> [ 3.887694] device: 'ptyse': device_add
> [ 3.891662] device: 'ptysf': device_add
> [ 3.895597] device: 'ptyt0': device_add
> [ 3.899535] device: 'ptyt1': device_add
> [ 3.903460] device: 'ptyt2': device_add
> [ 3.907400] device: 'ptyt3': device_add
> [ 3.911346] device: 'ptyt4': device_add
> [ 3.915279] device: 'ptyt5': device_add
> [ 3.919234] device: 'ptyt6': device_add
> [ 3.923180] device: 'ptyt7': device_add
> [ 3.927147] device: 'ptyt8': device_add
> [ 3.931082] device: 'ptyt9': device_add
> [ 3.935026] device: 'ptyta': device_add
> [ 3.938995] device: 'ptytb': device_add
> [ 3.942943] device: 'ptytc': device_add
> [ 3.946901] device: 'ptytd': device_add
> [ 3.950845] device: 'ptyte': device_add
> [ 3.954804] device: 'ptytf': device_add
> [ 3.958735] device: 'ptyu0': device_add
> [ 3.962688] device: 'ptyu1': device_add
> [ 3.966649] device: 'ptyu2': device_add
> [ 3.970599] device: 'ptyu3': device_add
> [ 3.974555] device: 'ptyu4': device_add
> [ 3.978499] device: 'ptyu5': device_add
> [ 3.982443] device: 'ptyu6': device_add
> [ 3.986400] device: 'ptyu7': device_add
> [ 3.990349] device: 'ptyu8': device_add
> [ 3.994303] device: 'ptyu9': device_add
> [ 3.998233] device: 'ptyua': device_add
> [ 4.002192] device: 'ptyub': device_add
> [ 4.006142] device: 'ptyuc': device_add
> [ 4.010090] device: 'ptyud': device_add
> [ 4.014015] device: 'ptyue': device_add
> [ 4.017962] device: 'ptyuf': device_add
> [ 4.021906] device: 'ptyv0': device_add
> [ 4.025858] device: 'ptyv1': device_add
> [ 4.029803] device: 'ptyv2': device_add
> [ 4.033787] device: 'ptyv3': device_add
> [ 4.037727] device: 'ptyv4': device_add
> [ 4.041662] device: 'ptyv5': device_add
> [ 4.045613] device: 'ptyv6': device_add
> [ 4.049571] device: 'ptyv7': device_add
> [ 4.053526] device: 'ptyv8': device_add
> [ 4.057476] device: 'ptyv9': device_add
> [ 4.061442] device: 'ptyva': device_add
> [ 4.065389] device: 'ptyvb': device_add
> [ 4.069355] device: 'ptyvc': device_add
> [ 4.073298] device: 'ptyvd': device_add
> [ 4.077242] device: 'ptyve': device_add
> [ 4.081191] device: 'ptyvf': device_add
> [ 4.085119] device: 'ptyw0': device_add
> [ 4.089070] device: 'ptyw1': device_add
> [ 4.092998] device: 'ptyw2': device_add
> [ 4.096949] device: 'ptyw3': device_add
> [ 4.100895] device: 'ptyw4': device_add
> [ 4.104839] device: 'ptyw5': device_add
> [ 4.108796] device: 'ptyw6': device_add
> [ 4.112733] device: 'ptyw7': device_add
> [ 4.116668] device: 'ptyw8': device_add
> [ 4.120628] device: 'ptyw9': device_add
> [ 4.124595] device: 'ptywa': device_add
> [ 4.128545] device: 'ptywb': device_add
> [ 4.132509] device: 'ptywc': device_add
> [ 4.136443] device: 'ptywd': device_add
> [ 4.140389] device: 'ptywe': device_add
> [ 4.144339] device: 'ptywf': device_add
> [ 4.148275] device: 'ptyx0': device_add
> [ 4.152190] device: 'ptyx1': device_add
> [ 4.156137] device: 'ptyx2': device_add
> [ 4.160067] device: 'ptyx3': device_add
> [ 4.164021] device: 'ptyx4': device_add
> [ 4.167974] device: 'ptyx5': device_add
> [ 4.171922] device: 'ptyx6': device_add
> [ 4.175872] device: 'ptyx7': device_add
> [ 4.179799] device: 'ptyx8': device_add
> [ 4.183734] device: 'ptyx9': device_add
> [ 4.187689] device: 'ptyxa': device_add
> [ 4.191647] device: 'ptyxb': device_add
> [ 4.195571] device: 'ptyxc': device_add
> [ 4.199544] device: 'ptyxd': device_add
> [ 4.203478] device: 'ptyxe': device_add
> [ 4.207429] device: 'ptyxf': device_add
> [ 4.211355] device: 'ptyy0': device_add
> [ 4.215318] device: 'ptyy1': device_add
> [ 4.219271] device: 'ptyy2': device_add
> [ 4.223214] device: 'ptyy3': device_add
> [ 4.227177] device: 'ptyy4': device_add
> [ 4.231143] device: 'ptyy5': device_add
> [ 4.235095] device: 'ptyy6': device_add
> [ 4.239049] device: 'ptyy7': device_add
> [ 4.243018] device: 'ptyy8': device_add
> [ 4.246968] device: 'ptyy9': device_add
> [ 4.250913] device: 'ptyya': device_add
> [ 4.254854] device: 'ptyyb': device_add
> [ 4.258815] device: 'ptyyc': device_add
> [ 4.262759] device: 'ptyyd': device_add
> [ 4.266696] device: 'ptyye': device_add
> [ 4.270640] device: 'ptyyf': device_add
> [ 4.274590] device: 'ptyz0': device_add
> [ 4.278527] device: 'ptyz1': device_add
> [ 4.282472] device: 'ptyz2': device_add
> [ 4.286447] device: 'ptyz3': device_add
> [ 4.290379] device: 'ptyz4': device_add
> [ 4.294315] device: 'ptyz5': device_add
> [ 4.298265] device: 'ptyz6': device_add
> [ 4.302194] device: 'ptyz7': device_add
> [ 4.306141] device: 'ptyz8': device_add
> [ 4.310113] device: 'ptyz9': device_add
> [ 4.314041] device: 'ptyza': device_add
> [ 4.317993] device: 'ptyzb': device_add
> [ 4.321939] device: 'ptyzc': device_add
> [ 4.325898] device: 'ptyzd': device_add
> [ 4.329876] device: 'ptyze': device_add
> [ 4.333819] device: 'ptyzf': device_add
> [ 4.337762] device: 'ptya0': device_add
> [ 4.341721] device: 'ptya1': device_add
> [ 4.345655] device: 'ptya2': device_add
> [ 4.349605] device: 'ptya3': device_add
> [ 4.353555] device: 'ptya4': device_add
> [ 4.357509] device: 'ptya5': device_add
> [ 4.361472] device: 'ptya6': device_add
> [ 4.365404] device: 'ptya7': device_add
> [ 4.369337] device: 'ptya8': device_add
> [ 4.373275] device: 'ptya9': device_add
> [ 4.377237] device: 'ptyaa': device_add
> [ 4.381172] device: 'ptyab': device_add
> [ 4.385136] device: 'ptyac': device_add
> [ 4.389063] device: 'ptyad': device_add
> [ 4.392997] device: 'ptyae': device_add
> [ 4.396939] device: 'ptyaf': device_add
> [ 4.400900] device: 'ptyb0': device_add
> [ 4.404833] device: 'ptyb1': device_add
> [ 4.408792] device: 'ptyb2': device_add
> [ 4.412751] device: 'ptyb3': device_add
> [ 4.416712] device: 'ptyb4': device_add
> [ 4.420639] device: 'ptyb5': device_add
> [ 4.424581] device: 'ptyb6': device_add
> [ 4.428561] device: 'ptyb7': device_add
> [ 4.432524] device: 'ptyb8': device_add
> [ 4.436465] device: 'ptyb9': device_add
> [ 4.440417] device: 'ptyba': device_add
> [ 4.444356] device: 'ptybb': device_add
> [ 4.448308] device: 'ptybc': device_add
> [ 4.452234] device: 'ptybd': device_add
> [ 4.456159] device: 'ptybe': device_add
> [ 4.460108] device: 'ptybf': device_add
> [ 4.464038] device: 'ptyc0': device_add
> [ 4.467973] device: 'ptyc1': device_add
> [ 4.471930] device: 'ptyc2': device_add
> [ 4.475885] device: 'ptyc3': device_add
> [ 4.479820] device: 'ptyc4': device_add
> [ 4.483784] device: 'ptyc5': device_add
> [ 4.487731] device: 'ptyc6': device_add
> [ 4.491673] device: 'ptyc7': device_add
> [ 4.495627] device: 'ptyc8': device_add
> [ 4.499583] device: 'ptyc9': device_add
> [ 4.503533] device: 'ptyca': device_add
> [ 4.507485] device: 'ptycb': device_add
> [ 4.511412] device: 'ptycc': device_add
> [ 4.515382] device: 'ptycd': device_add
> [ 4.519324] device: 'ptyce': device_add
> [ 4.523252] device: 'ptycf': device_add
> [ 4.527191] device: 'ptyd0': device_add
> [ 4.531126] device: 'ptyd1': device_add
> [ 4.535065] device: 'ptyd2': device_add
> [ 4.539031] device: 'ptyd3': device_add
> [ 4.542987] device: 'ptyd4': device_add
> [ 4.546944] device: 'ptyd5': device_add
> [ 4.550892] device: 'ptyd6': device_add
> [ 4.554849] device: 'ptyd7': device_add
> [ 4.558809] device: 'ptyd8': device_add
> [ 4.562757] device: 'ptyd9': device_add
> [ 4.566709] device: 'ptyda': device_add
> [ 4.570643] device: 'ptydb': device_add
> [ 4.574568] device: 'ptydc': device_add
> [ 4.578508] device: 'ptydd': device_add
> [ 4.582458] device: 'ptyde': device_add
> [ 4.586394] device: 'ptydf': device_add
> [ 4.590345] device: 'ptye0': device_add
> [ 4.594279] device: 'ptye1': device_add
> [ 4.598208] device: 'ptye2': device_add
> [ 4.602142] device: 'ptye3': device_add
> [ 4.606085] device: 'ptye4': device_add
> [ 4.610027] device: 'ptye5': device_add
> [ 4.613964] device: 'ptye6': device_add
> [ 4.617920] device: 'ptye7': device_add
> [ 4.621871] device: 'ptye8': device_add
> [ 4.625811] device: 'ptye9': device_add
> [ 4.629767] device: 'ptyea': device_add
> [ 4.633728] device: 'ptyeb': device_add
> [ 4.637667] device: 'ptyec': device_add
> [ 4.641619] device: 'ptyed': device_add
> [ 4.645572] device: 'ptyee': device_add
> [ 4.649509] device: 'ptyef': device_add
> [ 4.653449] device: 'ttyp0': device_add
> [ 4.657388] device: 'ttyp1': device_add
> [ 4.661322] device: 'ttyp2': device_add
> [ 4.665253] device: 'ttyp3': device_add
> [ 4.669218] device: 'ttyp4': device_add
> [ 4.673170] device: 'ttyp5': device_add
> [ 4.677115] device: 'ttyp6': device_add
> [ 4.681082] device: 'ttyp7': device_add
> [ 4.685022] device: 'ttyp8': device_add
> [ 4.688976] device: 'ttyp9': device_add
> [ 4.692915] device: 'ttypa': device_add
> [ 4.696850] device: 'ttypb': device_add
> [ 4.700809] device: 'ttypc': device_add
> [ 4.704771] device: 'ttypd': device_add
> [ 4.708714] device: 'ttype': device_add
> [ 4.712662] device: 'ttypf': device_add
> [ 4.716633] device: 'ttyq0': device_add
> [ 4.720567] device: 'ttyq1': device_add
> [ 4.724504] device: 'ttyq2': device_add
> [ 4.728451] device: 'ttyq3': device_add
> [ 4.732403] device: 'ttyq4': device_add
> [ 4.736348] device: 'ttyq5': device_add
> [ 4.740291] device: 'ttyq6': device_add
> [ 4.744259] device: 'ttyq7': device_add
> [ 4.748204] device: 'ttyq8': device_add
> [ 4.752141] device: 'ttyq9': device_add
> [ 4.756094] device: 'ttyqa': device_add
> [ 4.760045] device: 'ttyqb': device_add
> [ 4.763989] device: 'ttyqc': device_add
> [ 4.767932] device: 'ttyqd': device_add
> [ 4.771874] device: 'ttyqe': device_add
> [ 4.775824] device: 'ttyqf': device_add
> [ 4.779770] device: 'ttyr0': device_add
> [ 4.783722] device: 'ttyr1': device_add
> [ 4.787674] device: 'ttyr2': device_add
> [ 4.791651] device: 'ttyr3': device_add
> [ 4.795606] device: 'ttyr4': device_add
> [ 4.799552] device: 'ttyr5': device_add
> [ 4.803504] device: 'ttyr6': device_add
> [ 4.807457] device: 'ttyr7': device_add
> [ 4.811412] device: 'ttyr8': device_add
> [ 4.815360] device: 'ttyr9': device_add
> [ 4.819290] device: 'ttyra': device_add
> [ 4.823231] device: 'ttyrb': device_add
> [ 4.827173] device: 'ttyrc': device_add
> [ 4.831106] device: 'ttyrd': device_add
> [ 4.835046] device: 'ttyre': device_add
> [ 4.838974] device: 'ttyrf': device_add
> [ 4.842916] device: 'ttys0': device_add
> [ 4.846852] device: 'ttys1': device_add
> [ 4.850792] device: 'ttys2': device_add
> [ 4.854756] device: 'ttys3': device_add
> [ 4.858717] device: 'ttys4': device_add
> [ 4.862670] device: 'ttys5': device_add
> [ 4.866612] device: 'ttys6': device_add
> [ 4.870587] device: 'ttys7': device_add
> [ 4.874522] device: 'ttys8': device_add
> [ 4.878458] device: 'ttys9': device_add
> [ 4.882426] device: 'ttysa': device_add
> [ 4.886369] device: 'ttysb': device_add
> [ 4.890338] device: 'ttysc': device_add
> [ 4.894289] device: 'ttysd': device_add
> [ 4.898238] device: 'ttyse': device_add
> [ 4.902193] device: 'ttysf': device_add
> [ 4.906135] device: 'ttyt0': device_add
> [ 4.910072] device: 'ttyt1': device_add
> [ 4.914011] device: 'ttyt2': device_add
> [ 4.917947] device: 'ttyt3': device_add
> [ 4.921891] device: 'ttyt4': device_add
> [ 4.925857] device: 'ttyt5': device_add
> [ 4.929803] device: 'ttyt6': device_add
> [ 4.933781] device: 'ttyt7': device_add
> [ 4.937720] device: 'ttyt8': device_add
> [ 4.941664] device: 'ttyt9': device_add
> [ 4.945633] device: 'ttyta': device_add
> [ 4.949584] device: 'ttytb': device_add
> [ 4.953542] device: 'ttytc': device_add
> [ 4.957491] device: 'ttytd': device_add
> [ 4.961416] device: 'ttyte': device_add
> [ 4.965351] device: 'ttytf': device_add
> [ 4.969278] device: 'ttyu0': device_add
> [ 4.973211] device: 'ttyu1': device_add
> [ 4.977183] device: 'ttyu2': device_add
> [ 4.981147] device: 'ttyu3': device_add
> [ 4.985083] device: 'ttyu4': device_add
> [ 4.989069] device: 'ttyu5': device_add
> [ 4.993004] device: 'ttyu6': device_add
> [ 4.996947] device: 'ttyu7': device_add
> [ 5.000915] device: 'ttyu8': device_add
> [ 5.004864] device: 'ttyu9': device_add
> [ 5.008809] device: 'ttyua': device_add
> [ 5.012787] device: 'ttyub': device_add
> [ 5.016741] device: 'ttyuc': device_add
> [ 5.020678] device: 'ttyud': device_add
> [ 5.024620] device: 'ttyue': device_add
> [ 5.028582] device: 'ttyuf': device_add
> [ 5.032538] device: 'ttyv0': device_add
> [ 5.036484] device: 'ttyv1': device_add
> [ 5.040424] device: 'ttyv2': device_add
> [ 5.044378] device: 'ttyv3': device_add
> [ 5.048338] device: 'ttyv4': device_add
> [ 5.052285] device: 'ttyv5': device_add
> [ 5.056236] device: 'ttyv6': device_add
> [ 5.060177] device: 'ttyv7': device_add
> [ 5.064125] device: 'ttyv8': device_add
> [ 5.068097] device: 'ttyv9': device_add
> [ 5.072028] device: 'ttyva': device_add
> [ 5.075978] device: 'ttyvb': device_add
> [ 5.079944] device: 'ttyvc': device_add
> [ 5.083881] device: 'ttyvd': device_add
> [ 5.087833] device: 'ttyve': device_add
> [ 5.091776] device: 'ttyvf': device_add
> [ 5.095722] device: 'ttyw0': device_add
> [ 5.099672] device: 'ttyw1': device_add
> [ 5.103618] device: 'ttyw2': device_add
> [ 5.107568] device: 'ttyw3': device_add
> [ 5.111495] device: 'ttyw4': device_add
> [ 5.115441] device: 'ttyw5': device_add
> [ 5.119380] device: 'ttyw6': device_add
> [ 5.123368] device: 'ttyw7': device_add
> [ 5.127314] device: 'ttyw8': device_add
> [ 5.131258] device: 'ttyw9': device_add
> [ 5.135208] device: 'ttywa': device_add
> [ 5.139177] device: 'ttywb': device_add
> [ 5.143159] device: 'ttywc': device_add
> [ 5.147113] device: 'ttywd': device_add
> [ 5.151059] device: 'ttywe': device_add
> [ 5.155032] device: 'ttywf': device_add
> [ 5.158957] device: 'ttyx0': device_add
> [ 5.162901] device: 'ttyx1': device_add
> [ 5.166870] device: 'ttyx2': device_add
> [ 5.170811] device: 'ttyx3': device_add
> [ 5.174752] device: 'ttyx4': device_add
> [ 5.178712] device: 'ttyx5': device_add
> [ 5.182646] device: 'ttyx6': device_add
> [ 5.186615] device: 'ttyx7': device_add
> [ 5.190556] device: 'ttyx8': device_add
> [ 5.194491] device: 'ttyx9': device_add
> [ 5.198418] device: 'ttyxa': device_add
> [ 5.202377] device: 'ttyxb': device_add
> [ 5.206303] device: 'ttyxc': device_add
> [ 5.210258] device: 'ttyxd': device_add
> [ 5.214210] device: 'ttyxe': device_add
> [ 5.218154] device: 'ttyxf': device_add
> [ 5.222094] device: 'ttyy0': device_add
> [ 5.226041] device: 'ttyy1': device_add
> [ 5.229985] device: 'ttyy2': device_add
> [ 5.233920] device: 'ttyy3': device_add
> [ 5.237883] device: 'ttyy4': device_add
> [ 5.241820] device: 'ttyy5': device_add
> [ 5.245767] device: 'ttyy6': device_add
> [ 5.249703] device: 'ttyy7': device_add
> [ 5.253672] device: 'ttyy8': device_add
> [ 5.257606] device: 'ttyy9': device_add
> [ 5.261556] device: 'ttyya': device_add
> [ 5.265496] device: 'ttyyb': device_add
> [ 5.269421] device: 'ttyyc': device_add
> [ 5.273357] device: 'ttyyd': device_add
> [ 5.277313] device: 'ttyye': device_add
> [ 5.281273] device: 'ttyyf': device_add
> [ 5.285209] device: 'ttyz0': device_add
> [ 5.289177] device: 'ttyz1': device_add
> [ 5.293118] device: 'ttyz2': device_add
> [ 5.297071] device: 'ttyz3': device_add
> [ 5.301035] device: 'ttyz4': device_add
> [ 5.304968] device: 'ttyz5': device_add
> [ 5.308938] device: 'ttyz6': device_add
> [ 5.312877] device: 'ttyz7': device_add
> [ 5.316807] device: 'ttyz8': device_add
> [ 5.320767] device: 'ttyz9': device_add
> [ 5.324715] device: 'ttyza': device_add
> [ 5.328648] device: 'ttyzb': device_add
> [ 5.332593] device: 'ttyzc': device_add
> [ 5.336559] device: 'ttyzd': device_add
> [ 5.340508] device: 'ttyze': device_add
> [ 5.344460] device: 'ttyzf': device_add
> [ 5.348397] device: 'ttya0': device_add
> [ 5.352361] device: 'ttya1': device_add
> [ 5.356289] device: 'ttya2': device_add
> [ 5.360226] device: 'ttya3': device_add
> [ 5.364181] device: 'ttya4': device_add
> [ 5.368122] device: 'ttya5': device_add
> [ 5.372055] device: 'ttya6': device_add
> [ 5.375994] device: 'ttya7': device_add
> [ 5.379934] device: 'ttya8': device_add
> [ 5.383862] device: 'ttya9': device_add
> [ 5.387825] device: 'ttyaa': device_add
> [ 5.391770] device: 'ttyab': device_add
> [ 5.395691] device: 'ttyac': device_add
> [ 5.399624] device: 'ttyad': device_add
> [ 5.403557] device: 'ttyae': device_add
> [ 5.407528] device: 'ttyaf': device_add
> [ 5.411479] device: 'ttyb0': device_add
> [ 5.415433] device: 'ttyb1': device_add
> [ 5.419376] device: 'ttyb2': device_add
> [ 5.423338] device: 'ttyb3': device_add
> [ 5.427272] device: 'ttyb4': device_add
> [ 5.431235] device: 'ttyb5': device_add
> [ 5.435164] device: 'ttyb6': device_add
> [ 5.439112] device: 'ttyb7': device_add
> [ 5.443070] device: 'ttyb8': device_add
> [ 5.447003] device: 'ttyb9': device_add
> [ 5.450958] device: 'ttyba': device_add
> [ 5.454917] device: 'ttybb': device_add
> [ 5.458849] device: 'ttybc': device_add
> [ 5.462812] device: 'ttybd': device_add
> [ 5.466797] device: 'ttybe': device_add
> [ 5.470772] device: 'ttybf': device_add
> [ 5.474736] device: 'ttyc0': device_add
> [ 5.478669] device: 'ttyc1': device_add
> [ 5.482610] device: 'ttyc2': device_add
> [ 5.486582] device: 'ttyc3': device_add
> [ 5.490544] device: 'ttyc4': device_add
> [ 5.494531] device: 'ttyc5': device_add
> [ 5.498478] device: 'ttyc6': device_add
> [ 5.502414] device: 'ttyc7': device_add
> [ 5.506395] device: 'ttyc8': device_add
> [ 5.510353] device: 'ttyc9': device_add
> [ 5.514276] device: 'ttyca': device_add
> [ 5.518243] device: 'ttycb': device_add
> [ 5.522186] device: 'ttycc': device_add
> [ 5.526144] device: 'ttycd': device_add
> [ 5.530090] device: 'ttyce': device_add
> [ 5.534039] device: 'ttycf': device_add
> [ 5.537997] device: 'ttyd0': device_add
> [ 5.541938] device: 'ttyd1': device_add
> [ 5.545884] device: 'ttyd2': device_add
> [ 5.549840] device: 'ttyd3': device_add
> [ 5.553800] device: 'ttyd4': device_add
> [ 5.557743] device: 'ttyd5': device_add
> [ 5.561712] device: 'ttyd6': device_add
> [ 5.565660] device: 'ttyd7': device_add
> [ 5.569604] device: 'ttyd8': device_add
> [ 5.573600] device: 'ttyd9': device_add
> [ 5.577562] device: 'ttyda': device_add
> [ 5.581547] device: 'ttydb': device_add
> [ 5.585505] device: 'ttydc': device_add
> [ 5.589465] device: 'ttydd': device_add
> [ 5.593419] device: 'ttyde': device_add
> [ 5.597372] device: 'ttydf': device_add
> [ 5.601311] device: 'ttye0': device_add
> [ 5.605279] device: 'ttye1': device_add
> [ 5.609259] device: 'ttye2': device_add
> [ 5.613199] device: 'ttye3': device_add
> [ 5.617211] device: 'ttye4': device_add
> [ 5.621153] device: 'ttye5': device_add
> [ 5.625099] device: 'ttye6': device_add
> [ 5.629045] device: 'ttye7': device_add
> [ 5.632989] device: 'ttye8': device_add
> [ 5.636939] device: 'ttye9': device_add
> [ 5.640894] device: 'ttyea': device_add
> [ 5.644851] device: 'ttyeb': device_add
> [ 5.648779] device: 'ttyec': device_add
> [ 5.652724] device: 'ttyed': device_add
> [ 5.656657] device: 'ttyee': device_add
> [ 5.660609] device: 'ttyef': device_add
> [ 5.664553] device: 'ptmx': device_add
> [ 5.668457] Serial: 8250/16550 driver, 4 ports, IRQ sharing disabled
> [ 5.674981] device: 'serial8250': device_add
> [ 5.679377] device: 'ttyS0': device_add
> [ 5.683422] device: 'ttyS1': device_add
> [ 5.687396] device: 'ttyS2': device_add
> [ 5.691400] device: 'ttyS3': device_add
> [ 5.695929] platform 840c000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.703851] platform 840d000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.711737] platform 840e000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.719789] platform 840c000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.727676] platform 840d000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.735536] platform 840e000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.743685] device: 'hw_random': device_add
> [ 5.748211] iproc-rng200 8402000.rng: hwrng registered
> [ 5.753969] device: '1:0': device_add
> [ 5.757799] device: 'ram0': device_add
> [ 5.761939] device: '1:1': device_add
> [ 5.765724] device: 'ram1': device_add
> [ 5.769767] device: '1:2': device_add
> [ 5.773566] device: 'ram2': device_add
> [ 5.777569] device: '1:3': device_add
> [ 5.781334] device: 'ram3': device_add
> [ 5.785340] device: '1:4': device_add
> [ 5.789099] device: 'ram4': device_add
> [ 5.793117] device: '1:5': device_add
> [ 5.796893] device: 'ram5': device_add
> [ 5.800888] device: '1:6': device_add
> [ 5.804674] device: 'ram6': device_add
> [ 5.808693] device: '1:7': device_add
> [ 5.812474] device: 'ram7': device_add
> [ 5.816504] device: '1:8': device_add
> [ 5.820256] device: 'ram8': device_add
> [ 5.824259] device: '1:9': device_add
> [ 5.828039] device: 'ram9': device_add
> [ 5.832072] device: '1:10': device_add
> [ 5.835929] device: 'ram10': device_add
> [ 5.840031] device: '1:11': device_add
> [ 5.843929] device: 'ram11': device_add
> [ 5.848033] device: '1:12': device_add
> [ 5.851908] device: 'ram12': device_add
> [ 5.856015] device: '1:13': device_add
> [ 5.859889] device: 'ram13': device_add
> [ 5.863998] device: '1:14': device_add
> [ 5.867855] device: 'ram14': device_add
> [ 5.871974] device: '1:15': device_add
> [ 5.875841] device: 'ram15': device_add
> [ 5.879976] brd: module loaded
> [ 5.883101] device: 'loop-control': device_add
> [ 5.887960] device: '7:0': device_add
> [ 5.891760] device: 'loop0': device_add
> [ 5.896215] device: '7:1': device_add
> [ 5.900005] device: 'loop1': device_add
> [ 5.904458] device: '7:2': device_add
> [ 5.908227] device: 'loop2': device_add
> [ 5.912686] device: '7:3': device_add
> [ 5.916467] device: 'loop3': device_add
> [ 5.920970] device: '7:4': device_add
> [ 5.924767] device: 'loop4': device_add
> [ 5.929216] device: '7:5': device_add
> [ 5.933017] device: 'loop5': device_add
> [ 5.937483] device: '7:6': device_add
> [ 5.941271] device: 'loop6': device_add
> [ 5.945724] device: '7:7': device_add
> [ 5.949513] device: 'loop7': device_add
> [ 5.953718] loop: module loaded
> [ 5.957414] platform 8b0a000.sata: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.965310] device: 'mtd-0': device_add
> [ 5.970044] platform 84b2120.qspi: probe deferral - supplier brcm_scmi@0 not ready
> [ 5.977854] brcmstb_qspi 8418000.mspi: using mspi mode
> [ 5.983168] device: 'spi0': device_add
> [ 5.988043] device: 'Fixed MDIO bus.0': device_add
> [ 5.993124] device: 'fixed-0': device_add
> [ 5.997362] libphy: Fixed MDIO Bus: probed
> [ 6.002042] platform 8f00000.ethernet: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.010377] e1000e: Intel(R) PRO/1000 Network Driver
> [ 6.015435] e1000e: Copyright(c) 1999 - 2015 Intel Corporation.
> [ 6.021476] pegasus: v0.9.3 (2013/04/25), Pegasus/Pegasus II USB Ethernet driver
> [ 6.029039] usbcore: registered new interface driver pegasus
> [ 6.034804] usbcore: registered new interface driver r8152
> [ 6.040405] usbcore: registered new interface driver asix
> [ 6.045918] usbcore: registered new interface driver ax88179_178a
> [ 6.052155] usbcore: registered new interface driver cdc_ether
> [ 6.058146] usbcore: registered new interface driver cdc_ncm
> [ 6.063989] device: 'usbmon0': device_add
> [ 6.068148] ehci_hcd: USB 2.0 'Enhanced' Host Controller (EHCI) Driver
> [ 6.074791] ehci-pci: EHCI PCI platform driver
> [ 6.079354] ohci_hcd: USB 1.1 'Open' Host Controller (OHCI) Driver
> [ 6.085630] ohci-pci: OHCI PCI platform driver
> [ 6.090220] usbcore: registered new interface driver cdc_acm
> [ 6.095991] cdc_acm: USB Abstract Control Model driver for USB modems and ISDN adapters
> [ 6.104131] usbcore: registered new interface driver usb-storage
> [ 6.110544] device: 'uinput': device_add
> [ 6.114617] i2c /dev entries driver
> [ 6.118853] brcmstb_thermal a581500.thermal: registered AVS TMON of-sensor driver
> [ 6.126867] device: 'watchdog': device_add
> [ 6.131143] device: 'watchdog0': device_add
> [ 6.135483] bcm7038-wdt 840a628.watchdog: Registered BCM7038 Watchdog
> [ 6.142519] sdhci: Secure Digital Host Controller Interface driver
> [ 6.148844] sdhci: Copyright(c) Pierre Ossman
> [ 6.153291] sdhci-pltfm: SDHCI platform and OF driver helper
> [ 6.159169] platform 84b0000.sdhci: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.159415] hid: raw HID events driver (C) Jiri Kosina
> [ 6.172246] device: 'uhid': device_add
> [ 6.176268] usbcore: registered new interface driver usbhid
> [ 6.181993] usbhid: USB HID core driver
> [ 6.599764] brcmstb-dpfe 9932000.dpfe-cpu: registered with API v3.
> [ 6.606251] device: 'timer': device_add
> [ 6.610336] device: 'snd-soc-dummy': device_add
> [ 6.615681] NET: Registered protocol family 10
> [ 6.620674] Segment Routing with IPv6
> [ 6.624441] sit: IPv6, IPv4 and MPLS over IPv4 tunneling driver
> [ 6.630479] device: 'sit0': device_add
> [ 6.634623] NET: Registered protocol family 17
> [ 6.639197] bridge: filtering via arp/ip/ip6tables is no longer available by default. Update your scripts to load br_netfilter if you ne
> ed this.
> [ 6.652522] 8021q: 802.1Q VLAN Support v1.8
> [ 6.656809] Key type dns_resolver registered
> [ 6.661246] Registering SWP/SWPB emulation handler
> [ 6.666163] device: 'cpu_dma_latency': device_add
> [ 6.671063] Loading compiled-in X.509 certificates
> [ 6.678526] devices_kset: Moving brcm_scmi@0 to end of list
> [ 6.684262] devices_kset: Moving 840c000.serial to end of list
> [ 6.690186] devices_kset: Moving 840d000.serial to end of list
> [ 6.696153] devices_kset: Moving 840e000.serial to end of list
> [ 6.702102] devices_kset: Moving 8d0f200.usb-phy to end of list
> [ 6.708111] devices_kset: Moving 8d00000.xhci_v2 to end of list
> [ 6.714129] devices_kset: Moving 8d00000.xhci_v2 to end of list
> [ 6.720155] devices_kset: Moving 8f00000.ethernet to end of list
> [ 6.726252] devices_kset: Moving 8b0a000.sata to end of list
> [ 6.732018] devices_kset: Moving 84b0000.sdhci to end of list
> [ 6.737888] devices_kset: Moving 84b2120.qspi to end of list
> [ 6.743637] devices_kset: Moving 8b20000.pcie to end of list
> [ 6.749451] platform brcm_scmi@0: probe deferral - supplier brcm_scmi_mailbox@0 not ready
> [ 6.757750] devices_kset: Moving 8b20000.pcie to end of list
> [ 6.763541] platform 8b20000.pcie: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.771258] devices_kset: Moving 840c000.serial to end of list
> [ 6.777231] platform 840c000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.785157] devices_kset: Moving 840d000.serial to end of list
> [ 6.791114] platform 840d000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.799016] devices_kset: Moving 840e000.serial to end of list
> [ 6.804973] platform 840e000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.812846] devices_kset: Moving 8b0a000.sata to end of list
> [ 6.818674] platform 8b0a000.sata: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.826399] devices_kset: Moving 84b2120.qspi to end of list
> [ 6.832209] platform 84b2120.qspi: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.839891] devices_kset: Moving 8f00000.ethernet to end of list
> [ 6.846066] platform 8f00000.ethernet: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.854131] devices_kset: Moving 84b0000.sdhci to end of list
> [ 6.860167] platform 84b0000.sdhci: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.860776] devices_kset: Moving brcm_scmi@0 to end of list
> [ 6.873679] devices_kset: Moving 840c000.serial to end of list
> [ 6.879584] devices_kset: Moving 840d000.serial to end of list
> [ 6.885551] devices_kset: Moving 840e000.serial to end of list
> [ 6.891491] devices_kset: Moving 8d0f200.usb-phy to end of list
> [ 6.897526] devices_kset: Moving 8d00000.xhci_v2 to end of list
> [ 6.903579] devices_kset: Moving 8d00000.xhci_v2 to end of list
> [ 6.909657] devices_kset: Moving 8f00000.ethernet to end of list
> [ 6.915797] devices_kset: Moving 8b0a000.sata to end of list
> [ 6.921580] devices_kset: Moving 84b0000.sdhci to end of list
> [ 6.927461] devices_kset: Moving 84b2120.qspi to end of list
> [ 6.933236] devices_kset: Moving 8b20000.pcie to end of list
> [ 6.939028] platform brcm_scmi@0: probe deferral - supplier brcm_scmi_mailbox@0 not ready
> [ 6.947346] devices_kset: Moving 8b20000.pcie to end of list
> [ 6.953117] platform 8b20000.pcie: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.960869] devices_kset: Moving 840c000.serial to end of list
> [ 6.966877] platform 840c000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.974751] devices_kset: Moving 840d000.serial to end of list
> [ 6.980754] platform 840d000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 6.988645] devices_kset: Moving 840e000.serial to end of list
> [ 6.994627] platform 840e000.serial: probe deferral - supplier brcm_scmi@0 not ready
> [ 7.002535] devices_kset: Moving 8b0a000.sata to end of list
> [ 7.008306] platform 8b0a000.sata: probe deferral - supplier brcm_scmi@0 not ready
> [ 7.016040] devices_kset: Moving 84b2120.qspi to end of list
> [ 7.021854] platform 84b2120.qspi: probe deferral - supplier brcm_scmi@0 not ready
> [ 7.029536] devices_kset: Moving 8f00000.ethernet to end of list
> [ 7.035698] platform 8f00000.ethernet: probe deferral - supplier brcm_scmi@0 not ready
> [ 7.043728] devices_kset: Moving 84b0000.sdhci to end of list
> [ 7.049741] device: 'ubi_ctrl': device_add
> [ 7.049773] platform 84b0000.sdhci: probe deferral - supplier brcm_scmi@0 not ready
> [ 7.054352] cfg80211: Loading compiled-in X.509 certificates for regulatory database
> [ 7.076406] cfg80211: Loaded X.509 cert 'sforshee: 00b28ddf47aef9cea7'
> [ 7.083122] ALSA device list:
> [ 7.086142] No soundcards found.
> [ 7.089708] platform regulatory.0: Direct firmware load for regulatory.db failed with error -2
> [ 7.098506] cfg80211: failed to load regulatory.db
> [ 7.115369] Freeing unused kernel memory: 14336K
> [ 7.129918] Run /init as init process
> [ 7.133673] with arguments:
> [ 7.136688] /init
> [ 7.139004] with environment:
> [ 7.142204] HOME=/
> [ 7.144622] TERM=linux
> [ 7.147370] dyndbg=file drivers/base/core.c +p
> [ 7.256086] random: crng init done
>
>
>
>
>
>
>
>
>
>
>
>
>
On Tue, Apr 27, 2021 at 09:33:31AM -0400, Jim Quinlan wrote:
> On Tue, Apr 27, 2021 at 3:48 AM Cristian Marussi <[email protected]>
> wrote:
>
> > Hi Florian,
> >
> > On Mon, Apr 26, 2021 at 02:47:52PM -0700, Florian Fainelli wrote:
> > > On 4/26/21 2:33 PM, Saravana Kannan wrote:
> > > > On Mon, Apr 26, 2021 at 1:51 PM Florian Fainelli <[email protected]>
> > wrote:
> > > >>
> > > >> Hi Saravana,
> > > >>
> > > >> Adding Sudeep and Christian, Al and Jim.
> > > >>
> > > >> On 3/2/21 1:11 PM, Saravana Kannan wrote:
> > > >>> This reverts commit 3e4c982f1ce75faf5314477b8da296d2d00919df.
> > > >>>
> > > >>> Since all reported issues due to fw_devlink=on should be addressed by
> > > >>> this series, revert the revert. fw_devlink=on Take II.
> > > >>>
> > > >>> Signed-off-by: Saravana Kannan <[email protected]>
> > > >>
> > > >> This change breaks booting on SCMI-based platforms such as
> > ARCH_BRCMSTB.
> > > >> If I revert this change or boot with fw_devlink=permissive, then our
> > > >> systems boot again. From a quick look, the SCMI clock provider was
> > never
> > > >> probed which means that our UART driver never got a chance to get its
> > > >> clock and we have no console -> timeout.
> > > >
> > > > We explicitly landed changes to handle this condition. So we'll see if
> > > > this is what is happening.
> > > >
> > > >> Al, AFAICT you had started to analyze this before in the context of
> > > >> SCMI, do you mind sharing what you had found?
> > > >>
> > > >> Saravana, is there any debugging output that we can help provide?
> > > >
> > > > Thanks for the report. Couple of things that can help:
> > > > 1. Example DTS file (the final board file so that I can get the full
> > DT view).
> > >
> > > Attached BCX972160DV.dts which is one such system.
> > >
> > > > 2. Point out the UART device node and the SCMI device node that you
> > > > suspect is causing the issue.
> > >
> > > The SCMI provider node is brcm_scmi@0 and its sub-node protocol@14 is
> > > the clock provider. The UART node is /rdb/serial@840c000.
> > >
> > > > 3. Boot logs with dev_dbg changed to dev_info in device_link_add() and
> > > > device_links_check_suppliers()
> > >
> > > OK, I did not want to modify the kernel so I just enabled dynamic debug
> > > prints from the command line, if this is too verbose let me know.
> > >
> > > Thank you for your prompt response.
> > > --
> >
> > I may say something obvious but, looking at the kernel log, it seems to me
> > that it is the SCMI subsystem as a whole that is never initialized and goes
> > through a lot of deferrals because its only transport, the brcmstb-mbox, is
> > in turn never found to be ready, then as a consequence your serial, relying
> > on scmi-clock, is never initialized too.
> >
> I believe that the brcmstb-mbox node is in our DT for backwards
> compatibility with our older Linux only. Note that we use the compatible
> string '"arm,scmi-smc", "arm,scmi"'; the former chooses SMC transport and
> ignores custom mailboxes such as brcmstb-mbox.
>
Right..so it is even more wrong that it is waiting for the mailboxes...but
looking at the DT:
brcm_scmi_mailbox@0 {
#mbox-cells = <0x01>;
compatible = "brcm,brcmstb-mbox";
status = "disabled";
linux,phandle = <0x04>;
phandle = <0x04>;
};
brcm_scmi@0 {
compatible = "arm,scmi-smc\0arm,scmi";
mboxes = <0x04 0x00 0x04 0x01>;
mbox-names = "tx\0rx";
shmem = <0x05>;
status = "disabled";
arm,smc-id = <0x83000400>;
interrupt-names = "a2p";
#address-cells = <0x01>;
#size-cells = <0x00>;
it seems to me that even though you declare an SMC based transport (and in fact
you define the smc-id too) you also still define mboxes (as a fallback I suppose)
referring to the brcm_scmi_mailbox phandle, and while this is ignored by the SCMI
driver (because you have selected a compatible SMC transport) I imagine this dep
is picked up by fw_devlink which in fact says:
> [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
and stalls waiting for it. (but I'm not really familiar on how fw_devlink
internals works really...so I maybe off in these regards)
Thanks,
Cristian
> Regards,
> Jim Quinlan
> Broadcom STB
>
> >
> > In JUNO in fact I can see the same pattern of deferrals during boot (if I
> > enable dev_info in dev_link) until the ARM MHU mailboxes are found and
> > initialized and then firmware:scmi stops being deferred and it is
> > initialized with all the stack and its consumers.
> >
> > I cannot see the brcm_scmi_mailbox@0 being found here instead.
> >
> > Thanks
> >
> > Cristian
> >
> >
> > > Florian
> >
> > > [ 0.000000] Linux version 5.12.0-gef1244124349
> > (fainelli@fainelli-desktop) (arm-linux-gcc (GCC) 8.3.0, GNU ld (GNU
> > Binutils) 2.32) #32 S
> > > MP Mon Apr 26 14:29:52 PDT 2021
> > > [ 0.000000] CPU: ARMv7 Processor [420f1000] revision 0 (ARMv7),
> > cr=30c5383d
> > > [ 0.000000] CPU: div instructions available: patching division code
> > > [ 0.000000] CPU: PIPT / VIPT nonaliasing data cache, VIPT aliasing
> > instruction cache
> > > [ 0.000000] OF: fdt: Machine model: BCX972160DV
> > > [ 0.000000] earlycon: ns16550a0 at MMIO32 0x000000000840c000 (options
> > '115200')
> > > [ 0.000000] printk: bootconsole [ns16550a0] enabled
> > > [ 0.000000] printk: debug: skip boot console de-registration.
> > > [ 0.000000] Memory policy: Data cache writealloc
> > > [ 0.000000] cma: Reserved 16 MiB at 0x000000006f000000
> > > [ 0.000000] Zone ranges:
> > > [ 0.000000] DMA [mem 0x0000000040000000-0x000000006fffffff]
> > > [ 0.000000] Normal empty
> > > [ 0.000000] HighMem [mem 0x0000000070000000-0x000000013fffffff]
> > > [ 0.000000] Movable zone start for each node
> > > [ 0.000000] Early memory node ranges
> > > [ 0.000000] node 0: [mem 0x0000000040000000-0x00000000fdffefff]
> > > [ 0.000000] node 0: [mem 0x00000000fdfff000-0x00000000ffffffff]
> > > [ 0.000000] node 0: [mem 0x0000000100000000-0x000000013fffffff]
> > > [ 0.000000] Initmem setup node 0 [mem
> > 0x0000000040000000-0x000000013fffffff]
> > > [ 0.000000] On node 0 totalpages: 1048576
> > > [ 0.000000] DMA zone: 1920 pages used for memmap
> > > [ 0.000000] DMA zone: 0 pages reserved
> > > [ 0.000000] DMA zone: 196608 pages, LIFO batch:63
> > > [ 0.000000] HighMem zone: 851968 pages, LIFO batch:63
> > > [ 0.000000] psci: probing for conduit method from DT.
> > > [ 0.000000] psci: PSCIv1.1 detected in firmware.
> > > [ 0.000000] psci: Using standard PSCI v0.2 function IDs
> > > [ 0.000000] psci: Trusted OS resident on physical CPU 0x0
> > > [ 0.000000] psci: SMC Calling Convention v1.1
> > > [ 0.000000] percpu: Embedded 20 pages/cpu s49804 r8192 d23924 u81920
> > > [ 0.000000] pcpu-alloc: s49804 r8192 d23924 u81920 alloc=20*4096
> > > [ 0.000000] pcpu-alloc: [0] 0 [0] 1 [0] 2 [0] 3
> > > [ 0.000000] Built 1 zonelists, mobility grouping on. Total pages:
> > 1046656
> > > [ 0.000000] Kernel command line: fw_devlink=on dyndbg="file
> > drivers/base/core.c +p" debug earlycon keep_bootcon
> > > [ 0.000000] Dentry cache hash table entries: 131072 (order: 7, 524288
> > bytes, linear)
> > > [ 0.000000] Inode-cache hash table entries: 65536 (order: 6, 262144
> > bytes, linear)
> > > [ 0.000000] mem auto-init: stack:off, heap alloc:off, heap free:off
> > > [ 0.000000] software IO TLB: mapped [mem
> > 0x0000000068400000-0x000000006c400000] (64MB)
> > > [ 0.000000] Memory: 3986968K/4194304K available (12288K kernel code,
> > 1210K rwdata, 2648K rodata, 14336K init, 246K bss, 190952K reserved
> > > , 16384K cma-reserved, 3358716K highmem)
> > > [ 0.000000] SLUB: HWalign=64, Order=0-3, MinObjects=0, CPUs=4, Nodes=1
> > > [ 0.000000] ftrace: allocating 37847 entries in 74 pages
> > > [ 0.000000] ftrace: allocated 74 pages with 3 groups
> > > [ 0.000000] rcu: Hierarchical RCU implementation.
> > > [ 0.000000] rcu: RCU event tracing is enabled.
> > > [ 0.000000] Rude variant of Tasks RCU enabled.
> > > [ 0.000000] rcu: RCU calculated value of scheduler-enlistment delay
> > is 100 jiffies.
> > > [ 0.000000] NR_IRQS: 16, nr_irqs: 16, preallocated irqs: 16
> > > [ 0.000000] GIC: Using split EOI/Deactivate mode
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@8400000, parent irq: 24)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@84a1000, parent irq: 25)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@84a1800, parent irq: 26)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@8410640, parent irq: 27)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@841a880, parent irq: 28)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@840a400, parent irq: 29)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@841a380, parent irq: 30)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@840a440, parent irq: 31)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@841a3c0, parent irq: 32)
> > > [ 0.000000] irq_brcmstb_l2: registered L2 intc
> > (/rdb/interrupt-controller@8419000, parent irq: 33)
> > > [ 0.000000] random: get_random_bytes called from
> > start_kernel+0x42c/0x60c with crng_init=0
> > > [ 0.000000] arch_timer: cp15 timer(s) running at 27.00MHz (phys).
> > > [ 0.000000] clocksource: arch_sys_counter: mask: 0xffffffffffffff
> > max_cycles: 0x63a1e71a3, max_idle_ns: 440795203123 ns
> > > [ 0.000001] sched_clock: 56 bits at 27MHz, resolution 37ns, wraps
> > every 4398046511093ns
> > > [ 0.008134] Switching to timer-based delay loop, resolution 37ns
> > > [ 0.014453] Console: colour dummy device 80x30
> > > [ 0.018988] printk: console [tty0] enabled
> > > [ 0.023186] Calibrating delay loop (skipped), value calculated using
> > timer frequency.. 54.00 BogoMIPS (lpj=27000)
> > > [ 0.033654] pid_max: default: 32768 minimum: 301
> > > [ 0.038432] Mount-cache hash table entries: 2048 (order: 1, 8192
> > bytes, linear)
> > > [ 0.045855] Mountpoint-cache hash table entries: 2048 (order: 1, 8192
> > bytes, linear)
> > > [ 0.054210] CPU: Testing write buffer coherency: ok
> > > [ 0.059444] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000
> > > [ 0.065526] Setting up static identity map for 0x40200000 - 0x40200060
> > > [ 0.072250] rcu: Hierarchical SRCU implementation.
> > > [ 0.078389] brcmstb: biuctrl: MCP: Enabling write pairing
> > > [ 0.084034] smp: Bringing up secondary CPUs ...
> > > [ 0.089532] CPU1: thread -1, cpu 1, socket 0, mpidr 80000001
> > > [ 0.090446] CPU2: thread -1, cpu 2, socket 0, mpidr 80000002
> > > [ 0.091316] CPU3: thread -1, cpu 3, socket 0, mpidr 80000003
> > > [ 0.091378] smp: Brought up 1 node, 4 CPUs
> > > [ 0.112826] SMP: Total of 4 processors activated (216.00 BogoMIPS).
> > > [ 0.119209] CPU: All CPU(s) started in HYP mode.
> > > [ 0.123902] CPU: Virtualization extensions available.
> > > [ 0.130052] devtmpfs: initialized
> > > [ 0.135401] device: 'platform': device_add
> > > [ 0.139664] device: 'cpu': device_add
> > > [ 0.143419] device: 'container': device_add
> > > [ 0.148274] VFP support v0.3: implementor 42 architecture 3 part 10
> > variant 0 rev 0
> > > [ 0.156107] device: 'workqueue': device_add
> > > [ 0.160469] clocksource: jiffies: mask: 0xffffffff max_cycles:
> > 0xffffffff, max_idle_ns: 1911260446275000 ns
> > > [ 0.170396] futex hash table entries: 1024 (order: 4, 65536 bytes,
> > linear)
> > > [ 0.178019] pinctrl core: initialized pinctrl subsystem
> > > [ 0.183502] device: 'reg-dummy': device_add
> > > [ 0.187874] device: 'regulator.0': device_add
> > > [ 0.192941] NET: Registered protocol family 16
> > > [ 0.198853] DMA: preallocated 256 KiB pool for atomic coherent
> > allocations
> > > [ 0.206556] device: 'vtcon0': device_add
> > > [ 0.210923] thermal_sys: Registered thermal governor 'step_wise'
> > > [ 0.210942] thermal_sys: Registered thermal governor 'user_space'
> > > [ 0.217182] device: 'thermal_zone0': device_add
> > > [ 0.228089] device: 'thermal_zone1': device_add
> > > [ 0.232819] cpuidle: using governor menu
> > > [ 0.236916] No ATAGs?
> > > [ 0.239331] hw-breakpoint: found 5 (+1 reserved) breakpoint and 4
> > watchpoint registers.
> > > [ 0.247505] hw-breakpoint: maximum watchpoint size is 8 bytes.
> > > [ 0.253693] /rdb/syscon@84a2400: Broadcom Brahma-B15 readahead cache
> > > [ 0.260356] device: 'soc0': device_add
> > > [ 0.264207] Serial: AMBA PL011 UART driver
> > > [ 0.268567] device: 'psci': device_add
> > > [ 0.272585] device: 'timer': device_add
> > > [ 0.276708] device: 'brcm_scmi_mailbox@0': device_add
> > > [ 0.281972] device: 'brcm_scmi@0': device_add
> > > [ 0.286506] device: 'platform:brcm_scmi_mailbox@0--platform
> > :brcm_scmi@0': device_add
> > > [ 0.294403] devices_kset: Moving brcm_scmi@0 to end of list
> > > [ 0.300086] platform brcm_scmi@0: Linked as a consumer to
> > brcm_scmi_mailbox@0
> > > [ 0.307400] device: 'rdb': device_add
> > > [ 0.311163] OF: /rdb/brcmstb-clks/cpu_mdiv_ch0@8161010: could not
> > get #clock-cells for /rdb/brcmstb-clks/cpu_ndiv_int@8161018
> > > [ 0.324194] device: 'platform:brcm_scmi@0--platform:rdb': device_add
> > > [ 0.330675] platform rdb: Linked as a sync state only consumer to
> > brcm_scmi@0
> > > [ 0.338229] device: '840c000.serial': device_add
> > > [ 0.342957] device: 'platform:brcm_scmi@0--platform:840c000.serial':
> > device_add
> > > [ 0.350440] devices_kset: Moving 840c000.serial to end of list
> > > [ 0.356358] platform 840c000.serial: Linked as a consumer to
> > brcm_scmi@0
> > > [ 0.363315] device: '840d000.serial': device_add
> > > [ 0.368047] device: 'platform:brcm_scmi@0--platform:840d000.serial':
> > device_add
> > > [ 0.375512] devices_kset: Moving 840d000.serial to end of list
> > > [ 0.381446] platform 840d000.serial: Linked as a consumer to
> > brcm_scmi@0
> > > [ 0.388373] device: '840e000.serial': device_add
> > > [ 0.393116] device: 'platform:brcm_scmi@0--platform:840e000.serial':
> > device_add
> > > [ 0.400573] devices_kset: Moving 840e000.serial to end of list
> > > [ 0.406506] platform 840e000.serial: Linked as a consumer to
> > brcm_scmi@0
> > > [ 0.413451] device: '8b2c800.rescal': device_add
> > > [ 0.418349] device: '8d0f200.usb-phy': device_add
> > > [ 0.423175] device: 'platform:brcm_scmi@0--platform:8d0f200.usb-phy':
> > device_add
> > > [ 0.430729] devices_kset: Moving 8d0f200.usb-phy to end of list
> > > [ 0.436794] platform 8d0f200.usb-phy: Linked as a consumer to
> > brcm_scmi@0
> > > [ 0.443804] device: '8d00000.xhci_v2': device_add
> > > [ 0.448650] device: 'platform:brcm_scmi@0--platform:8d00000.xhci_v2':
> > device_add
> > > [ 0.456190] devices_kset: Moving 8d00000.xhci_v2 to end of list
> > > [ 0.462212] platform 8d00000.xhci_v2: Linked as a consumer to
> > brcm_scmi@0
> > > [ 0.469118] device:
> > 'platform:8d0f200.usb-phy--platform:8d00000.xhci_v2': device_add
> > > [ 0.477025] devices_kset: Moving 8d00000.xhci_v2 to end of list
> > > [ 0.483053] platform 8d00000.xhci_v2: Linked as a consumer to
> > 8d0f200.usb-phy
> > > [ 0.490510] device: '8f00000.ethernet': device_add
> > > [ 0.495435] device: 'platform:brcm_scmi@0--platform:8f00000.ethernet':
> > device_add
> > > [ 0.503067] devices_kset: Moving 8f00000.ethernet to end of list
> > > [ 0.509174] platform 8f00000.ethernet: Linked as a consumer to
> > brcm_scmi@0
> > > [ 0.516310] device: '840a500.gpio': device_add
> > > [ 0.520962] device: '841a400.gpio': device_add
> > > [ 0.525641] device: '8b0a000.sata': device_add
> > > [ 0.530223] device: 'platform:brcm_scmi@0--platform:8b0a000.sata':
> > device_add
> > > [ 0.537529] devices_kset: Moving 8b0a000.sata to end of list
> > > [ 0.543275] platform 8b0a000.sata: Linked as a consumer to brcm_scmi@0
> > > [ 0.549992] device: '8b08100.sata_phy': device_add
> > > [ 0.554899] device:
> > 'platform:8b08100.sata_phy--platform:8b0a000.sata': device_add
> > > [ 0.562609] platform 8b0a000.sata: Linked as a sync state only
> > consumer to 8b08100.sata_phy
> > > [ 0.571259] device: '84b0000.sdhci': device_add
> > > [ 0.575894] device: 'platform:brcm_scmi@0--platform:84b0000.sdhci':
> > device_add
> > > [ 0.583294] devices_kset: Moving 84b0000.sdhci to end of list
> > > [ 0.589133] platform 84b0000.sdhci: Linked as a consumer to
> > brcm_scmi@0
> > > [ 0.596079] device: '83fc000.gisb-arb': device_add
> > > [ 0.601132] device: '841a840.waketimer': device_add
> > > [ 0.606194] device: 'a581500.thermal': device_add
> > > [ 0.611102] device: '8410000.aon-ctrl': device_add
> > > [ 0.616025] device: 'rdb:memory_controllers': device_add
> > > [ 0.621502] device: 'rdb:memory_controllers:memc@0': device_add
> > > [ 0.627652] device: '9900000.memc-gen': device_add
> > > [ 0.632658] device: '9903000.memc-arb': device_add
> > > [ 0.637649] device: '9920000.ddr-phy': device_add
> > > [ 0.642559] device: '9908000.shimphy': device_add
> > > [ 0.647470] device: '9902000.memc-ddr': device_add
> > > [ 0.652578] device: '84b3400.spi': device_add
> > > [ 0.657422] device: '84b2120.qspi': device_add
> > > [ 0.662032] device: 'platform:brcm_scmi@0--platform:84b2120.qspi':
> > device_add
> > > [ 0.669346] devices_kset: Moving 84b2120.qspi to end of list
> > > [ 0.675099] platform 84b2120.qspi: Linked as a consumer to brcm_scmi@0
> > > [ 0.681851] device: '8404000.syscon': device_add
> > > [ 0.686640] device: '8404534.syscon': device_add
> > > [ 0.691437] device: '8404100.syscon': device_add
> > > [ 0.696226] device: '8404124.syscon': device_add
> > > [ 0.700992] device: '8410700.syscon': device_add
> > > [ 0.705773] device: '841070c.syscon': device_add
> > > [ 0.710565] device: '9903004.syscon': device_add
> > > [ 0.715348] device: '8404084.syscon': device_add
> > > [ 0.720141] device: '84040a4.syscon': device_add
> > > [ 0.724924] device: '84b0344.syscon': device_add
> > > [ 0.729715] device: '84b1344.syscon': device_add
> > > [ 0.734500] device: '84b1500.syscon': device_add
> > > [ 0.739283] device: '84b037c.syscon': device_add
> > > [ 0.744090] device: '84b137c.syscon': device_add
> > > [ 0.748875] device: '84040ac.syscon': device_add
> > > [ 0.753711] device: '840a628.watchdog': device_add
> > > [ 0.758676] device: '83b0000.bsp': device_add
> > > [ 0.763261] device: '8418000.mspi': device_add
> > > [ 0.767901] device: '8408000.pwm': device_add
> > > [ 0.772519] device: '9700000.gpu': device_add
> > > [ 0.777022] device: 'rdb:gpu-mmu@9705000': device_add
> > > [ 0.782254] device: '9910000.syscon': device_add
> > > [ 0.787033] device: '9909000.syscon': device_add
> > > [ 0.791869] device: '9932000.dpfe-cpu': device_add
> > > [ 0.796819] device: '8402000.rng': device_add
> > > [ 0.801358] device: '8404318.reset-controller': device_add
> > > [ 0.807013] device: '84a2400.syscon': device_add
> > > [ 0.811815] device: '8452000.syscon': device_add
> > > [ 0.816581] device: 'reboot': device_add
> > > [ 0.820800] device: 'pmu': device_add
> > > [ 0.824740] device: '8b20000.pcie': device_add
> > > [ 0.829402] device: 'platform:brcm_scmi@0--platform:8b20000.pcie':
> > device_add
> > > [ 0.836711] devices_kset: Moving 8b20000.pcie to end of list
> > > [ 0.842472] platform 8b20000.pcie: Linked as a consumer to brcm_scmi@0
> > > [ 0.849151] device: 'smpboot': device_add
> > > [ 0.853324] device: '100000.sram': device_add
> > > [ 0.857885] device: 'cpu0': device_add
> > > [ 0.861770] device: 'cpu1': device_add
> > > [ 0.865667] device: 'cpu2': device_add
> > > [ 0.869530] device: 'cpu3': device_add
> > > [ 0.883847] device: 'writeback': device_add
> > > [ 0.889865] SCSI subsystem initialized
> > > [ 0.893864] libata version 3.00 loaded.
> > > [ 0.897999] usbcore: registered new interface driver usbfs
> > > [ 0.903747] usbcore: registered new interface driver hub
> > > [ 0.909224] usbcore: registered new device driver usb
> > > [ 0.914426] mc: Linux media interface: v0.10
> > > [ 0.918809] videodev: Linux video capture interface: v2.00
> > > [ 0.924438] pps_core: LinuxPPS API ver. 1 registered
> > > [ 0.929498] pps_core: Software ver. 5.3.6 - Copyright 2005-2007
> > Rodolfo Giometti <[email protected]>
> > > [ 0.938794] PTP clock support registered
> > > [ 0.942998] platform brcm_scmi@0: probe deferral - supplier
> > brcm_scmi_mailbox@0 not ready
> > > [ 0.951637] Advanced Linux Sound Architecture Driver Initialized.
> > > [ 0.957934] device: 'lo': device_add
> > > [ 0.961904] device: 'rfkill': device_add
> > > [ 0.966762] clocksource: Switched to clocksource arch_sys_counter
> > > [ 1.003627] device: 'mem': device_add
> > > [ 1.007505] device: 'null': device_add
> > > [ 1.011388] device: 'port': device_add
> > > [ 1.015245] device: 'zero': device_add
> > > [ 1.019096] device: 'full': device_add
> > > [ 1.022942] device: 'random': device_add
> > > [ 1.026950] device: 'urandom': device_add
> > > [ 1.031079] device: 'kmsg': device_add
> > > [ 1.035020] device: 'tty': device_add
> > > [ 1.038831] device: 'console': device_add
> > > [ 1.042974] device: 'tty0': device_add
> > > [ 1.046862] device: 'vcs': device_add
> > > [ 1.050700] device: 'vcsu': device_add
> > > [ 1.054551] device: 'vcsa': device_add
> > > [ 1.058402] device: 'vcs1': device_add
> > > [ 1.062280] device: 'vcsu1': device_add
> > > [ 1.066224] device: 'vcsa1': device_add
> > > [ 1.070199] device: 'tty1': device_add
> > > [ 1.074081] device: 'tty2': device_add
> > > [ 1.077945] device: 'tty3': device_add
> > > [ 1.081798] device: 'tty4': device_add
> > > [ 1.085686] device: 'tty5': device_add
> > > [ 1.089517] device: 'tty6': device_add
> > > [ 1.093411] device: 'tty7': device_add
> > > [ 1.097295] device: 'tty8': device_add
> > > [ 1.101167] device: 'tty9': device_add
> > > [ 1.105025] device: 'tty10': device_add
> > > [ 1.109001] device: 'tty11': device_add
> > > [ 1.112966] device: 'tty12': device_add
> > > [ 1.116899] device: 'tty13': device_add
> > > [ 1.120850] device: 'tty14': device_add
> > > [ 1.124795] device: 'tty15': device_add
> > > [ 1.128739] device: 'tty16': device_add
> > > [ 1.132684] device: 'tty17': device_add
> > > [ 1.136623] device: 'tty18': device_add
> > > [ 1.140617] device: 'tty19': device_add
> > > [ 1.144553] device: 'tty20': device_add
> > > [ 1.148485] device: 'tty21': device_add
> > > [ 1.152405] device: 'tty22': device_add
> > > [ 1.156351] device: 'tty23': device_add
> > > [ 1.160287] device: 'tty24': device_add
> > > [ 1.164224] device: 'tty25': device_add
> > > [ 1.168171] device: 'tty26': device_add
> > > [ 1.172123] device: 'tty27': device_add
> > > [ 1.176078] device: 'tty28': device_add
> > > [ 1.180033] device: 'tty29': device_add
> > > [ 1.183996] device: 'tty30': device_add
> > > [ 1.187955] device: 'tty31': device_add
> > > [ 1.191913] device: 'tty32': device_add
> > > [ 1.195867] device: 'tty33': device_add
> > > [ 1.199802] device: 'tty34': device_add
> > > [ 1.203742] device: 'tty35': device_add
> > > [ 1.207702] device: 'tty36': device_add
> > > [ 1.211651] device: 'tty37': device_add
> > > [ 1.215628] device: 'tty38': device_add
> > > [ 1.219599] device: 'tty39': device_add
> > > [ 1.223560] device: 'tty40': device_add
> > > [ 1.227530] device: 'tty41': device_add
> > > [ 1.231491] device: 'tty42': device_add
> > > [ 1.235429] device: 'tty43': device_add
> > > [ 1.239398] device: 'tty44': device_add
> > > [ 1.243334] device: 'tty45': device_add
> > > [ 1.247261] device: 'tty46': device_add
> > > [ 1.251269] device: 'tty47': device_add
> > > [ 1.255217] device: 'tty48': device_add
> > > [ 1.259169] device: 'tty49': device_add
> > > [ 1.263136] device: 'tty50': device_add
> > > [ 1.267075] device: 'tty51': device_add
> > > [ 1.271072] device: 'tty52': device_add
> > > [ 1.275026] device: 'tty53': device_add
> > > [ 1.278965] device: 'tty54': device_add
> > > [ 1.282958] device: 'tty55': device_add
> > > [ 1.286905] device: 'tty56': device_add
> > > [ 1.290840] device: 'tty57': device_add
> > > [ 1.294797] device: 'tty58': device_add
> > > [ 1.298736] device: 'tty59': device_add
> > > [ 1.302709] device: 'tty60': device_add
> > > [ 1.306673] device: 'tty61': device_add
> > > [ 1.310628] device: 'tty62': device_add
> > > [ 1.314609] device: 'tty63': device_add
> > > [ 1.318700] NET: Registered protocol family 2
> > > [ 1.323375] tcp_listen_portaddr_hash hash table entries: 512 (order:
> > 0, 6144 bytes, linear)
> > > [ 1.331890] TCP established hash table entries: 8192 (order: 3, 32768
> > bytes, linear)
> > > [ 1.339829] TCP bind hash table entries: 8192 (order: 4, 65536 bytes,
> > linear)
> > > [ 1.347106] TCP: Hash tables configured (established 8192 bind 8192)
> > > [ 1.353670] UDP hash table entries: 512 (order: 2, 16384 bytes,
> > linear)
> > > [ 1.360436] UDP-Lite hash table entries: 512 (order: 2, 16384 bytes,
> > linear)
> > > [ 1.367702] NET: Registered protocol family 1
> > > [ 1.375093] RPC: Registered named UNIX socket transport module.
> > > [ 1.381137] RPC: Registered udp transport module.
> > > [ 1.385948] RPC: Registered tcp transport module.
> > > [ 1.390729] RPC: Registered tcp NFSv4.1 backchannel transport module.
> > > [ 1.397603] device: 'regulatory.0': device_add
> > > [ 1.402302] PCI: CLS 0 bytes, default 64
> > > [ 3.410785] device: 'clocksource': device_add
> > > [ 3.415283] device: 'clocksource0': device_add
> > > [ 3.420011] device: 'clockevents': device_add
> > > [ 3.424466] device: 'clockevent0': device_add
> > > [ 3.428918] device: 'clockevent1': device_add
> > > [ 3.433358] device: 'clockevent2': device_add
> > > [ 3.437820] device: 'clockevent3': device_add
> > > [ 3.442277] device: 'broadcast': device_add
> > > [ 3.446657] device: 'software': device_add
> > > [ 3.450861] device: 'tracepoint': device_add
> > > [ 3.455213] device: 'uprobe': device_add
> > > [ 3.459228] device: 'breakpoint': device_add
> > > [ 3.463589] Initialise system trusted keyrings
> > > [ 3.468304] workingset: timestamp_bits=30 max_order=20 bucket_order=0
> > > [ 3.477332] squashfs: version 4.0 (2009/01/31) Phillip Lougher
> > > [ 3.483696] NFS: Registering the id_resolver key type
> > > [ 3.488880] Key type id_resolver registered
> > > [ 3.493160] Key type id_legacy registered
> > > [ 3.497284] nfs4filelayout_init: NFSv4 File Layout Driver
> > Registering...
> > > [ 3.504137] nfs4flexfilelayout_init: NFSv4 Flexfile Layout Driver
> > Registering...
> > > [ 3.511656] jffs2: version 2.2. (NAND) ? 2001-2006 Red Hat, Inc.
> > > [ 3.518014] fuse: init (API version 7.33)
> > > [ 3.522124] device: 'fuse': device_add
> > > [ 3.526097] device: 'cuse': device_add
> > > [ 3.530132] Key type asymmetric registered
> > > [ 3.534299] Asymmetric key parser 'x509' registered
> > > [ 3.539303] bounce: pool size: 64 pages
> > > [ 3.543245] Block layer SCSI generic (bsg) driver version 0.4 loaded
> > (major 247)
> > > [ 3.550780] io scheduler mq-deadline registered
> > > [ 3.555372] io scheduler kyber registered
> > > [ 3.559815] brcm-gisb-arb 83fc000.gisb-arb: registered irqs: 53, 54
> > > [ 3.566619] device: 'phy-8b08100.sata_phy.0': device_add
> > > [ 3.572114] brcm-sata-phy 8b08100.sata_phy: registered 1 port(s)
> > > [ 3.578762] device: 'gpiochip0': device_add
> > > [ 3.583122] device: 'gpiochip0': device_add
> > > [ 3.587461] device: 'gpiochip1': device_add
> > > [ 3.591811] device: 'gpiochip32': device_add
> > > [ 3.596239] device: 'wakeup0': device_add
> > > [ 3.600547] device: 'gpiochip2': device_add
> > > [ 3.604914] device: 'gpiochip64': device_add
> > > [ 3.609343] device: 'gpiochip3': device_add
> > > [ 3.613662] device: 'gpiochip96': device_add
> > > [ 3.618077] device: 'wakeup1': device_add
> > > [ 3.622741] platform 8b20000.pcie: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 3.631146] brcmstb-pm: error mapping DDR PHY
> > > [ 3.635609] brcmstb-pm: probe of 8410000.aon-ctrl failed with error
> > -22
> > > [ 3.642854] device: 'ptyp0': device_add
> > > [ 3.646907] device: 'ptyp1': device_add
> > > [ 3.650874] device: 'ptyp2': device_add
> > > [ 3.654826] device: 'ptyp3': device_add
> > > [ 3.658785] device: 'ptyp4': device_add
> > > [ 3.662719] device: 'ptyp5': device_add
> > > [ 3.666673] device: 'ptyp6': device_add
> > > [ 3.670626] device: 'ptyp7': device_add
> > > [ 3.674571] device: 'ptyp8': device_add
> > > [ 3.678508] device: 'ptyp9': device_add
> > > [ 3.682469] device: 'ptypa': device_add
> > > [ 3.686417] device: 'ptypb': device_add
> > > [ 3.690368] device: 'ptypc': device_add
> > > [ 3.694310] device: 'ptypd': device_add
> > > [ 3.698243] device: 'ptype': device_add
> > > [ 3.702181] device: 'ptypf': device_add
> > > [ 3.706121] device: 'ptyq0': device_add
> > > [ 3.710059] device: 'ptyq1': device_add
> > > [ 3.714037] device: 'ptyq2': device_add
> > > [ 3.717993] device: 'ptyq3': device_add
> > > [ 3.721935] device: 'ptyq4': device_add
> > > [ 3.725896] device: 'ptyq5': device_add
> > > [ 3.729849] device: 'ptyq6': device_add
> > > [ 3.733799] device: 'ptyq7': device_add
> > > [ 3.737773] device: 'ptyq8': device_add
> > > [ 3.741709] device: 'ptyq9': device_add
> > > [ 3.745700] device: 'ptyqa': device_add
> > > [ 3.749652] device: 'ptyqb': device_add
> > > [ 3.753596] device: 'ptyqc': device_add
> > > [ 3.757547] device: 'ptyqd': device_add
> > > [ 3.761493] device: 'ptyqe': device_add
> > > [ 3.765428] device: 'ptyqf': device_add
> > > [ 3.769388] device: 'ptyr0': device_add
> > > [ 3.773339] device: 'ptyr1': device_add
> > > [ 3.777295] device: 'ptyr2': device_add
> > > [ 3.781233] device: 'ptyr3': device_add
> > > [ 3.785182] device: 'ptyr4': device_add
> > > [ 3.789136] device: 'ptyr5': device_add
> > > [ 3.793066] device: 'ptyr6': device_add
> > > [ 3.796995] device: 'ptyr7': device_add
> > > [ 3.800928] device: 'ptyr8': device_add
> > > [ 3.804864] device: 'ptyr9': device_add
> > > [ 3.808812] device: 'ptyra': device_add
> > > [ 3.812777] device: 'ptyrb': device_add
> > > [ 3.816722] device: 'ptyrc': device_add
> > > [ 3.820706] device: 'ptyrd': device_add
> > > [ 3.824652] device: 'ptyre': device_add
> > > [ 3.828611] device: 'ptyrf': device_add
> > > [ 3.832535] device: 'ptys0': device_add
> > > [ 3.836481] device: 'ptys1': device_add
> > > [ 3.840415] device: 'ptys2': device_add
> > > [ 3.844333] device: 'ptys3': device_add
> > > [ 3.848259] device: 'ptys4': device_add
> > > [ 3.852204] device: 'ptys5': device_add
> > > [ 3.856163] device: 'ptys6': device_add
> > > [ 3.860135] device: 'ptys7': device_add
> > > [ 3.864072] device: 'ptys8': device_add
> > > [ 3.868010] device: 'ptys9': device_add
> > > [ 3.871952] device: 'ptysa': device_add
> > > [ 3.875902] device: 'ptysb': device_add
> > > [ 3.879847] device: 'ptysc': device_add
> > > [ 3.883786] device: 'ptysd': device_add
> > > [ 3.887694] device: 'ptyse': device_add
> > > [ 3.891662] device: 'ptysf': device_add
> > > [ 3.895597] device: 'ptyt0': device_add
> > > [ 3.899535] device: 'ptyt1': device_add
> > > [ 3.903460] device: 'ptyt2': device_add
> > > [ 3.907400] device: 'ptyt3': device_add
> > > [ 3.911346] device: 'ptyt4': device_add
> > > [ 3.915279] device: 'ptyt5': device_add
> > > [ 3.919234] device: 'ptyt6': device_add
> > > [ 3.923180] device: 'ptyt7': device_add
> > > [ 3.927147] device: 'ptyt8': device_add
> > > [ 3.931082] device: 'ptyt9': device_add
> > > [ 3.935026] device: 'ptyta': device_add
> > > [ 3.938995] device: 'ptytb': device_add
> > > [ 3.942943] device: 'ptytc': device_add
> > > [ 3.946901] device: 'ptytd': device_add
> > > [ 3.950845] device: 'ptyte': device_add
> > > [ 3.954804] device: 'ptytf': device_add
> > > [ 3.958735] device: 'ptyu0': device_add
> > > [ 3.962688] device: 'ptyu1': device_add
> > > [ 3.966649] device: 'ptyu2': device_add
> > > [ 3.970599] device: 'ptyu3': device_add
> > > [ 3.974555] device: 'ptyu4': device_add
> > > [ 3.978499] device: 'ptyu5': device_add
> > > [ 3.982443] device: 'ptyu6': device_add
> > > [ 3.986400] device: 'ptyu7': device_add
> > > [ 3.990349] device: 'ptyu8': device_add
> > > [ 3.994303] device: 'ptyu9': device_add
> > > [ 3.998233] device: 'ptyua': device_add
> > > [ 4.002192] device: 'ptyub': device_add
> > > [ 4.006142] device: 'ptyuc': device_add
> > > [ 4.010090] device: 'ptyud': device_add
> > > [ 4.014015] device: 'ptyue': device_add
> > > [ 4.017962] device: 'ptyuf': device_add
> > > [ 4.021906] device: 'ptyv0': device_add
> > > [ 4.025858] device: 'ptyv1': device_add
> > > [ 4.029803] device: 'ptyv2': device_add
> > > [ 4.033787] device: 'ptyv3': device_add
> > > [ 4.037727] device: 'ptyv4': device_add
> > > [ 4.041662] device: 'ptyv5': device_add
> > > [ 4.045613] device: 'ptyv6': device_add
> > > [ 4.049571] device: 'ptyv7': device_add
> > > [ 4.053526] device: 'ptyv8': device_add
> > > [ 4.057476] device: 'ptyv9': device_add
> > > [ 4.061442] device: 'ptyva': device_add
> > > [ 4.065389] device: 'ptyvb': device_add
> > > [ 4.069355] device: 'ptyvc': device_add
> > > [ 4.073298] device: 'ptyvd': device_add
> > > [ 4.077242] device: 'ptyve': device_add
> > > [ 4.081191] device: 'ptyvf': device_add
> > > [ 4.085119] device: 'ptyw0': device_add
> > > [ 4.089070] device: 'ptyw1': device_add
> > > [ 4.092998] device: 'ptyw2': device_add
> > > [ 4.096949] device: 'ptyw3': device_add
> > > [ 4.100895] device: 'ptyw4': device_add
> > > [ 4.104839] device: 'ptyw5': device_add
> > > [ 4.108796] device: 'ptyw6': device_add
> > > [ 4.112733] device: 'ptyw7': device_add
> > > [ 4.116668] device: 'ptyw8': device_add
> > > [ 4.120628] device: 'ptyw9': device_add
> > > [ 4.124595] device: 'ptywa': device_add
> > > [ 4.128545] device: 'ptywb': device_add
> > > [ 4.132509] device: 'ptywc': device_add
> > > [ 4.136443] device: 'ptywd': device_add
> > > [ 4.140389] device: 'ptywe': device_add
> > > [ 4.144339] device: 'ptywf': device_add
> > > [ 4.148275] device: 'ptyx0': device_add
> > > [ 4.152190] device: 'ptyx1': device_add
> > > [ 4.156137] device: 'ptyx2': device_add
> > > [ 4.160067] device: 'ptyx3': device_add
> > > [ 4.164021] device: 'ptyx4': device_add
> > > [ 4.167974] device: 'ptyx5': device_add
> > > [ 4.171922] device: 'ptyx6': device_add
> > > [ 4.175872] device: 'ptyx7': device_add
> > > [ 4.179799] device: 'ptyx8': device_add
> > > [ 4.183734] device: 'ptyx9': device_add
> > > [ 4.187689] device: 'ptyxa': device_add
> > > [ 4.191647] device: 'ptyxb': device_add
> > > [ 4.195571] device: 'ptyxc': device_add
> > > [ 4.199544] device: 'ptyxd': device_add
> > > [ 4.203478] device: 'ptyxe': device_add
> > > [ 4.207429] device: 'ptyxf': device_add
> > > [ 4.211355] device: 'ptyy0': device_add
> > > [ 4.215318] device: 'ptyy1': device_add
> > > [ 4.219271] device: 'ptyy2': device_add
> > > [ 4.223214] device: 'ptyy3': device_add
> > > [ 4.227177] device: 'ptyy4': device_add
> > > [ 4.231143] device: 'ptyy5': device_add
> > > [ 4.235095] device: 'ptyy6': device_add
> > > [ 4.239049] device: 'ptyy7': device_add
> > > [ 4.243018] device: 'ptyy8': device_add
> > > [ 4.246968] device: 'ptyy9': device_add
> > > [ 4.250913] device: 'ptyya': device_add
> > > [ 4.254854] device: 'ptyyb': device_add
> > > [ 4.258815] device: 'ptyyc': device_add
> > > [ 4.262759] device: 'ptyyd': device_add
> > > [ 4.266696] device: 'ptyye': device_add
> > > [ 4.270640] device: 'ptyyf': device_add
> > > [ 4.274590] device: 'ptyz0': device_add
> > > [ 4.278527] device: 'ptyz1': device_add
> > > [ 4.282472] device: 'ptyz2': device_add
> > > [ 4.286447] device: 'ptyz3': device_add
> > > [ 4.290379] device: 'ptyz4': device_add
> > > [ 4.294315] device: 'ptyz5': device_add
> > > [ 4.298265] device: 'ptyz6': device_add
> > > [ 4.302194] device: 'ptyz7': device_add
> > > [ 4.306141] device: 'ptyz8': device_add
> > > [ 4.310113] device: 'ptyz9': device_add
> > > [ 4.314041] device: 'ptyza': device_add
> > > [ 4.317993] device: 'ptyzb': device_add
> > > [ 4.321939] device: 'ptyzc': device_add
> > > [ 4.325898] device: 'ptyzd': device_add
> > > [ 4.329876] device: 'ptyze': device_add
> > > [ 4.333819] device: 'ptyzf': device_add
> > > [ 4.337762] device: 'ptya0': device_add
> > > [ 4.341721] device: 'ptya1': device_add
> > > [ 4.345655] device: 'ptya2': device_add
> > > [ 4.349605] device: 'ptya3': device_add
> > > [ 4.353555] device: 'ptya4': device_add
> > > [ 4.357509] device: 'ptya5': device_add
> > > [ 4.361472] device: 'ptya6': device_add
> > > [ 4.365404] device: 'ptya7': device_add
> > > [ 4.369337] device: 'ptya8': device_add
> > > [ 4.373275] device: 'ptya9': device_add
> > > [ 4.377237] device: 'ptyaa': device_add
> > > [ 4.381172] device: 'ptyab': device_add
> > > [ 4.385136] device: 'ptyac': device_add
> > > [ 4.389063] device: 'ptyad': device_add
> > > [ 4.392997] device: 'ptyae': device_add
> > > [ 4.396939] device: 'ptyaf': device_add
> > > [ 4.400900] device: 'ptyb0': device_add
> > > [ 4.404833] device: 'ptyb1': device_add
> > > [ 4.408792] device: 'ptyb2': device_add
> > > [ 4.412751] device: 'ptyb3': device_add
> > > [ 4.416712] device: 'ptyb4': device_add
> > > [ 4.420639] device: 'ptyb5': device_add
> > > [ 4.424581] device: 'ptyb6': device_add
> > > [ 4.428561] device: 'ptyb7': device_add
> > > [ 4.432524] device: 'ptyb8': device_add
> > > [ 4.436465] device: 'ptyb9': device_add
> > > [ 4.440417] device: 'ptyba': device_add
> > > [ 4.444356] device: 'ptybb': device_add
> > > [ 4.448308] device: 'ptybc': device_add
> > > [ 4.452234] device: 'ptybd': device_add
> > > [ 4.456159] device: 'ptybe': device_add
> > > [ 4.460108] device: 'ptybf': device_add
> > > [ 4.464038] device: 'ptyc0': device_add
> > > [ 4.467973] device: 'ptyc1': device_add
> > > [ 4.471930] device: 'ptyc2': device_add
> > > [ 4.475885] device: 'ptyc3': device_add
> > > [ 4.479820] device: 'ptyc4': device_add
> > > [ 4.483784] device: 'ptyc5': device_add
> > > [ 4.487731] device: 'ptyc6': device_add
> > > [ 4.491673] device: 'ptyc7': device_add
> > > [ 4.495627] device: 'ptyc8': device_add
> > > [ 4.499583] device: 'ptyc9': device_add
> > > [ 4.503533] device: 'ptyca': device_add
> > > [ 4.507485] device: 'ptycb': device_add
> > > [ 4.511412] device: 'ptycc': device_add
> > > [ 4.515382] device: 'ptycd': device_add
> > > [ 4.519324] device: 'ptyce': device_add
> > > [ 4.523252] device: 'ptycf': device_add
> > > [ 4.527191] device: 'ptyd0': device_add
> > > [ 4.531126] device: 'ptyd1': device_add
> > > [ 4.535065] device: 'ptyd2': device_add
> > > [ 4.539031] device: 'ptyd3': device_add
> > > [ 4.542987] device: 'ptyd4': device_add
> > > [ 4.546944] device: 'ptyd5': device_add
> > > [ 4.550892] device: 'ptyd6': device_add
> > > [ 4.554849] device: 'ptyd7': device_add
> > > [ 4.558809] device: 'ptyd8': device_add
> > > [ 4.562757] device: 'ptyd9': device_add
> > > [ 4.566709] device: 'ptyda': device_add
> > > [ 4.570643] device: 'ptydb': device_add
> > > [ 4.574568] device: 'ptydc': device_add
> > > [ 4.578508] device: 'ptydd': device_add
> > > [ 4.582458] device: 'ptyde': device_add
> > > [ 4.586394] device: 'ptydf': device_add
> > > [ 4.590345] device: 'ptye0': device_add
> > > [ 4.594279] device: 'ptye1': device_add
> > > [ 4.598208] device: 'ptye2': device_add
> > > [ 4.602142] device: 'ptye3': device_add
> > > [ 4.606085] device: 'ptye4': device_add
> > > [ 4.610027] device: 'ptye5': device_add
> > > [ 4.613964] device: 'ptye6': device_add
> > > [ 4.617920] device: 'ptye7': device_add
> > > [ 4.621871] device: 'ptye8': device_add
> > > [ 4.625811] device: 'ptye9': device_add
> > > [ 4.629767] device: 'ptyea': device_add
> > > [ 4.633728] device: 'ptyeb': device_add
> > > [ 4.637667] device: 'ptyec': device_add
> > > [ 4.641619] device: 'ptyed': device_add
> > > [ 4.645572] device: 'ptyee': device_add
> > > [ 4.649509] device: 'ptyef': device_add
> > > [ 4.653449] device: 'ttyp0': device_add
> > > [ 4.657388] device: 'ttyp1': device_add
> > > [ 4.661322] device: 'ttyp2': device_add
> > > [ 4.665253] device: 'ttyp3': device_add
> > > [ 4.669218] device: 'ttyp4': device_add
> > > [ 4.673170] device: 'ttyp5': device_add
> > > [ 4.677115] device: 'ttyp6': device_add
> > > [ 4.681082] device: 'ttyp7': device_add
> > > [ 4.685022] device: 'ttyp8': device_add
> > > [ 4.688976] device: 'ttyp9': device_add
> > > [ 4.692915] device: 'ttypa': device_add
> > > [ 4.696850] device: 'ttypb': device_add
> > > [ 4.700809] device: 'ttypc': device_add
> > > [ 4.704771] device: 'ttypd': device_add
> > > [ 4.708714] device: 'ttype': device_add
> > > [ 4.712662] device: 'ttypf': device_add
> > > [ 4.716633] device: 'ttyq0': device_add
> > > [ 4.720567] device: 'ttyq1': device_add
> > > [ 4.724504] device: 'ttyq2': device_add
> > > [ 4.728451] device: 'ttyq3': device_add
> > > [ 4.732403] device: 'ttyq4': device_add
> > > [ 4.736348] device: 'ttyq5': device_add
> > > [ 4.740291] device: 'ttyq6': device_add
> > > [ 4.744259] device: 'ttyq7': device_add
> > > [ 4.748204] device: 'ttyq8': device_add
> > > [ 4.752141] device: 'ttyq9': device_add
> > > [ 4.756094] device: 'ttyqa': device_add
> > > [ 4.760045] device: 'ttyqb': device_add
> > > [ 4.763989] device: 'ttyqc': device_add
> > > [ 4.767932] device: 'ttyqd': device_add
> > > [ 4.771874] device: 'ttyqe': device_add
> > > [ 4.775824] device: 'ttyqf': device_add
> > > [ 4.779770] device: 'ttyr0': device_add
> > > [ 4.783722] device: 'ttyr1': device_add
> > > [ 4.787674] device: 'ttyr2': device_add
> > > [ 4.791651] device: 'ttyr3': device_add
> > > [ 4.795606] device: 'ttyr4': device_add
> > > [ 4.799552] device: 'ttyr5': device_add
> > > [ 4.803504] device: 'ttyr6': device_add
> > > [ 4.807457] device: 'ttyr7': device_add
> > > [ 4.811412] device: 'ttyr8': device_add
> > > [ 4.815360] device: 'ttyr9': device_add
> > > [ 4.819290] device: 'ttyra': device_add
> > > [ 4.823231] device: 'ttyrb': device_add
> > > [ 4.827173] device: 'ttyrc': device_add
> > > [ 4.831106] device: 'ttyrd': device_add
> > > [ 4.835046] device: 'ttyre': device_add
> > > [ 4.838974] device: 'ttyrf': device_add
> > > [ 4.842916] device: 'ttys0': device_add
> > > [ 4.846852] device: 'ttys1': device_add
> > > [ 4.850792] device: 'ttys2': device_add
> > > [ 4.854756] device: 'ttys3': device_add
> > > [ 4.858717] device: 'ttys4': device_add
> > > [ 4.862670] device: 'ttys5': device_add
> > > [ 4.866612] device: 'ttys6': device_add
> > > [ 4.870587] device: 'ttys7': device_add
> > > [ 4.874522] device: 'ttys8': device_add
> > > [ 4.878458] device: 'ttys9': device_add
> > > [ 4.882426] device: 'ttysa': device_add
> > > [ 4.886369] device: 'ttysb': device_add
> > > [ 4.890338] device: 'ttysc': device_add
> > > [ 4.894289] device: 'ttysd': device_add
> > > [ 4.898238] device: 'ttyse': device_add
> > > [ 4.902193] device: 'ttysf': device_add
> > > [ 4.906135] device: 'ttyt0': device_add
> > > [ 4.910072] device: 'ttyt1': device_add
> > > [ 4.914011] device: 'ttyt2': device_add
> > > [ 4.917947] device: 'ttyt3': device_add
> > > [ 4.921891] device: 'ttyt4': device_add
> > > [ 4.925857] device: 'ttyt5': device_add
> > > [ 4.929803] device: 'ttyt6': device_add
> > > [ 4.933781] device: 'ttyt7': device_add
> > > [ 4.937720] device: 'ttyt8': device_add
> > > [ 4.941664] device: 'ttyt9': device_add
> > > [ 4.945633] device: 'ttyta': device_add
> > > [ 4.949584] device: 'ttytb': device_add
> > > [ 4.953542] device: 'ttytc': device_add
> > > [ 4.957491] device: 'ttytd': device_add
> > > [ 4.961416] device: 'ttyte': device_add
> > > [ 4.965351] device: 'ttytf': device_add
> > > [ 4.969278] device: 'ttyu0': device_add
> > > [ 4.973211] device: 'ttyu1': device_add
> > > [ 4.977183] device: 'ttyu2': device_add
> > > [ 4.981147] device: 'ttyu3': device_add
> > > [ 4.985083] device: 'ttyu4': device_add
> > > [ 4.989069] device: 'ttyu5': device_add
> > > [ 4.993004] device: 'ttyu6': device_add
> > > [ 4.996947] device: 'ttyu7': device_add
> > > [ 5.000915] device: 'ttyu8': device_add
> > > [ 5.004864] device: 'ttyu9': device_add
> > > [ 5.008809] device: 'ttyua': device_add
> > > [ 5.012787] device: 'ttyub': device_add
> > > [ 5.016741] device: 'ttyuc': device_add
> > > [ 5.020678] device: 'ttyud': device_add
> > > [ 5.024620] device: 'ttyue': device_add
> > > [ 5.028582] device: 'ttyuf': device_add
> > > [ 5.032538] device: 'ttyv0': device_add
> > > [ 5.036484] device: 'ttyv1': device_add
> > > [ 5.040424] device: 'ttyv2': device_add
> > > [ 5.044378] device: 'ttyv3': device_add
> > > [ 5.048338] device: 'ttyv4': device_add
> > > [ 5.052285] device: 'ttyv5': device_add
> > > [ 5.056236] device: 'ttyv6': device_add
> > > [ 5.060177] device: 'ttyv7': device_add
> > > [ 5.064125] device: 'ttyv8': device_add
> > > [ 5.068097] device: 'ttyv9': device_add
> > > [ 5.072028] device: 'ttyva': device_add
> > > [ 5.075978] device: 'ttyvb': device_add
> > > [ 5.079944] device: 'ttyvc': device_add
> > > [ 5.083881] device: 'ttyvd': device_add
> > > [ 5.087833] device: 'ttyve': device_add
> > > [ 5.091776] device: 'ttyvf': device_add
> > > [ 5.095722] device: 'ttyw0': device_add
> > > [ 5.099672] device: 'ttyw1': device_add
> > > [ 5.103618] device: 'ttyw2': device_add
> > > [ 5.107568] device: 'ttyw3': device_add
> > > [ 5.111495] device: 'ttyw4': device_add
> > > [ 5.115441] device: 'ttyw5': device_add
> > > [ 5.119380] device: 'ttyw6': device_add
> > > [ 5.123368] device: 'ttyw7': device_add
> > > [ 5.127314] device: 'ttyw8': device_add
> > > [ 5.131258] device: 'ttyw9': device_add
> > > [ 5.135208] device: 'ttywa': device_add
> > > [ 5.139177] device: 'ttywb': device_add
> > > [ 5.143159] device: 'ttywc': device_add
> > > [ 5.147113] device: 'ttywd': device_add
> > > [ 5.151059] device: 'ttywe': device_add
> > > [ 5.155032] device: 'ttywf': device_add
> > > [ 5.158957] device: 'ttyx0': device_add
> > > [ 5.162901] device: 'ttyx1': device_add
> > > [ 5.166870] device: 'ttyx2': device_add
> > > [ 5.170811] device: 'ttyx3': device_add
> > > [ 5.174752] device: 'ttyx4': device_add
> > > [ 5.178712] device: 'ttyx5': device_add
> > > [ 5.182646] device: 'ttyx6': device_add
> > > [ 5.186615] device: 'ttyx7': device_add
> > > [ 5.190556] device: 'ttyx8': device_add
> > > [ 5.194491] device: 'ttyx9': device_add
> > > [ 5.198418] device: 'ttyxa': device_add
> > > [ 5.202377] device: 'ttyxb': device_add
> > > [ 5.206303] device: 'ttyxc': device_add
> > > [ 5.210258] device: 'ttyxd': device_add
> > > [ 5.214210] device: 'ttyxe': device_add
> > > [ 5.218154] device: 'ttyxf': device_add
> > > [ 5.222094] device: 'ttyy0': device_add
> > > [ 5.226041] device: 'ttyy1': device_add
> > > [ 5.229985] device: 'ttyy2': device_add
> > > [ 5.233920] device: 'ttyy3': device_add
> > > [ 5.237883] device: 'ttyy4': device_add
> > > [ 5.241820] device: 'ttyy5': device_add
> > > [ 5.245767] device: 'ttyy6': device_add
> > > [ 5.249703] device: 'ttyy7': device_add
> > > [ 5.253672] device: 'ttyy8': device_add
> > > [ 5.257606] device: 'ttyy9': device_add
> > > [ 5.261556] device: 'ttyya': device_add
> > > [ 5.265496] device: 'ttyyb': device_add
> > > [ 5.269421] device: 'ttyyc': device_add
> > > [ 5.273357] device: 'ttyyd': device_add
> > > [ 5.277313] device: 'ttyye': device_add
> > > [ 5.281273] device: 'ttyyf': device_add
> > > [ 5.285209] device: 'ttyz0': device_add
> > > [ 5.289177] device: 'ttyz1': device_add
> > > [ 5.293118] device: 'ttyz2': device_add
> > > [ 5.297071] device: 'ttyz3': device_add
> > > [ 5.301035] device: 'ttyz4': device_add
> > > [ 5.304968] device: 'ttyz5': device_add
> > > [ 5.308938] device: 'ttyz6': device_add
> > > [ 5.312877] device: 'ttyz7': device_add
> > > [ 5.316807] device: 'ttyz8': device_add
> > > [ 5.320767] device: 'ttyz9': device_add
> > > [ 5.324715] device: 'ttyza': device_add
> > > [ 5.328648] device: 'ttyzb': device_add
> > > [ 5.332593] device: 'ttyzc': device_add
> > > [ 5.336559] device: 'ttyzd': device_add
> > > [ 5.340508] device: 'ttyze': device_add
> > > [ 5.344460] device: 'ttyzf': device_add
> > > [ 5.348397] device: 'ttya0': device_add
> > > [ 5.352361] device: 'ttya1': device_add
> > > [ 5.356289] device: 'ttya2': device_add
> > > [ 5.360226] device: 'ttya3': device_add
> > > [ 5.364181] device: 'ttya4': device_add
> > > [ 5.368122] device: 'ttya5': device_add
> > > [ 5.372055] device: 'ttya6': device_add
> > > [ 5.375994] device: 'ttya7': device_add
> > > [ 5.379934] device: 'ttya8': device_add
> > > [ 5.383862] device: 'ttya9': device_add
> > > [ 5.387825] device: 'ttyaa': device_add
> > > [ 5.391770] device: 'ttyab': device_add
> > > [ 5.395691] device: 'ttyac': device_add
> > > [ 5.399624] device: 'ttyad': device_add
> > > [ 5.403557] device: 'ttyae': device_add
> > > [ 5.407528] device: 'ttyaf': device_add
> > > [ 5.411479] device: 'ttyb0': device_add
> > > [ 5.415433] device: 'ttyb1': device_add
> > > [ 5.419376] device: 'ttyb2': device_add
> > > [ 5.423338] device: 'ttyb3': device_add
> > > [ 5.427272] device: 'ttyb4': device_add
> > > [ 5.431235] device: 'ttyb5': device_add
> > > [ 5.435164] device: 'ttyb6': device_add
> > > [ 5.439112] device: 'ttyb7': device_add
> > > [ 5.443070] device: 'ttyb8': device_add
> > > [ 5.447003] device: 'ttyb9': device_add
> > > [ 5.450958] device: 'ttyba': device_add
> > > [ 5.454917] device: 'ttybb': device_add
> > > [ 5.458849] device: 'ttybc': device_add
> > > [ 5.462812] device: 'ttybd': device_add
> > > [ 5.466797] device: 'ttybe': device_add
> > > [ 5.470772] device: 'ttybf': device_add
> > > [ 5.474736] device: 'ttyc0': device_add
> > > [ 5.478669] device: 'ttyc1': device_add
> > > [ 5.482610] device: 'ttyc2': device_add
> > > [ 5.486582] device: 'ttyc3': device_add
> > > [ 5.490544] device: 'ttyc4': device_add
> > > [ 5.494531] device: 'ttyc5': device_add
> > > [ 5.498478] device: 'ttyc6': device_add
> > > [ 5.502414] device: 'ttyc7': device_add
> > > [ 5.506395] device: 'ttyc8': device_add
> > > [ 5.510353] device: 'ttyc9': device_add
> > > [ 5.514276] device: 'ttyca': device_add
> > > [ 5.518243] device: 'ttycb': device_add
> > > [ 5.522186] device: 'ttycc': device_add
> > > [ 5.526144] device: 'ttycd': device_add
> > > [ 5.530090] device: 'ttyce': device_add
> > > [ 5.534039] device: 'ttycf': device_add
> > > [ 5.537997] device: 'ttyd0': device_add
> > > [ 5.541938] device: 'ttyd1': device_add
> > > [ 5.545884] device: 'ttyd2': device_add
> > > [ 5.549840] device: 'ttyd3': device_add
> > > [ 5.553800] device: 'ttyd4': device_add
> > > [ 5.557743] device: 'ttyd5': device_add
> > > [ 5.561712] device: 'ttyd6': device_add
> > > [ 5.565660] device: 'ttyd7': device_add
> > > [ 5.569604] device: 'ttyd8': device_add
> > > [ 5.573600] device: 'ttyd9': device_add
> > > [ 5.577562] device: 'ttyda': device_add
> > > [ 5.581547] device: 'ttydb': device_add
> > > [ 5.585505] device: 'ttydc': device_add
> > > [ 5.589465] device: 'ttydd': device_add
> > > [ 5.593419] device: 'ttyde': device_add
> > > [ 5.597372] device: 'ttydf': device_add
> > > [ 5.601311] device: 'ttye0': device_add
> > > [ 5.605279] device: 'ttye1': device_add
> > > [ 5.609259] device: 'ttye2': device_add
> > > [ 5.613199] device: 'ttye3': device_add
> > > [ 5.617211] device: 'ttye4': device_add
> > > [ 5.621153] device: 'ttye5': device_add
> > > [ 5.625099] device: 'ttye6': device_add
> > > [ 5.629045] device: 'ttye7': device_add
> > > [ 5.632989] device: 'ttye8': device_add
> > > [ 5.636939] device: 'ttye9': device_add
> > > [ 5.640894] device: 'ttyea': device_add
> > > [ 5.644851] device: 'ttyeb': device_add
> > > [ 5.648779] device: 'ttyec': device_add
> > > [ 5.652724] device: 'ttyed': device_add
> > > [ 5.656657] device: 'ttyee': device_add
> > > [ 5.660609] device: 'ttyef': device_add
> > > [ 5.664553] device: 'ptmx': device_add
> > > [ 5.668457] Serial: 8250/16550 driver, 4 ports, IRQ sharing disabled
> > > [ 5.674981] device: 'serial8250': device_add
> > > [ 5.679377] device: 'ttyS0': device_add
> > > [ 5.683422] device: 'ttyS1': device_add
> > > [ 5.687396] device: 'ttyS2': device_add
> > > [ 5.691400] device: 'ttyS3': device_add
> > > [ 5.695929] platform 840c000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.703851] platform 840d000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.711737] platform 840e000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.719789] platform 840c000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.727676] platform 840d000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.735536] platform 840e000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.743685] device: 'hw_random': device_add
> > > [ 5.748211] iproc-rng200 8402000.rng: hwrng registered
> > > [ 5.753969] device: '1:0': device_add
> > > [ 5.757799] device: 'ram0': device_add
> > > [ 5.761939] device: '1:1': device_add
> > > [ 5.765724] device: 'ram1': device_add
> > > [ 5.769767] device: '1:2': device_add
> > > [ 5.773566] device: 'ram2': device_add
> > > [ 5.777569] device: '1:3': device_add
> > > [ 5.781334] device: 'ram3': device_add
> > > [ 5.785340] device: '1:4': device_add
> > > [ 5.789099] device: 'ram4': device_add
> > > [ 5.793117] device: '1:5': device_add
> > > [ 5.796893] device: 'ram5': device_add
> > > [ 5.800888] device: '1:6': device_add
> > > [ 5.804674] device: 'ram6': device_add
> > > [ 5.808693] device: '1:7': device_add
> > > [ 5.812474] device: 'ram7': device_add
> > > [ 5.816504] device: '1:8': device_add
> > > [ 5.820256] device: 'ram8': device_add
> > > [ 5.824259] device: '1:9': device_add
> > > [ 5.828039] device: 'ram9': device_add
> > > [ 5.832072] device: '1:10': device_add
> > > [ 5.835929] device: 'ram10': device_add
> > > [ 5.840031] device: '1:11': device_add
> > > [ 5.843929] device: 'ram11': device_add
> > > [ 5.848033] device: '1:12': device_add
> > > [ 5.851908] device: 'ram12': device_add
> > > [ 5.856015] device: '1:13': device_add
> > > [ 5.859889] device: 'ram13': device_add
> > > [ 5.863998] device: '1:14': device_add
> > > [ 5.867855] device: 'ram14': device_add
> > > [ 5.871974] device: '1:15': device_add
> > > [ 5.875841] device: 'ram15': device_add
> > > [ 5.879976] brd: module loaded
> > > [ 5.883101] device: 'loop-control': device_add
> > > [ 5.887960] device: '7:0': device_add
> > > [ 5.891760] device: 'loop0': device_add
> > > [ 5.896215] device: '7:1': device_add
> > > [ 5.900005] device: 'loop1': device_add
> > > [ 5.904458] device: '7:2': device_add
> > > [ 5.908227] device: 'loop2': device_add
> > > [ 5.912686] device: '7:3': device_add
> > > [ 5.916467] device: 'loop3': device_add
> > > [ 5.920970] device: '7:4': device_add
> > > [ 5.924767] device: 'loop4': device_add
> > > [ 5.929216] device: '7:5': device_add
> > > [ 5.933017] device: 'loop5': device_add
> > > [ 5.937483] device: '7:6': device_add
> > > [ 5.941271] device: 'loop6': device_add
> > > [ 5.945724] device: '7:7': device_add
> > > [ 5.949513] device: 'loop7': device_add
> > > [ 5.953718] loop: module loaded
> > > [ 5.957414] platform 8b0a000.sata: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.965310] device: 'mtd-0': device_add
> > > [ 5.970044] platform 84b2120.qspi: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 5.977854] brcmstb_qspi 8418000.mspi: using mspi mode
> > > [ 5.983168] device: 'spi0': device_add
> > > [ 5.988043] device: 'Fixed MDIO bus.0': device_add
> > > [ 5.993124] device: 'fixed-0': device_add
> > > [ 5.997362] libphy: Fixed MDIO Bus: probed
> > > [ 6.002042] platform 8f00000.ethernet: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.010377] e1000e: Intel(R) PRO/1000 Network Driver
> > > [ 6.015435] e1000e: Copyright(c) 1999 - 2015 Intel Corporation.
> > > [ 6.021476] pegasus: v0.9.3 (2013/04/25), Pegasus/Pegasus II USB
> > Ethernet driver
> > > [ 6.029039] usbcore: registered new interface driver pegasus
> > > [ 6.034804] usbcore: registered new interface driver r8152
> > > [ 6.040405] usbcore: registered new interface driver asix
> > > [ 6.045918] usbcore: registered new interface driver ax88179_178a
> > > [ 6.052155] usbcore: registered new interface driver cdc_ether
> > > [ 6.058146] usbcore: registered new interface driver cdc_ncm
> > > [ 6.063989] device: 'usbmon0': device_add
> > > [ 6.068148] ehci_hcd: USB 2.0 'Enhanced' Host Controller (EHCI) Driver
> > > [ 6.074791] ehci-pci: EHCI PCI platform driver
> > > [ 6.079354] ohci_hcd: USB 1.1 'Open' Host Controller (OHCI) Driver
> > > [ 6.085630] ohci-pci: OHCI PCI platform driver
> > > [ 6.090220] usbcore: registered new interface driver cdc_acm
> > > [ 6.095991] cdc_acm: USB Abstract Control Model driver for USB modems
> > and ISDN adapters
> > > [ 6.104131] usbcore: registered new interface driver usb-storage
> > > [ 6.110544] device: 'uinput': device_add
> > > [ 6.114617] i2c /dev entries driver
> > > [ 6.118853] brcmstb_thermal a581500.thermal: registered AVS TMON
> > of-sensor driver
> > > [ 6.126867] device: 'watchdog': device_add
> > > [ 6.131143] device: 'watchdog0': device_add
> > > [ 6.135483] bcm7038-wdt 840a628.watchdog: Registered BCM7038 Watchdog
> > > [ 6.142519] sdhci: Secure Digital Host Controller Interface driver
> > > [ 6.148844] sdhci: Copyright(c) Pierre Ossman
> > > [ 6.153291] sdhci-pltfm: SDHCI platform and OF driver helper
> > > [ 6.159169] platform 84b0000.sdhci: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.159415] hid: raw HID events driver (C) Jiri Kosina
> > > [ 6.172246] device: 'uhid': device_add
> > > [ 6.176268] usbcore: registered new interface driver usbhid
> > > [ 6.181993] usbhid: USB HID core driver
> > > [ 6.599764] brcmstb-dpfe 9932000.dpfe-cpu: registered with API v3.
> > > [ 6.606251] device: 'timer': device_add
> > > [ 6.610336] device: 'snd-soc-dummy': device_add
> > > [ 6.615681] NET: Registered protocol family 10
> > > [ 6.620674] Segment Routing with IPv6
> > > [ 6.624441] sit: IPv6, IPv4 and MPLS over IPv4 tunneling driver
> > > [ 6.630479] device: 'sit0': device_add
> > > [ 6.634623] NET: Registered protocol family 17
> > > [ 6.639197] bridge: filtering via arp/ip/ip6tables is no longer
> > available by default. Update your scripts to load br_netfilter if you ne
> > > ed this.
> > > [ 6.652522] 8021q: 802.1Q VLAN Support v1.8
> > > [ 6.656809] Key type dns_resolver registered
> > > [ 6.661246] Registering SWP/SWPB emulation handler
> > > [ 6.666163] device: 'cpu_dma_latency': device_add
> > > [ 6.671063] Loading compiled-in X.509 certificates
> > > [ 6.678526] devices_kset: Moving brcm_scmi@0 to end of list
> > > [ 6.684262] devices_kset: Moving 840c000.serial to end of list
> > > [ 6.690186] devices_kset: Moving 840d000.serial to end of list
> > > [ 6.696153] devices_kset: Moving 840e000.serial to end of list
> > > [ 6.702102] devices_kset: Moving 8d0f200.usb-phy to end of list
> > > [ 6.708111] devices_kset: Moving 8d00000.xhci_v2 to end of list
> > > [ 6.714129] devices_kset: Moving 8d00000.xhci_v2 to end of list
> > > [ 6.720155] devices_kset: Moving 8f00000.ethernet to end of list
> > > [ 6.726252] devices_kset: Moving 8b0a000.sata to end of list
> > > [ 6.732018] devices_kset: Moving 84b0000.sdhci to end of list
> > > [ 6.737888] devices_kset: Moving 84b2120.qspi to end of list
> > > [ 6.743637] devices_kset: Moving 8b20000.pcie to end of list
> > > [ 6.749451] platform brcm_scmi@0: probe deferral - supplier
> > brcm_scmi_mailbox@0 not ready
> > > [ 6.757750] devices_kset: Moving 8b20000.pcie to end of list
> > > [ 6.763541] platform 8b20000.pcie: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.771258] devices_kset: Moving 840c000.serial to end of list
> > > [ 6.777231] platform 840c000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.785157] devices_kset: Moving 840d000.serial to end of list
> > > [ 6.791114] platform 840d000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.799016] devices_kset: Moving 840e000.serial to end of list
> > > [ 6.804973] platform 840e000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.812846] devices_kset: Moving 8b0a000.sata to end of list
> > > [ 6.818674] platform 8b0a000.sata: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.826399] devices_kset: Moving 84b2120.qspi to end of list
> > > [ 6.832209] platform 84b2120.qspi: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.839891] devices_kset: Moving 8f00000.ethernet to end of list
> > > [ 6.846066] platform 8f00000.ethernet: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.854131] devices_kset: Moving 84b0000.sdhci to end of list
> > > [ 6.860167] platform 84b0000.sdhci: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.860776] devices_kset: Moving brcm_scmi@0 to end of list
> > > [ 6.873679] devices_kset: Moving 840c000.serial to end of list
> > > [ 6.879584] devices_kset: Moving 840d000.serial to end of list
> > > [ 6.885551] devices_kset: Moving 840e000.serial to end of list
> > > [ 6.891491] devices_kset: Moving 8d0f200.usb-phy to end of list
> > > [ 6.897526] devices_kset: Moving 8d00000.xhci_v2 to end of list
> > > [ 6.903579] devices_kset: Moving 8d00000.xhci_v2 to end of list
> > > [ 6.909657] devices_kset: Moving 8f00000.ethernet to end of list
> > > [ 6.915797] devices_kset: Moving 8b0a000.sata to end of list
> > > [ 6.921580] devices_kset: Moving 84b0000.sdhci to end of list
> > > [ 6.927461] devices_kset: Moving 84b2120.qspi to end of list
> > > [ 6.933236] devices_kset: Moving 8b20000.pcie to end of list
> > > [ 6.939028] platform brcm_scmi@0: probe deferral - supplier
> > brcm_scmi_mailbox@0 not ready
> > > [ 6.947346] devices_kset: Moving 8b20000.pcie to end of list
> > > [ 6.953117] platform 8b20000.pcie: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.960869] devices_kset: Moving 840c000.serial to end of list
> > > [ 6.966877] platform 840c000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.974751] devices_kset: Moving 840d000.serial to end of list
> > > [ 6.980754] platform 840d000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 6.988645] devices_kset: Moving 840e000.serial to end of list
> > > [ 6.994627] platform 840e000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 7.002535] devices_kset: Moving 8b0a000.sata to end of list
> > > [ 7.008306] platform 8b0a000.sata: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 7.016040] devices_kset: Moving 84b2120.qspi to end of list
> > > [ 7.021854] platform 84b2120.qspi: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 7.029536] devices_kset: Moving 8f00000.ethernet to end of list
> > > [ 7.035698] platform 8f00000.ethernet: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 7.043728] devices_kset: Moving 84b0000.sdhci to end of list
> > > [ 7.049741] device: 'ubi_ctrl': device_add
> > > [ 7.049773] platform 84b0000.sdhci: probe deferral - supplier
> > brcm_scmi@0 not ready
> > > [ 7.054352] cfg80211: Loading compiled-in X.509 certificates for
> > regulatory database
> > > [ 7.076406] cfg80211: Loaded X.509 cert 'sforshee: 00b28ddf47aef9cea7'
> > > [ 7.083122] ALSA device list:
> > > [ 7.086142] No soundcards found.
> > > [ 7.089708] platform regulatory.0: Direct firmware load for
> > regulatory.db failed with error -2
> > > [ 7.098506] cfg80211: failed to load regulatory.db
> > > [ 7.115369] Freeing unused kernel memory: 14336K
> > > [ 7.129918] Run /init as init process
> > > [ 7.133673] with arguments:
> > > [ 7.136688] /init
> > > [ 7.139004] with environment:
> > > [ 7.142204] HOME=/
> > > [ 7.144622] TERM=linux
> > > [ 7.147370] dyndbg=file drivers/base/core.c +p
> > > [ 7.256086] random: crng init done
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> >
> >
> >
On Tue, Apr 27, 2021 at 03:11:16PM +0100, Cristian Marussi wrote:
> On Tue, Apr 27, 2021 at 09:33:31AM -0400, Jim Quinlan wrote:
[...]
> > >
> > I believe that the brcmstb-mbox node is in our DT for backwards
> > compatibility with our older Linux only. Note that we use the compatible
> > string '"arm,scmi-smc", "arm,scmi"'; the former chooses SMC transport and
> > ignores custom mailboxes such as brcmstb-mbox.
> >
>
> Right..so it is even more wrong that it is waiting for the mailboxes...but
> looking at the DT:
>
> brcm_scmi_mailbox@0 {
> #mbox-cells = <0x01>;
> compatible = "brcm,brcmstb-mbox";
> status = "disabled";
> linux,phandle = <0x04>;
> phandle = <0x04>;
> };
>
> brcm_scmi@0 {
> compatible = "arm,scmi-smc\0arm,scmi";
> mboxes = <0x04 0x00 0x04 0x01>;
> mbox-names = "tx\0rx";
> shmem = <0x05>;
> status = "disabled";
> arm,smc-id = <0x83000400>;
> interrupt-names = "a2p";
> #address-cells = <0x01>;
> #size-cells = <0x00>;
>
> it seems to me that even though you declare an SMC based transport (and in fact
> you define the smc-id too) you also still define mboxes (as a fallback I suppose)
> referring to the brcm_scmi_mailbox phandle, and while this is ignored by the SCMI
> driver (because you have selected a compatible SMC transport) I imagine this dep
> is picked up by fw_devlink which in fact says:
>
> > [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
>
> and stalls waiting for it. (but I'm not really familiar on how fw_devlink
> internals works really...so I maybe off in these regards)
>
I was about to mention/ask the same when I saw Jim's reply. I see you have
already asked that. Couple of my opinions based on my very limited knowledge
on fw_devlink and how it works.
1. Since we have different compatible for SMC and mailbox, I am not sure
if it correct to leave mailbox information in scmi node. Once we have
proper yaml scheme, we must flag that error IMO.
2. IIUC, the fw_devlink might use information from DT to establish the
dependency and having mailbox information in this context may be
considered wrong as there is no dependency if it is using SMC.
--
Regards,
Sudeep
On 4/27/2021 8:10 AM, Sudeep Holla wrote:
> On Tue, Apr 27, 2021 at 03:11:16PM +0100, Cristian Marussi wrote:
>> On Tue, Apr 27, 2021 at 09:33:31AM -0400, Jim Quinlan wrote:
> [...]
>>>>
>>> I believe that the brcmstb-mbox node is in our DT for backwards
>>> compatibility with our older Linux only. Note that we use the compatible
>>> string '"arm,scmi-smc", "arm,scmi"'; the former chooses SMC transport and
>>> ignores custom mailboxes such as brcmstb-mbox.
>>>
>>
>> Right..so it is even more wrong that it is waiting for the mailboxes...but
>> looking at the DT:
>>
>> brcm_scmi_mailbox@0 {
>> #mbox-cells = <0x01>;
>> compatible = "brcm,brcmstb-mbox";
>> status = "disabled";
>> linux,phandle = <0x04>;
>> phandle = <0x04>;
>> };
>>
>> brcm_scmi@0 {
>> compatible = "arm,scmi-smc\0arm,scmi";
>> mboxes = <0x04 0x00 0x04 0x01>;
>> mbox-names = "tx\0rx";
>> shmem = <0x05>;
>> status = "disabled";
>> arm,smc-id = <0x83000400>;
>> interrupt-names = "a2p";
>> #address-cells = <0x01>;
>> #size-cells = <0x00>;
>>
>> it seems to me that even though you declare an SMC based transport (and in fact
>> you define the smc-id too) you also still define mboxes (as a fallback I suppose)
>> referring to the brcm_scmi_mailbox phandle, and while this is ignored by the SCMI
>> driver (because you have selected a compatible SMC transport) I imagine this dep
>> is picked up by fw_devlink which in fact says:
>>
>>> [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
>>
>> and stalls waiting for it. (but I'm not really familiar on how fw_devlink
>> internals works really...so I maybe off in these regards)
>>
>
> I was about to mention/ask the same when I saw Jim's reply. I see you have
> already asked that. Couple of my opinions based on my very limited knowledge
> on fw_devlink and how it works.
>
> 1. Since we have different compatible for SMC and mailbox, I am not sure
> if it correct to leave mailbox information in scmi node. Once we have
> proper yaml scheme, we must flag that error IMO.
This is a self inflicted problem that we have in that the bootloader
provides a Device Tree to the kernel which is massaged in different ways
and intends to stay backwards compatible as much as possible. And indeed
after removing the 'mboxes' property gets us going with fw_devlink=on.
>
> 2. IIUC, the fw_devlink might use information from DT to establish the
> dependency and having mailbox information in this context may be
> considered wrong as there is no dependency if it is using SMC.
Right, unfortunately, short of having some special casing for SCMI and
checking that if we have both an "arm,smc-id" and "mboxes" phandle we
should prefer the former, there is not probably much that can be done
here. Do we want to do that?
Thank you all for looking into this!
--
Florian
On Tue, Apr 27, 2021 at 8:10 AM Sudeep Holla <[email protected]> wrote:
>
> On Tue, Apr 27, 2021 at 03:11:16PM +0100, Cristian Marussi wrote:
> > On Tue, Apr 27, 2021 at 09:33:31AM -0400, Jim Quinlan wrote:
> [...]
> > > >
> > > I believe that the brcmstb-mbox node is in our DT for backwards
> > > compatibility with our older Linux only. Note that we use the compatible
> > > string '"arm,scmi-smc", "arm,scmi"'; the former chooses SMC transport and
> > > ignores custom mailboxes such as brcmstb-mbox.
> > >
> >
> > Right..so it is even more wrong that it is waiting for the mailboxes...but
> > looking at the DT:
> >
> > brcm_scmi_mailbox@0 {
> > #mbox-cells = <0x01>;
> > compatible = "brcm,brcmstb-mbox";
> > status = "disabled";
> > linux,phandle = <0x04>;
> > phandle = <0x04>;
> > };
> >
> > brcm_scmi@0 {
> > compatible = "arm,scmi-smc\0arm,scmi";
> > mboxes = <0x04 0x00 0x04 0x01>;
> > mbox-names = "tx\0rx";
> > shmem = <0x05>;
> > status = "disabled";
> > arm,smc-id = <0x83000400>;
> > interrupt-names = "a2p";
> > #address-cells = <0x01>;
> > #size-cells = <0x00>;
> >
> > it seems to me that even though you declare an SMC based transport (and in fact
> > you define the smc-id too) you also still define mboxes (as a fallback I suppose)
> > referring to the brcm_scmi_mailbox phandle, and while this is ignored by the SCMI
> > driver (because you have selected a compatible SMC transport) I imagine this dep
> > is picked up by fw_devlink which in fact says:
> >
> > > [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
> >
> > and stalls waiting for it. (but I'm not really familiar on how fw_devlink
> > internals works really...so I maybe off in these regards)
Cristian,
Great debugging work for not having worked on this before! Your
comments about the dependencies are right. If you grep the logs for
"probe deferral", you'll see these lines and more:
[ 0.942998] platform brcm_scmi@0: probe deferral - supplier
brcm_scmi_mailbox@0 not ready
[ 3.622741] platform 8b20000.pcie: probe deferral - supplier
brcm_scmi@0 not ready
[ 5.695929] platform 840c000.serial: probe deferral - supplier
brcm_scmi@0 not ready
Florian,
Sorry I wasn't clear in my earlier email. I was asking for the path to
the board file DT in upstream so I could look at it and the files it
references. I didn't mean to ask for an "decompiled" DTS attachment.
The decompiled ones make it a pain to track the phandles.
The part that's confusing to me is that the mbox node is disabled in
the DT you attached. fw_devlink is smart enough to ignore disabled
nodes. Is it getting enabled by the bootloader? Can you please try
deleting the reference to the brcm_scmi_mailbox from the scmi node and
see if it helps? Or leave it really disabled?
Also, as a separate test of workarounds, can you please add
deferred_probe_timeout=1 to your commandline and see if it helps? I'm
assuming you have modules enabled? Otherwise, the existing smarts in
fw_devlink to ignore devices with no drivers would have kicked in too.
> I was about to mention/ask the same when I saw Jim's reply. I see you have
> already asked that. Couple of my opinions based on my very limited knowledge
> on fw_devlink and how it works.
>
> 1. Since we have different compatible for SMC and mailbox, I am not sure
> if it correct to leave mailbox information in scmi node. Once we have
> proper yaml scheme, we must flag that error IMO.
>
> 2. IIUC, the fw_devlink might use information from DT to establish the
> dependency and having mailbox information in this context may be
> considered wrong as there is no dependency if it is using SMC.
If this mbox reference from scmi is wrong for the current kernel and
never used, then I'd recommend deleting that.
-Saravana
On Tue, Apr 27, 2021 at 9:25 AM Florian Fainelli <[email protected]> wrote:
>
>
>
> On 4/27/2021 8:10 AM, Sudeep Holla wrote:
> > On Tue, Apr 27, 2021 at 03:11:16PM +0100, Cristian Marussi wrote:
> >> On Tue, Apr 27, 2021 at 09:33:31AM -0400, Jim Quinlan wrote:
> > [...]
> >>>>
> >>> I believe that the brcmstb-mbox node is in our DT for backwards
> >>> compatibility with our older Linux only. Note that we use the compatible
> >>> string '"arm,scmi-smc", "arm,scmi"'; the former chooses SMC transport and
> >>> ignores custom mailboxes such as brcmstb-mbox.
> >>>
> >>
> >> Right..so it is even more wrong that it is waiting for the mailboxes...but
> >> looking at the DT:
> >>
> >> brcm_scmi_mailbox@0 {
> >> #mbox-cells = <0x01>;
> >> compatible = "brcm,brcmstb-mbox";
> >> status = "disabled";
> >> linux,phandle = <0x04>;
> >> phandle = <0x04>;
> >> };
> >>
> >> brcm_scmi@0 {
> >> compatible = "arm,scmi-smc\0arm,scmi";
> >> mboxes = <0x04 0x00 0x04 0x01>;
> >> mbox-names = "tx\0rx";
> >> shmem = <0x05>;
> >> status = "disabled";
> >> arm,smc-id = <0x83000400>;
> >> interrupt-names = "a2p";
> >> #address-cells = <0x01>;
> >> #size-cells = <0x00>;
> >>
> >> it seems to me that even though you declare an SMC based transport (and in fact
> >> you define the smc-id too) you also still define mboxes (as a fallback I suppose)
> >> referring to the brcm_scmi_mailbox phandle, and while this is ignored by the SCMI
> >> driver (because you have selected a compatible SMC transport) I imagine this dep
> >> is picked up by fw_devlink which in fact says:
> >>
> >>> [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
> >>
> >> and stalls waiting for it. (but I'm not really familiar on how fw_devlink
> >> internals works really...so I maybe off in these regards)
> >>
> >
> > I was about to mention/ask the same when I saw Jim's reply. I see you have
> > already asked that. Couple of my opinions based on my very limited knowledge
> > on fw_devlink and how it works.
> >
> > 1. Since we have different compatible for SMC and mailbox, I am not sure
> > if it correct to leave mailbox information in scmi node. Once we have
> > proper yaml scheme, we must flag that error IMO.
>
> This is a self inflicted problem that we have in that the bootloader
> provides a Device Tree to the kernel which is massaged in different ways
> and intends to stay backwards compatible as much as possible. And indeed
> after removing the 'mboxes' property gets us going with fw_devlink=on.
I'm sure you'll see my other email and reply to it, so I'll not
respond to this part.
>
> >
> > 2. IIUC, the fw_devlink might use information from DT to establish the
> > dependency and having mailbox information in this context may be
> > considered wrong as there is no dependency if it is using SMC.
>
> Right, unfortunately, short of having some special casing for SCMI and
> checking that if we have both an "arm,smc-id" and "mboxes" phandle we
> should prefer the former, there is not probably much that can be done
> here. Do we want to do that?
Definite no for special casing per device nodes in fw_devlink code :)
-Saravana
On Tue, Apr 27, 2021 at 09:24:55AM -0700, Florian Fainelli wrote:
[...]
> This is a self inflicted problem that we have in that the bootloader
> provides a Device Tree to the kernel which is massaged in different ways
> and intends to stay backwards compatible as much as possible. And indeed
> after removing the 'mboxes' property gets us going with fw_devlink=on.
>
I assume the bootloader checks the presence of SMC support and modifies
the DT node accordingly. Can't it remove the mbox properties as it make
no sense with SMC compatible ? However ...
> >
> > 2. IIUC, the fw_devlink might use information from DT to establish the
> > dependency and having mailbox information in this context may be
> > considered wrong as there is no dependency if it is using SMC.
>
> Right, unfortunately, short of having some special casing for SCMI and
> checking that if we have both an "arm,smc-id" and "mboxes" phandle we
> should prefer the former, there is not probably much that can be done
> here. Do we want to do that?
I *think* we could do that in the SCMI drivers, but:
1. I am not sure if that helps fw_devlinks if they are deriving the info
purely based on DT
2. I am also afraid that someone might come up with exactly opposite
requirement that let us prefer mailbox over SMC as they would use
SMC only if h/w lacks proper mailbox support. I fear that we will get
into rabbit hole trying to do something like that.
--
Regards,
Sudeep
On Tue, Apr 27, 2021 at 09:28:16AM -0700, Saravana Kannan wrote:
> On Tue, Apr 27, 2021 at 9:25 AM Florian Fainelli <[email protected]> wrote:
[...]
> > Right, unfortunately, short of having some special casing for SCMI and
> > checking that if we have both an "arm,smc-id" and "mboxes" phandle we
> > should prefer the former, there is not probably much that can be done
> > here. Do we want to do that?
>
> Definite no for special casing per device nodes in fw_devlink code :)
>
Thanks for the confirmation, but I had already assumed so with 100% certainty
in my last reply ????.
--
Regards,
Sudeep
On 4/27/2021 9:24 AM, Saravana Kannan wrote:
> On Tue, Apr 27, 2021 at 8:10 AM Sudeep Holla <[email protected]> wrote:
>>
>> On Tue, Apr 27, 2021 at 03:11:16PM +0100, Cristian Marussi wrote:
>>> On Tue, Apr 27, 2021 at 09:33:31AM -0400, Jim Quinlan wrote:
>> [...]
>>>>>
>>>> I believe that the brcmstb-mbox node is in our DT for backwards
>>>> compatibility with our older Linux only. Note that we use the compatible
>>>> string '"arm,scmi-smc", "arm,scmi"'; the former chooses SMC transport and
>>>> ignores custom mailboxes such as brcmstb-mbox.
>>>>
>>>
>>> Right..so it is even more wrong that it is waiting for the mailboxes...but
>>> looking at the DT:
>>>
>>> brcm_scmi_mailbox@0 {
>>> #mbox-cells = <0x01>;
>>> compatible = "brcm,brcmstb-mbox";
>>> status = "disabled";
>>> linux,phandle = <0x04>;
>>> phandle = <0x04>;
>>> };
>>>
>>> brcm_scmi@0 {
>>> compatible = "arm,scmi-smc\0arm,scmi";
>>> mboxes = <0x04 0x00 0x04 0x01>;
>>> mbox-names = "tx\0rx";
>>> shmem = <0x05>;
>>> status = "disabled";
>>> arm,smc-id = <0x83000400>;
>>> interrupt-names = "a2p";
>>> #address-cells = <0x01>;
>>> #size-cells = <0x00>;
>>>
>>> it seems to me that even though you declare an SMC based transport (and in fact
>>> you define the smc-id too) you also still define mboxes (as a fallback I suppose)
>>> referring to the brcm_scmi_mailbox phandle, and while this is ignored by the SCMI
>>> driver (because you have selected a compatible SMC transport) I imagine this dep
>>> is picked up by fw_devlink which in fact says:
>>>
>>>> [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
>>>
>>> and stalls waiting for it. (but I'm not really familiar on how fw_devlink
>>> internals works really...so I maybe off in these regards)
>
> Cristian,
>
> Great debugging work for not having worked on this before! Your
> comments about the dependencies are right. If you grep the logs for
> "probe deferral", you'll see these lines and more:
>
> [ 0.942998] platform brcm_scmi@0: probe deferral - supplier
> brcm_scmi_mailbox@0 not ready
> [ 3.622741] platform 8b20000.pcie: probe deferral - supplier
> brcm_scmi@0 not ready
> [ 5.695929] platform 840c000.serial: probe deferral - supplier
> brcm_scmi@0 not ready
>
> Florian,
>
> Sorry I wasn't clear in my earlier email. I was asking for the path to
> the board file DT in upstream so I could look at it and the files it
> references. I didn't mean to ask for an "decompiled" DTS attachment.
> The decompiled ones make it a pain to track the phandles.
Our Device Tree sources are not in the kernel since the bootloader
provides a FDT to the kernel which is massaged in different ways to
support a single binary for a multitude of reference boards and chip
variants. That FDT is 90% auto-generated offline from scripts and about
10% runtime patched based on our whim. We should probably still aim for
some visibility into these Device Tree files by the kernel community.
>
> The part that's confusing to me is that the mbox node is disabled in
> the DT you attached. fw_devlink is smart enough to ignore disabled
> nodes. Is it getting enabled by the bootloader? Can you please try
> deleting the reference to the brcm_scmi_mailbox from the scmi node and
> see if it helps? Or leave it really disabled?
Removing the 'mboxes' phandle works, see my other reply to Sudeep and I
should have captured the DT from the Linux prompt after it has been
finalized and where the mbox node is marked as enabled unfortunately.
>
> Also, as a separate test of workarounds, can you please add
> deferred_probe_timeout=1 to your commandline and see if it helps? I'm
> assuming you have modules enabled? Otherwise, the existing smarts in
> fw_devlink to ignore devices with no drivers would have kicked in too.
deferred_probe_timeout=1 does help however all of these drivers are
built into the kernel at the moment and so ultimately we reach
user-space with no console driver registered.
>
>> I was about to mention/ask the same when I saw Jim's reply. I see you have
>> already asked that. Couple of my opinions based on my very limited knowledge
>> on fw_devlink and how it works.
>>
>> 1. Since we have different compatible for SMC and mailbox, I am not sure
>> if it correct to leave mailbox information in scmi node. Once we have
>> proper yaml scheme, we must flag that error IMO.
>>
>> 2. IIUC, the fw_devlink might use information from DT to establish the
>> dependency and having mailbox information in this context may be
>> considered wrong as there is no dependency if it is using SMC.
>
> If this mbox reference from scmi is wrong for the current kernel and
> never used, then I'd recommend deleting that.
Yes that seems to be the way forward unless we want to set
fw_devlink=permissive on the command line, either should hopefully be an
option.
Thanks a lot for your response!
--
Florian
On 4/27/2021 9:39 AM, Sudeep Holla wrote:
> On Tue, Apr 27, 2021 at 09:24:55AM -0700, Florian Fainelli wrote:
>
> [...]
>
>> This is a self inflicted problem that we have in that the bootloader
>> provides a Device Tree to the kernel which is massaged in different ways
>> and intends to stay backwards compatible as much as possible. And indeed
>> after removing the 'mboxes' property gets us going with fw_devlink=on.
>>
>
> I assume the bootloader checks the presence of SMC support and modifies
> the DT node accordingly. Can't it remove the mbox properties as it make
> no sense with SMC compatible ? However ...
The bootloader has always assumed the SMC support was there from the day
we introduced it because it was. What changed is the way we advertised
to Linux that support. We used to have a custom mailbox driver that
would be pretty much what the ARM SMC transport eventually came to be.
Since we still support earlier kernels that were deployed with the old
mailbox we cannot arbitrarily break that setup, especially as our
customers tend to be slow in picking up new kernel versions, fortunately
before they get to 5.13 we can mandate a new bootloader that may not be
compatible with their 4.1 kernel anymore, or at least not without some
backporting of the ARM SMC transport, that's all fair IMHO.
>
>>>
>>> 2. IIUC, the fw_devlink might use information from DT to establish the
>>> dependency and having mailbox information in this context may be
>>> considered wrong as there is no dependency if it is using SMC.
>>
>> Right, unfortunately, short of having some special casing for SCMI and
>> checking that if we have both an "arm,smc-id" and "mboxes" phandle we
>> should prefer the former, there is not probably much that can be done
>> here. Do we want to do that?
>
> I *think* we could do that in the SCMI drivers, but:
> 1. I am not sure if that helps fw_devlinks if they are deriving the info
> purely based on DT
> 2. I am also afraid that someone might come up with exactly opposite
> requirement that let us prefer mailbox over SMC as they would use
> SMC only if h/w lacks proper mailbox support. I fear that we will get
> into rabbit hole trying to do something like that.
That is true, and to get to the SCMI driver, even the base protocol you
must have been probed, so we have a nice chicken and egg problem. I
highly appreciate your time understanding the context and trying to find
a solution it is pretty clear that we must fix our FDT now.
--
Florian
Hi Florian,
On Tue, Apr 27, 2021 at 6:50 PM Florian Fainelli <[email protected]> wrote:
> On 4/27/2021 9:39 AM, Sudeep Holla wrote:
> > On Tue, Apr 27, 2021 at 09:24:55AM -0700, Florian Fainelli wrote:
> >
> > [...]
> >
> >> This is a self inflicted problem that we have in that the bootloader
> >> provides a Device Tree to the kernel which is massaged in different ways
> >> and intends to stay backwards compatible as much as possible. And indeed
> >> after removing the 'mboxes' property gets us going with fw_devlink=on.
> >>
> >
> > I assume the bootloader checks the presence of SMC support and modifies
> > the DT node accordingly. Can't it remove the mbox properties as it make
> > no sense with SMC compatible ? However ...
>
> The bootloader has always assumed the SMC support was there from the day
> we introduced it because it was. What changed is the way we advertised
> to Linux that support. We used to have a custom mailbox driver that
> would be pretty much what the ARM SMC transport eventually came to be.
>
> Since we still support earlier kernels that were deployed with the old
> mailbox we cannot arbitrarily break that setup, especially as our
> customers tend to be slow in picking up new kernel versions, fortunately
> before they get to 5.13 we can mandate a new bootloader that may not be
> compatible with their 4.1 kernel anymore, or at least not without some
> backporting of the ARM SMC transport, that's all fair IMHO.
>
> >>> 2. IIUC, the fw_devlink might use information from DT to establish the
> >>> dependency and having mailbox information in this context may be
> >>> considered wrong as there is no dependency if it is using SMC.
> >>
> >> Right, unfortunately, short of having some special casing for SCMI and
> >> checking that if we have both an "arm,smc-id" and "mboxes" phandle we
> >> should prefer the former, there is not probably much that can be done
> >> here. Do we want to do that?
> >
> > I *think* we could do that in the SCMI drivers, but:
> > 1. I am not sure if that helps fw_devlinks if they are deriving the info
> > purely based on DT
> > 2. I am also afraid that someone might come up with exactly opposite
> > requirement that let us prefer mailbox over SMC as they would use
> > SMC only if h/w lacks proper mailbox support. I fear that we will get
> > into rabbit hole trying to do something like that.
>
> That is true, and to get to the SCMI driver, even the base protocol you
> must have been probed, so we have a nice chicken and egg problem. I
> highly appreciate your time understanding the context and trying to find
> a solution it is pretty clear that we must fix our FDT now.
Alternatively, you can have a quirk in the kernel that removes the
phandle from the FDT during early boot.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
On Tue, Apr 27, 2021 at 9:47 AM Florian Fainelli <[email protected]> wrote:
>
>
>
> On 4/27/2021 9:24 AM, Saravana Kannan wrote:
> > On Tue, Apr 27, 2021 at 8:10 AM Sudeep Holla <[email protected]> wrote:
> >>
> >> On Tue, Apr 27, 2021 at 03:11:16PM +0100, Cristian Marussi wrote:
> >>> On Tue, Apr 27, 2021 at 09:33:31AM -0400, Jim Quinlan wrote:
> >> [...]
> >>>>>
> >>>> I believe that the brcmstb-mbox node is in our DT for backwards
> >>>> compatibility with our older Linux only. Note that we use the compatible
> >>>> string '"arm,scmi-smc", "arm,scmi"'; the former chooses SMC transport and
> >>>> ignores custom mailboxes such as brcmstb-mbox.
> >>>>
> >>>
> >>> Right..so it is even more wrong that it is waiting for the mailboxes...but
> >>> looking at the DT:
> >>>
> >>> brcm_scmi_mailbox@0 {
> >>> #mbox-cells = <0x01>;
> >>> compatible = "brcm,brcmstb-mbox";
> >>> status = "disabled";
> >>> linux,phandle = <0x04>;
> >>> phandle = <0x04>;
> >>> };
> >>>
> >>> brcm_scmi@0 {
> >>> compatible = "arm,scmi-smc\0arm,scmi";
> >>> mboxes = <0x04 0x00 0x04 0x01>;
> >>> mbox-names = "tx\0rx";
> >>> shmem = <0x05>;
> >>> status = "disabled";
> >>> arm,smc-id = <0x83000400>;
> >>> interrupt-names = "a2p";
> >>> #address-cells = <0x01>;
> >>> #size-cells = <0x00>;
> >>>
> >>> it seems to me that even though you declare an SMC based transport (and in fact
> >>> you define the smc-id too) you also still define mboxes (as a fallback I suppose)
> >>> referring to the brcm_scmi_mailbox phandle, and while this is ignored by the SCMI
> >>> driver (because you have selected a compatible SMC transport) I imagine this dep
> >>> is picked up by fw_devlink which in fact says:
> >>>
> >>>> [ 0.300086] platform brcm_scmi@0: Linked as a consumer to brcm_scmi_mailbox@0
> >>>
> >>> and stalls waiting for it. (but I'm not really familiar on how fw_devlink
> >>> internals works really...so I maybe off in these regards)
> >
> > Cristian,
> >
> > Great debugging work for not having worked on this before! Your
> > comments about the dependencies are right. If you grep the logs for
> > "probe deferral", you'll see these lines and more:
> >
> > [ 0.942998] platform brcm_scmi@0: probe deferral - supplier
> > brcm_scmi_mailbox@0 not ready
> > [ 3.622741] platform 8b20000.pcie: probe deferral - supplier
> > brcm_scmi@0 not ready
> > [ 5.695929] platform 840c000.serial: probe deferral - supplier
> > brcm_scmi@0 not ready
> >
> > Florian,
> >
> > Sorry I wasn't clear in my earlier email. I was asking for the path to
> > the board file DT in upstream so I could look at it and the files it
> > references. I didn't mean to ask for an "decompiled" DTS attachment.
> > The decompiled ones make it a pain to track the phandles.
>
> Our Device Tree sources are not in the kernel since the bootloader
> provides a FDT to the kernel which is massaged in different ways to
> support a single binary for a multitude of reference boards and chip
> variants. That FDT is 90% auto-generated offline from scripts and about
> 10% runtime patched based on our whim. We should probably still aim for
> some visibility into these Device Tree files by the kernel community.
>
> >
> > The part that's confusing to me is that the mbox node is disabled in
> > the DT you attached. fw_devlink is smart enough to ignore disabled
> > nodes. Is it getting enabled by the bootloader? Can you please try
> > deleting the reference to the brcm_scmi_mailbox from the scmi node and
> > see if it helps? Or leave it really disabled?
>
> Removing the 'mboxes' phandle works, see my other reply to Sudeep and I
> should have captured the DT from the Linux prompt after it has been
> finalized and where the mbox node is marked as enabled unfortunately.
>
> >
> > Also, as a separate test of workarounds, can you please add
> > deferred_probe_timeout=1 to your commandline and see if it helps? I'm
> > assuming you have modules enabled? Otherwise, the existing smarts in
> > fw_devlink to ignore devices with no drivers would have kicked in too.
>
> deferred_probe_timeout=1 does help however all of these drivers are
> built into the kernel at the moment and so ultimately we reach
> user-space with no console driver registered.
Whether all the required drivers are built in already or not doesn't
matter for this workaround. fw_devlink can't tell if you are just
about to load a module that'll probe the mailbox. If CONFIG_MODULES is
disabled, then it can tell no more drivers are getting loaded by the
time you hit late_initcall_sync() and it would have automatically
applied this workaround without deferred_probe_timeout=1.
>
> >
> >> I was about to mention/ask the same when I saw Jim's reply. I see you have
> >> already asked that. Couple of my opinions based on my very limited knowledge
> >> on fw_devlink and how it works.
> >>
> >> 1. Since we have different compatible for SMC and mailbox, I am not sure
> >> if it correct to leave mailbox information in scmi node. Once we have
> >> proper yaml scheme, we must flag that error IMO.
> >>
> >> 2. IIUC, the fw_devlink might use information from DT to establish the
> >> dependency and having mailbox information in this context may be
> >> considered wrong as there is no dependency if it is using SMC.
> >
> > If this mbox reference from scmi is wrong for the current kernel and
> > never used, then I'd recommend deleting that.
>
> Yes that seems to be the way forward unless we want to set
> fw_devlink=permissive on the command line, either should hopefully be an
> option.
I read all the other emails from Sudeep, Geert and you. I'll just
respond to all of them here.
My preferred order of the workarouds:
1. Fix the DT sent to the kernel.
2. If deferred_probe_timeout=1 doesn't break anything else, use that.
This is better than (4).
3. Geert's early boot quirk suggestion.
4. fw_devlink=permissive (least preferred because this might mask
issues with fw_devlink=on in your future changes).
Changing the SCMI driver itself won't help fw_devlink.
Thanks,
Saravana
On Tue, Apr 27, 2021 at 09:47:08AM -0700, Florian Fainelli wrote:
> On 4/27/2021 9:24 AM, Saravana Kannan wrote:
[...]
> > The part that's confusing to me is that the mbox node is disabled in
> > the DT you attached. fw_devlink is smart enough to ignore disabled
> > nodes. Is it getting enabled by the bootloader? Can you please try
> > deleting the reference to the brcm_scmi_mailbox from the scmi node and
> > see if it helps? Or leave it really disabled?
>
> Removing the 'mboxes' phandle works, see my other reply to Sudeep and I
> should have captured the DT from the Linux prompt after it has been
> finalized and where the mbox node is marked as enabled unfortunately.
I know Saravana has compiled the list of workarounds in preferred order
which excludes this which is good. Anyways I will convey my point. Disabling
mailbox controller node may not be always available option as it could be
providing multiple channels and the stale mboxes reference in SCMI might
be just one of the consumer. Removing mboxes seems only reasonable solution
to me in such scenario.
--
Regards,
Sudeep