Dear Alexandre Belloni,

On Thu, 30 Jul 2015 02:22:02 +0200, Alexandre Belloni wrote:

 static int at91_reset_probe(struct platform_device *pdev)
 {
 	int ret;
 
-	if (pdev->dev.of_node)
-		ret = at91_reset_of_probe(pdev);
-	else
-		ret = at91_reset_platform_probe(pdev);
 
+	ret = at91_reset_of_probe(pdev);

Maybe it no longer makes sense to have separate at91_reset_probe() and
at91_reset_of_probe() functions then.

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com