Messages in this thread | | | From | Pranith Kumar <> | Date | Thu, 22 Jan 2015 23:03:34 -0500 | Subject | Re: [PATCH v2 2/2] powerpc/powernv: Skip registering log region when CONFIG_PRINTK=n |
| |
On Thu, Jan 22, 2015 at 5:35 PM, Stewart Smith <stewart@linux.vnet.ibm.com> wrote: > Pranith Kumar <bobby.prani@gmail.com> writes: >> On Thu, Jan 22, 2015 at 12:19 AM, Michael Ellerman <mpe@ellerman.id.au> wrote: >>> On Wed, 2015-01-21 at 21:26 -0500, Pranith Kumar wrote: >>>> When CONFIG_PRINTK=n, log_buf_addr_get() returns NULL and log_buf_len_get() >>>> return 0. Check for these return values and skip registering the dump buffer. >>>> >>>> Signed-off-by: Pranith Kumar <bobby.prani@gmail.com> >>>> CC: Michael Ellerman <mpe@ellerman.id.au> >>>> --- >>>> arch/powerpc/platforms/powernv/opal.c | 6 ++++++ >>>> 1 file changed, 6 insertions(+) >>> >>> What changed since v1? I don't see anything? >>> >> >> Nothing in this patch but there is a v2 of the first patch in this >> series, so had to resend this patch with a v2. > > You also missed my Reviewed-by :) >
Sincere apologies for the oversight. Maintainers, could you please add this Reviewed-by tag when picking this up?
Reviewed-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Thanks!
-- Pranith
| |