>> +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