On Tue, Dec 06, 2016 at 03:50:52PM -0800, Laura Abbott wrote: > > __pa_symbol is technically the marcro that should be used for kernel > symbols. Switch to this as a pre-requisite for DEBUG_VIRTUAL which > will do bounds checking. > > Tested-by: James Morse <james.mo...@arm.com> > Signed-off-by: Laura Abbott <labb...@redhat.com>
This looks good to me; I have a (very minor) nit below, but either way: Reviewed-by: Mark Rutland <mark.rutl...@arm.com> Tested-by: Mark Rutland <mark.rutl...@arm.com> > diff --git a/arch/arm64/kernel/acpi_parking_protocol.c > b/arch/arm64/kernel/acpi_parking_protocol.c > index a32b401..ca880ce 100644 > --- a/arch/arm64/kernel/acpi_parking_protocol.c > +++ b/arch/arm64/kernel/acpi_parking_protocol.c > @@ -18,6 +18,7 @@ > */ > #include <linux/acpi.h> > #include <linux/types.h> > +#include <linux/mm.h> Nit: please keep includes alphabetically ordered, at least where they're ordered today. Some files are already a mess, but it would be nice to keep the well-ordered ones ordered. Thanks, Mark.