From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755028AbaIHVv2 (ORCPT ); Mon, 8 Sep 2014 17:51:28 -0400 Received: from mail-oa0-f44.google.com ([209.85.219.44]:40849 "EHLO mail-oa0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754306AbaIHVv0 (ORCPT ); Mon, 8 Sep 2014 17:51:26 -0400 MIME-Version: 1.0 In-Reply-To: References: <1409159852-7249-1-git-send-email-mathieu.poirier@linaro.org> <1409159852-7249-9-git-send-email-mathieu.poirier@linaro.org> Date: Mon, 8 Sep 2014 15:51:26 -0600 Message-ID: Subject: Re: [PATCH 08/11 v5] coresight: adding documentation for coresight From: Mathieu Poirier To: Linus Walleij Cc: Thomas Petazzoni , Will Deacon , Russell King - ARM Linux , Greg KH , Arnd Bergmann , John Stultz , Pratik Patel , Vikas Varshney , Al Grant , Jonas Svennebring , James King , Panchaxari Prasannamurthy Tumkur , Kaixu Xia , Marcin Jabrzyk , "r.sengupta@samsung.com" , Robert Marklund , Tony Armitstead , Patch Tracking , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Apologies on the late reply - I was doing a quick review of your earlier comments before getting ready for another submission when I remembered I hadn't address this one. Pls see below. On 3 September 2014 02:43, Linus Walleij wrote: > On Wed, Aug 27, 2014 at 7:17 PM, wrote: > >> From: Mathieu Poirier >> >> Documentation for the coresight framework and drivers. >> >> Signed-off-by: Mathieu Poirier > (...) >> Documentation/trace/coresight.txt | 237 ++++++++++++++++++++++++++++++++++++++ > > I guess you agreed on this location? If it's ARM-specific, > Documentation/arm/coresight.txt > is an alternative. I didn't have prior agreement with anyone before - it was simply dropped there to get the conversation started. Very well, I'll move it to @arm. > >> +The file cstrace.bin can be decompressed using "ptm2human", DS-5 or Trace32. > > Is ptm2human a candidate for the kernel tools/ directory? A fair amount of work would have to be invested in "ptm2human" to make it production ready. > > Maybe not now but later? > > +root@linaro-developer:/sys/kernel/debug/coresight# ls > > Maybe reduce noise here: > > root:/sys/kernel/debug/coresight# ls > > The reader probably does not need to know the name of your > development system. Ok. > > Yours, > Linus Walleij From mboxrd@z Thu Jan 1 00:00:00 1970 From: mathieu.poirier@linaro.org (Mathieu Poirier) Date: Mon, 8 Sep 2014 15:51:26 -0600 Subject: [PATCH 08/11 v5] coresight: adding documentation for coresight In-Reply-To: References: <1409159852-7249-1-git-send-email-mathieu.poirier@linaro.org> <1409159852-7249-9-git-send-email-mathieu.poirier@linaro.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Apologies on the late reply - I was doing a quick review of your earlier comments before getting ready for another submission when I remembered I hadn't address this one. Pls see below. On 3 September 2014 02:43, Linus Walleij wrote: > On Wed, Aug 27, 2014 at 7:17 PM, wrote: > >> From: Mathieu Poirier >> >> Documentation for the coresight framework and drivers. >> >> Signed-off-by: Mathieu Poirier > (...) >> Documentation/trace/coresight.txt | 237 ++++++++++++++++++++++++++++++++++++++ > > I guess you agreed on this location? If it's ARM-specific, > Documentation/arm/coresight.txt > is an alternative. I didn't have prior agreement with anyone before - it was simply dropped there to get the conversation started. Very well, I'll move it to @arm. > >> +The file cstrace.bin can be decompressed using "ptm2human", DS-5 or Trace32. > > Is ptm2human a candidate for the kernel tools/ directory? A fair amount of work would have to be invested in "ptm2human" to make it production ready. > > Maybe not now but later? > > +root at linaro-developer:/sys/kernel/debug/coresight# ls > > Maybe reduce noise here: > > root:/sys/kernel/debug/coresight# ls > > The reader probably does not need to know the name of your > development system. Ok. > > Yours, > Linus Walleij