From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mga01.intel.com ([192.55.52.88]:50802 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030626AbaGCJfG (ORCPT ); Thu, 3 Jul 2014 05:35:06 -0400 Message-ID: <1404380071.5102.57.camel@smile.fi.intel.com> (sfid-20140703_113612_070551_2060FB50) Subject: Re: [PATCH 1/6] lib / string_helpers: clean up test suite From: Andy Shevchenko To: Andrew Morton Cc: "John W. Linville" , Johannes Berg , devel@driverdev.osuosl.org, linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman Date: Thu, 03 Jul 2014 12:34:31 +0300 In-Reply-To: <20140702145410.e1885ad85c25a6f4b6cef2c4@linux-foundation.org> References: <1404307229-19186-1-git-send-email-andriy.shevchenko@linux.intel.com> <1404307229-19186-2-git-send-email-andriy.shevchenko@linux.intel.com> <20140702145410.e1885ad85c25a6f4b6cef2c4@linux-foundation.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Wed, 2014-07-02 at 14:54 -0700, Andrew Morton wrote: > On Wed, 2 Jul 2014 16:20:24 +0300 Andy Shevchenko wrote: [] > > + pr_err("Test '%s' failed: flags = %u\n", name, flags); > > + > > + print_hex_dump(KERN_WARNING, "Input: ", DUMP_PREFIX_NONE, 16, 1, > > + in, p, true); > > + print_hex_dump(KERN_WARNING, "Expected: ", DUMP_PREFIX_NONE, 16, 1, > > + out_test, q_test, true); > > + print_hex_dump(KERN_WARNING, "Got: ", DUMP_PREFIX_NONE, 16, 1, > > + out_real, q_real, true); > > Seems strange to mix KERN_ERR and KERN_WARNING. The code's always been > that way, but maybe it can be improved. Will fix this. -- Andy Shevchenko Intel Finland Oy