Hi all, any feedback on this patch? It's needed for msm8953 to boot properly. Regards Luca On Sonntag, 20. Februar 2022 21:18:58 CET Luca Weiss wrote: > On msm8953 the channel seems to be already opened when booting Linux but > we still need to open it for communication with regulators etc. > > Signed-off-by: Luca Weiss <luca@xxxxxxxxx> > --- > Changes in v2: > - rework original patch, don't drop condition completely but allow force > opening rpm_requests channel > > drivers/rpmsg/qcom_smd.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/drivers/rpmsg/qcom_smd.c b/drivers/rpmsg/qcom_smd.c > index 540e027f08c4..887e21ca51f2 100644 > --- a/drivers/rpmsg/qcom_smd.c > +++ b/drivers/rpmsg/qcom_smd.c > @@ -1288,9 +1288,14 @@ static void qcom_channel_state_worker(struct > work_struct *work) if (channel->state != SMD_CHANNEL_CLOSED) > continue; > > + /* > + * Always open rpm_requests, even when already opened which is > + * required on some SoCs like msm8953. > + */ > remote_state = GET_RX_CHANNEL_INFO(channel, state); > if (remote_state != SMD_CHANNEL_OPENING && > - remote_state != SMD_CHANNEL_OPENED) > + remote_state != SMD_CHANNEL_OPENED && > + strcmp(channel->name, "rpm_requests")) > continue; > > if (channel->registered)