On 14/09/2026 13:41, Albert Esteve wrote:
static int spm_cpuidle_drv_probe(struct platform_device *pdev)
{
+ struct qcom_scm *scm = qcom_scm_get();
int cpu, ret;
- if (!qcom_scm_is_available())
+ if (!scm)
return -EPROBE_DEFER;
Is this equivalent code ?
Its also a bit confusing to me that the next patch adds a .available
field to the qcom_scm struct.
I see where you do get drvdata but I haven't managed to find in the
series where the data gets set - haven't asked an LLM either - I think
the logic to associate the scm pointer and get it should live together
in one patch so that the logic is easier to find and reason about.
---
bod
