Hi Alistair, the patch looks quite good only a few notes inline. PS: It would be cool to have a log to previous patch versions. On 21-03-15 22:27, Alistair Francis wrote: ... > +&crypto { > + status = "disabled"; > +}; Why do you disable the crypto subsystem? > +&dma_apbh { > + status = "disabled"; > +}; Why do you disable this dma controller? > + > +&sdma { > + status = "okay"; > +}; This is the default state so you can drop this node. Regards, Marco