From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751852AbbCUMCe (ORCPT ); Sat, 21 Mar 2015 08:02:34 -0400 Received: from cpsmtpb-ews07.kpnxchange.com ([213.75.39.10]:62638 "EHLO cpsmtpb-ews07.kpnxchange.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751386AbbCUMCd (ORCPT ); Sat, 21 Mar 2015 08:02:33 -0400 Message-ID: <1426939351.15426.7.camel@x220> Subject: Re: [PATCH v1 03/11] stm class: dummy_stm: Add dummy driver for testing stm class From: Paul Bolle To: Alexander Shishkin Cc: Greg Kroah-Hartman , linux-kernel@vger.kernel.org, mathieu.poirier@linaro.org, peter.lachner@intel.com, norbert.schulz@intel.com, keven.boell@intel.com, yann.fouassier@intel.com, laurent.fert@intel.com, Pratik Patel Date: Sat, 21 Mar 2015 13:02:31 +0100 In-Reply-To: <1426872598-68807-4-git-send-email-alexander.shishkin@linux.intel.com> References: <1426872598-68807-1-git-send-email-alexander.shishkin@linux.intel.com> <1426872598-68807-4-git-send-email-alexander.shishkin@linux.intel.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4 (3.10.4-4.fc20) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 21 Mar 2015 12:02:31.0232 (UTC) FILETIME=[E8B4CC00:01D063CE] X-RcptDomain: vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org All I spotted is the same two nits as in 04/11. On Fri, 2015-03-20 at 19:29 +0200, Alexander Shishkin wrote: > --- a/drivers/hwtracing/stm/Kconfig > +++ b/drivers/hwtracing/stm/Kconfig > +config STM_DUMMY > + tristate "Dummy STM driver" Tab instead of eight spaces, please. > + default N You meant default n as n is a magic symbol. But n is the default anyway, so I think you might as well drop this line. > + help > + This is a simple dummy device that pretends to be an stm device > + and discards your data. Use for stm class testing. > + > + If you don't know what this is, say N. Paul Bolle