Thread (3 messages) 3 messages, 2 authors, 2021-02-01

Re: [PATCH v2 0/2] of: property: Add fw_devlink support for more props

From: Saravana Kannan <hidden>
Date: 2021-02-01 20:06:37
Also in: linux-tegra, lkml

Possibly related (same subject, not in this thread)

On Mon, Feb 1, 2021 at 2:52 AM Martin Kaiser [off-list ref] wrote:
Hi Saravana,

Thus wrote Saravana Kannan (saravanak@google.com):
quoted
This series [1] has a high chance of fixing it for you if
CONFIG_MODULES is disabled in your set up. Can you give it a shot?
sure. This fixes things for me if CONFIG_MODULES is disabled. Booting is
still stuck if modules are enabled.
quoted
The real problem is that arch/arm/mach-imx/avic.c doesn't set the
OF_POPULATED flag for the "fsl,avic" node. fw_devlink uses this
information to know that this device node will never have a struct
device created for it. The proper way to do this for root IRQCHIP
nodes is to use IRQCHIP_DECLARE(). I Cc'ed you on a clean up patch for
IMX [2], can you please give [2] a shot *without* [1] and with
CONFIG_MODULES enabled? Things should boot properly with this
combination too.
This works as well.
Thanks for testing both. Mind giving Tested-by for [1] too?

-Saravana
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help