On Wed, Sep 28, 2016 at 9:59 AM, Moritz Fischer <moritz.fischer@xxxxxxxxx> wrote: > Add compatible string for NI version of cros-ec-spi. > > Signed-off-by: Moritz Fischer <moritz.fischer@xxxxxxxxx> > --- > drivers/mfd/cros_ec_spi.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/mfd/cros_ec_spi.c b/drivers/mfd/cros_ec_spi.c > index ebe9b94..e4504ec 100644 > --- a/drivers/mfd/cros_ec_spi.c > +++ b/drivers/mfd/cros_ec_spi.c > @@ -708,6 +708,7 @@ static SIMPLE_DEV_PM_OPS(cros_ec_spi_pm_ops, cros_ec_spi_suspend, > cros_ec_spi_resume); > > static const struct of_device_id cros_ec_spi_of_match[] = { > + { .compatible = "ni,cros-ec-spi", }, I am confused: if the 2 controllers are the same, why do we need the new compat string, at least in the driver? Shouldn't DTS simply use "google,cros-ec-spi", or maybe both (compatible = "ni,cros-ec-spi", "google,cros-ec-spi")? DT folks, is there clear instructions somewhere as to when introduction of new compatible is warranted, along with the rules of how to settle on common fallback (i.e. how to make sure that, let's say, Linux, U-boot and FreeBSD use the same fallback compatible for the same series of devices)? > { .compatible = "google,cros-ec-spi", }, > { /* sentinel */ }, > }; > -- > 2.7.4 > Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html