Attention is currently required from: Arthur Heymans, Cliff Huang, Lance Zhao, Nico Huber, Paul Menzel, Subrata Banik, Subrata Banik, Tim Wawrzynczak.
5 comments:
Patchset:
First of all, don't worry about this comment. I'm merely curious. If you think […]
The CBFS files are specific to board sub types, it's convenient to be able to specify that different subtypes of board have different MTCL data
File src/acpi/acpigen.c:
Patch Set #8, Line 1940: * Generate ACPI AML code for MTCL method.
Looking around, it seems there really are only standardized things […]
Thanks! I ended up just moving this functionality to the mtcl.c file, and completely isolating the MTCL specific functionality. I think it cleaned up the implementation quite a bit.
File src/drivers/wifi/generic/mtcl.c:
Patch Set #8, Line 52: cbfs_map(WIFI_MTCL_CBFS_DEFAULT_FILENAME, &mtcl_bin_len);
Why not use this pointer instead of having the caller provide a buffer (which may be too small btw). […]
Refactored the file to do this. Put all functionality related to reading the file and writing to the SSDT in the function write_mtcl_function.
File src/include/mtcl.h:
Patch Set #8, Line 21: __packed
Not necessary as all entries are u8?
I think I'll leave it as packed only because it's used to do a cast from an array. I think that helps with clarity, let me know if you think it deserves an explicit comment saying as much.
File src/vendorcode/google/chromeos/mtcl.c:
Patch Set #6, Line 59: if (mtcl_bin_len != sizeof(struct wifi_mtcl)) {
> and keep them localized to this file. […]
Once I moved the acpigen bits to mtcl.c, I was able to completely isolate this implementation detail and remove the sizeof and avoid allocating the memory ahead of time. I think this cleans up the implementation in a worthwhile way.
To view, visit change 80170. To unsubscribe, or for help writing mail filters, visit settings.