On Wed, Sep 23, 2026 at 11:00 PM Bryan O'Donoghue <[email protected]> wrote: > > On 14/09/2026 13:41, Albert Esteve wrote: > > int qcom_scm_pas_auth_and_reset(u32 pas_id) > > { > > - return __qcom_scm_pas_auth_and_reset(__scm->dev, pas_id); > > + struct qcom_scm *scm = __scm; > > + > > + return __qcom_scm_pas_auth_and_reset(scm->dev, pas_id); > > } > > EXPORT_SYMBOL_GPL(qcom_scm_pas_auth_and_reset); > > Why make this change at all though ? > > It takes a pointer to the file static _scm > > The reset of the patch looks straight-forward to me but this change > caught my eye.
Yeah, this change isn't useful. I aimed to avoid passing __scm to function calls but it does nothing in this patch, it'd be better to change it directly in the next patch that changes the signature. BR, Albert > > --- > bod >
