Messages in this thread | | | Subject | Re: [PATCH v3 07/10] ata: pata_platform: Merge pata_of_platform into pata_platform | From | Sergey Shtylyov <> | Date | Mon, 27 Dec 2021 23:54:00 +0300 |
| |
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
| |