Hi Tomasz,
On 20 March 2014 06:05, Tomasz Nowicki tomasz.nowicki@linaro.org wrote:
if (!table_size)table_size = table_header->length;If table_size == 0, that would causes 'entry' and 'table_end' are equal and no entries at all. For that case I would suggest to print error and return.
Hm. tbl_size and table_header->length are not the same. This needs to be revisited.
Changelog is saying that acpi_parse_entries() takes already mapped table and parses entries. IMO, caller should takes care about unmapping table. It can be still parsed using different ID later on.
Although I generally agree that mapping/umapping should be done in the caller, it seems umapping in this manner is baked in APCI core code and I didn't want to change too many things at once without causing unintentional side effects elsewhere.
Thanks, Ashwin