Russ Dill <[email protected]> writes:
> On Tue, Aug 6, 2013 at 10:49 AM, Dave Gerlach <[email protected]> wrote:
[...]
>> +static int wkup_m3_probe(struct platform_device *pdev)
>> +{
>> + int irq, ret = 0;
>> + struct resource *mem;
>> +
>> + pm_runtime_enable(&pdev->dev);
>> +
>> + ret = pm_runtime_get_sync(&pdev->dev);
>> + if (IS_ERR_VALUE(ret)) {
>> + dev_err(&pdev->dev, "pm_runtime_get_sync() failed\n");
>> + return ret;
>> + }
>> +
>> + irq = platform_get_irq(pdev, 0);
>> + if (!irq) {
>> + dev_err(wkup_m3->dev, "no irq resource\n");
>
> &pdev->dev
>
>> + ret = -ENXIO;
>> + goto err;
>> + }
>> +
>> + mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>> + if (!mem) {
>> + dev_err(wkup_m3->dev, "no memory resource\n");
>
> &pdev->dev
For future reference, when reviewing, please trim to only relevant
content/context, especially on large patches so maintainers/reviewers do
not have to find 2 one-line comments in a huge amount of irrelevant
context.
Thanks,
Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
