>> +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 ��.n��������+%������w��{.n������_���v��z����n�r������&��z�ޗ�zf���h���~����������_��+v���)ߣ�