On Tue, Jul 14, 2015 at 10:54:42AM +0100, Mark Brown wrote: > On Tue, Jul 14, 2015 at 11:19:56AM +0200, Uwe Kleine-König wrote: > > > static const struct of_device_id mpc512x_psc_spi_of_match[] = { > > - { .compatible = "fsl,mpc5121-psc-spi", }, > > + { .compatible = "fsl,mpc5121-psc-spi", .data = (void *)TYPE_MPC5121 }, > > + { .compatible = "fsl,mpc5125-psc-spi", .data = (void *)TYPE_MPC5125 }, > > {}, > > The code seems fine but this should update the binding document to > include the new compatible string. I don't find fsl,mpc5121-psc-spi documented either. The best I found is ocumentation/devicetree/bindings/powerpc/fsl/mpc5121-psc.txt which describes fsl,mpc5121-psc-uart and fsl,mpc5121-psc. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html