>> +Andriy who originally raised the concern over the ACPI resource
>> assumptions in
>> the previous version. Andriy, this resource allocation looks to be a 
>> substantial improvement to me. Do you have any further concerns?

>So, regarding to the second patch

>1. In excerpts like following

>if (IS_ERR(addr)) {
>               dev_err(&pdev->dev, "Failed to map resouce for BIOS DATA\n");
>               return PTR_ERR(addr);
>       }

>No need to have an error message. Core already has something to print at that 
>point.

Core error message will not show which resource is failed,
Anyway, I will made this change

Reply via email to