Messages in this thread | | | Date | Thu, 9 Aug 2018 11:03:18 -0700 | From | Brian Norris <> | Subject | Re: [PATCH v3 0/7] firmware: coreboot: Fix probe and simplify code |
| |
Hi Stepehen,
On Thu, Aug 09, 2018 at 10:17:15AM -0700, Stephen Boyd wrote: > This series reworks the coreboot firmware driver a bit to fix some bugs > and then simplify the code by changing the design to get rid of the > different platform drivers, remap memory with memremap(), and use devm > to simplify error paths. There's some risk in changing to memremap() but > I think that should work out alright. It will either return memory > directly, or fallback to ioremap_cache() as has been done before. > > Changes from v2: > * A bunch more patches > * Fix iounmap missing on error path > * Fix bus getting unregistered on driver unbind and never registered > again > * Request exclusive access to table region > * Pull out table signature check to driver probe > > Changes from v1: > * Split out fixlet for DT based driver from platform driver change > > Cc: Wei-Ning Huang <wnhuang@chromium.org> > Cc: Julius Werner <jwerner@chromium.org> > Cc: Brian Norris <briannorris@chromium.org> > Cc: Samuel Holland <samuel@sholland.org> > > Stephen Boyd (7): > firmware: coreboot: Let OF core populate platform device > firmware: coreboot: Unmap ioregion on failure > firmware: coreboot: Make bus registration symmetric > firmware: coreboot: Collapse platform drivers into bus core > firmware: coreboot: Remap RAM with memremap() instead of ioremap() > firmware: coreboot: Only populate devices in coreboot_table_init() > firmware: coreboot: Request table region for exclusive access
I gave patches 1 and 2 closer review (and individual replies); the remaining patches look good to me too, so for the whole series:
Reviewed-by: Brian Norris <briannorris@chromium.org>
> drivers/firmware/google/Kconfig | 26 ++-- > drivers/firmware/google/Makefile | 2 - > drivers/firmware/google/coreboot_table-acpi.c | 88 ------------ > drivers/firmware/google/coreboot_table-of.c | 82 ----------- > drivers/firmware/google/coreboot_table.c | 135 +++++++++++++----- > drivers/firmware/google/coreboot_table.h | 6 - > 6 files changed, 107 insertions(+), 232 deletions(-) > delete mode 100644 drivers/firmware/google/coreboot_table-acpi.c > delete mode 100644 drivers/firmware/google/coreboot_table-of.c > > -- > Sent by a computer through tubes >
| |