On 26.03.2016 12:14, Marek Vasut wrote: > Hi! > > I noticed that arch/arc/boot/dts/axs10x_mb.dtsi uses "altr," prefix in > the DT compatible string: > > mmc@0x15000 { > compatible = "altr,socfpga-dw-mshc"; > reg = < 0x15000 0x400 >; > num-slots = < 1 >; > fifo-depth = < 16 >; > card-detect-delay = < 200 >; > clocks = <&apbclk>, <&mmcclk>; > clock-names = "biu", "ciu"; > interrupts = < 7 >; > bus-width = < 4 >; > }; > > I don't think this is OK, since ARC is unrelated to Altera, which is > what the "altr," prefix stands for. I think the socfpga-dw-mshc shim > should be extended with another compatibility string, something like > "snps,arc-dw-mshc" and the axs10x_mb.dtsi should be adjusted > accordingly. What do you think ? > There is "snps,dw-mshc" described in Documentation/devicetree/bindings/mmc/synopsys-dw-mshc.txt and supported by dw_mmc host controller driver. -- With best wishes, Vladimir -- 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