Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753135AbbHWTNr (ORCPT ); Sun, 23 Aug 2015 15:13:47 -0400 Received: from smtp02.smtpout.orange.fr ([80.12.242.124]:18646 "EHLO smtp.smtpout.orange.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752439AbbHWTNq (ORCPT ); Sun, 23 Aug 2015 15:13:46 -0400 X-ME-Helo: belgarion X-ME-Auth: amFyem1pay5yb2JlcnRAb3JhbmdlLmZy X-ME-Date: Sun, 23 Aug 2015 21:13:45 +0200 X-ME-IP: 92.136.77.243 From: Robert Jarzmik To: Ezequiel Garcia Cc: Ezequiel Garcia , David Woodhouse , Brian Norris , linux-mtd@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] mtd: nand: pxa3xx_nand: add register access debug References: <1439323034-8820-1-git-send-email-robert.jarzmik@free.fr> <20150818042426.GA27858@laptop> <874mjwa1w6.fsf@belgarion.home> X-URL: http://belgarath.falguerolles.org/ Date: Sun, 23 Aug 2015 21:09:31 +0200 In-Reply-To: <874mjwa1w6.fsf@belgarion.home> (Robert Jarzmik's message of "Tue, 18 Aug 2015 20:26:49 +0200") Message-ID: <87r3mt3jpw.fsf@belgarion.home> User-Agent: Gnus/5.130008 (Ma Gnus v0.8) Emacs/24.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1281 Lines: 36 Robert Jarzmik writes: > Ezequiel Garcia writes: > >> On 11 Aug 09:57 PM, Robert Jarzmik wrote: >>> Add verbose debug for register accesses. This enables easier debugging >>> by following where and how hardware is stimulated, and how it answers. >>> >> >> I really don't see why we want this patch. It's probably an useful hack to >> use in some cases, but can't see why we would want it in mainline. >> >> Feel free to prove me wrong. > Why not. > > Imagine that there is a bug in the driver, such as the one with the status bits > clearing. You can't reproduce it, and the person who has the hardware to > reproduce it is not skilled enough to do any debug. > > What are the tools and what will you ask this tester to do in order to debug and > solve his problem, as a good driver maintainer ? > > Amongst the drivers I wrote in the past, this approach was the most reliable to > capture traces to debug remotely. Ezequiel, do you agree with my point ? Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/