On 14 December 2016 at 03:10, Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> wrote: > Add a Socionext SoC specific compatible (suggested by Rob Herring). > > No SoC specific data are associated with the compatible strings for > now, but other SoC vendors may use this IP and want to differentiate > IP variants in the future. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> This looks good to me. I intend to apply this as fix, although deferring to first get an ack from Rob - to make sure we get it right this time. :-) Kind regards Uffe > --- > > Changes in v2: > - Add "uniphier" to the compatible to make it more SoC-specific > > Documentation/devicetree/bindings/mmc/sdhci-cadence.txt | 6 ++++-- > drivers/mmc/host/sdhci-cadence.c | 1 + > 2 files changed, 5 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mmc/sdhci-cadence.txt b/Documentation/devicetree/bindings/mmc/sdhci-cadence.txt > index 750374f..c0f37cb 100644 > --- a/Documentation/devicetree/bindings/mmc/sdhci-cadence.txt > +++ b/Documentation/devicetree/bindings/mmc/sdhci-cadence.txt > @@ -1,7 +1,9 @@ > * Cadence SD/SDIO/eMMC Host Controller > > Required properties: > -- compatible: should be "cdns,sd4hc". > +- compatible: should be one of the following: > + "cdns,sd4hc" - default of the IP > + "socionext,uniphier-sd4hc" - for Socionext UniPhier SoCs > - reg: offset and length of the register set for the device. > - interrupts: a single interrupt specifier. > - clocks: phandle to the input clock. > @@ -19,7 +21,7 @@ if supported. See mmc.txt for details. > > Example: > emmc: sdhci@5a000000 { > - compatible = "cdns,sd4hc"; > + compatible = "socionext,uniphier-sd4hc", "cdns,sd4hc"; > reg = <0x5a000000 0x400>; > interrupts = <0 78 4>; > clocks = <&clk 4>; > diff --git a/drivers/mmc/host/sdhci-cadence.c b/drivers/mmc/host/sdhci-cadence.c > index 1501cfd..4b0ecb9 100644 > --- a/drivers/mmc/host/sdhci-cadence.c > +++ b/drivers/mmc/host/sdhci-cadence.c > @@ -262,6 +262,7 @@ static int sdhci_cdns_probe(struct platform_device *pdev) > } > > static const struct of_device_id sdhci_cdns_match[] = { > + { .compatible = "socionext,uniphier-sd4hc" }, > { .compatible = "cdns,sd4hc" }, > { /* sentinel */ } > }; > -- > 2.7.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html