On Sun, Aug 14, 2022 at 11:31 AM Khalid Masum <khalid.masum.92@xxxxxxxxx> wrote:That is a good information to know. Thanks.
According to the TODO, In sw_bus_master_add, bus->multi_link is to be
populated with properties from FW node props. Make this happen by
creating a new fwnode_handle flag FWNODE_FLAG_MULTI_LINKED and use
the flag to store the multi_link value from intel_link_startup. Use
this flag to initialize bus->multi_link.
...
/*
* Initialize multi_link flag
- * TODO: populate this flag by reading property from FW node
*/
- bus->multi_link = false;
+ bus->multi_link = (fwnode->flags & FWNODE_FLAG_MULTI_LINKED)
+ == FWNODE_FLAG_MULTI_LINKED;
NAK (as far as I understood the context of the comment and the change itself).
These flags are for devlink, we do not mix FW properties with those
internal flags anyhow. The comment suggests that this should be az
property. Also commit message doesn't explain the relation to devlink.