On Sat, Oct 11, 2014 at 04:52:04PM +0200, Karl Beldan wrote:
> On 28/09/14 23:24:04, Greg Ungerer wrote:
> > @@ -378,6 +452,10 @@ static int orion_spi_probe(struct platform_device *pdev)

> > +	of_id = of_match_device(orion_spi_of_match_table, &pdev->dev);
> > +	devdata = of_id->data;
> > +	spi->devdata = devdata;

> Won't of_match_device return NULL on non-DT systems, and further
> references oops ?

Yes.