[PATCH v2] firmware: arm_scmi: Queue in scmi layer for mailbox implementation
Sudeep Holla
sudeep.holla at arm.com
Mon Oct 14 13:29:07 PDT 2024
On Mon, Oct 14, 2024 at 10:21:28AM -0700, Florian Fainelli wrote:
> On 10/14/24 02:55, Sudeep Holla wrote:
> > On Sun, Oct 13, 2024 at 10:26:49AM +0100, Cristian Marussi wrote:
> > > On Fri, Oct 11, 2024 at 12:15:07PM -0700, Justin Chen wrote:
> > > >
> > > > Yes, I see the implementation. Looks like it returns the position in the
> > > > ring buffer. I also confirmed with CONFIG_DEBUG_MUTEXES which triggers a
> > > > warning.
> > > >
> > > > What about this?
> > > > if (ret >= 0)
> > > > ret = 0
> > > > else
> > > > mutex_unlock(&smbox->chan_lock);
> > > >
> > > > A bit easier to read IMO.
> > >
> > > Oh yes much better definitely...or, maybe, even more simply to read:
> > >
> > > ...
> > >
> > > mutex_lock(&smbox->chan_lock);
> > > ret = mbox_send_message(smbox->chan, xfer);
> > > if (ret < 0) {
> > > mutex_unlock(&smbox->chan_lock);
> > > return ret;
> > > }
> > >
> > > return 0;
> > > }
> > >
> > > .... up to You...not sure what Sudeep prefers...
> > >
> >
> > I like this better. Also I was hoping Justin would send v3 soonish, I want
> > to send PR for fixes soon. So I have gone ahead and fixed all your comments
> > myself. I had seen the doxygen one from builder already and had fixed it up
> > last week when I added this to -next.
> >
>
> Here's his v3:
>
> https://lore.kernel.org/all/20241014160717.1678953-1-justin.chen@broadcom.com/
>
> we are on the other side of the pond ;)
I do understand 😉, it was my own rush and hence I did the changes myself.
But since I haven't tagged, I will pull it instead. BTW looks like the
arm-scmi list was dropped in v3 or my filters are misbehaving. Thanks for the
link.
--
Regards,
Sudeep
More information about the linux-arm-kernel
mailing list