Re: [PATCH v2 2/7] of: property: fw_devlink: Report unsupported supplier linking

From: Herve Codina

Date: Wed Sep 16 2026 - 13:51:34 EST


Hi Ernest,

On Thu, 10 Sep 2026 16:16:43 +0200
Ernest Van Hoecke <ernestvanhoecke@xxxxxxxxx> wrote:

> From: Ernest Van Hoecke <ernest.vanhoecke@xxxxxxxxxxx>
>
> OF supplier linking is deliberately disabled on OLPC and Intel CE4100
> systems. The add_links operation currently returns zero on these systems,
> so callers cannot distinguish unsupported linking from success.
>
> Return -EOPNOTSUPP when supplier linking is unsupported. This allows OF
> unit tests to skip link assertions on these systems without exposing the
> private platform-support predicate.
>
> The driver core ignores the add_links return value and marks the fwnode
> as processed regardless, so its link creation and retry behavior remain
> unchanged.
>
> Signed-off-by: Ernest Van Hoecke <ernest.vanhoecke@xxxxxxxxxxx>
> ---
> drivers/of/property.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>

Reviewed-by: Herve Codina <herve.codina@xxxxxxxxxxx>

Best regards,
Hervé