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