Re: [PATCH v10 12/24] firmware: arm_scmi: Add support for boot-on Telemetry

Fayssal Benmlih <[email protected]>
Newsgroups org.kernel.vger.arm-scmi,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
Hi Cristian,

The group configuration lookup error handling reported on V7 still appears
unchanged.

scmi_telemetry_de_groups_init() calls:

        scmi_telemetry_group_config_lookup(ti, &rinfo->grps[i]);

for every group but discards every return value. It then exposes all groups
through rinfo->num_groups.

If firmware rejects one lookup or a transport error occurs, that group is
therefore exposed with zero/default cached state and enumeration is reported
as successful.

Please propagate the first error, or record that the affected group state
is unavailable and preserve an explicit partial-enumeration indication.

Thanks,
Fayçal
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.