Hello! On 12/25/21 8:16 PM, Andy Shevchenko wrote: [...] > ... > >> + if ((pdev->num_resources != 3) && (pdev->num_resources != 2)) { >> + dev_err(&pdev->dev, "invalid number of resources\n"); >> + return -EINVAL; > > return dev_err_probe(); ? > >> + } > > ... > >> + if (!dev_of_node(&pdev->dev)) >> + ret = pata_platform_get_pdata(pdev, priv); >> + else >> + ret = pata_of_platform_get_pdata(pdev, priv); > > What the difference between them? One parses DT props into the private structure, the other inits this structure without DT... > Can't you unify them and leave only > DT related part separately? He can't -- grep *defconfig for PATA_PLATFORM=, please. [...] MBR, Sergey