Re: [PATCH] firmware: arm_scmi: Make scmi core independent of transport type

From: Arnd Bergmann
Date: Thu Jan 09 2020 - 05:15:30 EST


On Thu, Jan 9, 2020 at 10:34 AM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> On 09-01-20, 09:18, Arnd Bergmann wrote:
> > On Fri, Nov 29, 2019 at 10:32 AM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:

> > This looks odd: rather than guessing the transport type based on
> > random DT properties, I would prefer to have it determined by
> > the device compatible string, and have different drivers bind
> > to one of them each, with each driver linking against a common
> > base implementation, either as separate modules or in one file.
>
> Since there are no platforms using the scmi binding in mainline kernel
> for now, it won't be difficult to add new compatible strings. So
> should this be done like:
>
> compatible = "arm,scmi", "arm,scmi-mailbox";
>
> or just
> compatible = "arm,scmi-mailbox";

I would keep compatibility with the existing binding and make a plain "arm,scmi"
mean the version with the mailbox, while for new transports, I would
require them to have both the existing compatible string and a more specific
one.

Arnd