Re: [PATCH v2 2/2] powerpc/powernv: Skip registering log region when CONFIG_PRINTK=n

From: Pranith Kumar
Date: Thu Jan 22 2015 - 23:04:15 EST


On Thu, Jan 22, 2015 at 5:35 PM, Stewart Smith
<stewart@xxxxxxxxxxxxxxxxxx> wrote:
> Pranith Kumar <bobby.prani@xxxxxxxxx> writes:
>> On Thu, Jan 22, 2015 at 12:19 AM, Michael Ellerman <mpe@xxxxxxxxxxxxxx> 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@xxxxxxxxx>
>>>> CC: Michael Ellerman <mpe@xxxxxxxxxxxxxx>
>>>> ---
>>>> 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@xxxxxxxxxxxxxxxxxx>

Thanks!

--
Pranith
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/