linux-kernel - Re: [RFC] dt-bindings: firmware: scmi: Introduce compatible string

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAL_JsqLePri5m_dE989poUV4auasAxFvgAiYuXVuZHqLcOBGMg@mail.gmail.com>
Date: Fri, 28 Feb 2025 08:17:03 -0600
From: Rob Herring <robh@...nel.org>
To: Sudeep Holla <sudeep.holla@....com>
Cc: Peng Fan <peng.fan@....nxp.com>, saravanak@...gle.com, cristian.marussi@....com, 
	krzk+dt@...nel.org, conor+dt@...nel.org, arm-scmi@...r.kernel.org, 
	linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, Peng Fan <peng.fan@....com>
Subject: Re: [RFC] dt-bindings: firmware: scmi: Introduce compatible string

On Fri, Feb 28, 2025 at 8:04 AM Sudeep Holla <sudeep.holla@....com> wrote:
>
> On Fri, Feb 28, 2025 at 07:34:09AM -0600, Rob Herring wrote:
> >
> > - The parent driver creates child devices. The child devices can
> > either reuse the parent DT node (i.e. set dev.of_node) or just get it
> > from the parent device if needed.
> >
>
> This is exactly what I was thinking to deal with the issue since this
> discussion started. I will give this a go. I believe this must solve
> the issue, but I didn't want to spit it out loud until I tried to hack
> and check.

The issue with fw_devlink is that it only checks the dependency of the
parent which won't be enough. When the parent's probe creates the
child device, that doesn't mean the child has probed. The child driver
might not be loaded and/or probe is async. I don't think there's
anyway for the parent probe to wait for child drivers to be probed and
ready. I think there's similar issues with the DWC3 wrapper and core
driver split.

Rob