From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751234AbeAYGsg (ORCPT ); Thu, 25 Jan 2018 01:48:36 -0500 Received: from mail-it0-f68.google.com ([209.85.214.68]:39876 "EHLO mail-it0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750734AbeAYGsd (ORCPT ); Thu, 25 Jan 2018 01:48:33 -0500 X-Google-Smtp-Source: AH8x227iE15dzbx+ohVlpbwoyQPVPaA8A1dyr95Dl15WyQBbo+8uu2b95gLHnZU/HxUKTN6Sihl0AQ== Subject: Re: [RFC PATCH v2 0/1] of: easier debugging for node life cycle issues To: Wolfram Sang , devicetree@vger.kernel.org Cc: Tyrel Datwyler , Geert Uytterhoeven , linux-renesas-soc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Rob Herring , Steven Rostedt , linux-kernel@vger.kernel.org References: <20180121143117.19805-1-wsa+renesas@sang-engineering.com> From: Frank Rowand Message-ID: <9c03edf1-564b-dd40-bbdf-e21e499869a6@gmail.com> Date: Wed, 24 Jan 2018 22:48:31 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <20180121143117.19805-1-wsa+renesas@sang-engineering.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/21/18 06:31, Wolfram Sang wrote: > I got a bug report for a DT node refcounting problem in the I2C subsystem. This > patch was a huge help in validating the bug report and the proposed solution. > So, I thought I bring it to attention again. Thanks Tyrel, for the initial > work! > > Note that I did not test the dynamic updates, only of_node_{get|put} so far. I > read that Tyrel checked dynamic updates extensively with this patch. And since > DT overlays are also used within our Renesas dev team, this will help there, as > well. It's been nine months since version 1. If you are going to include the dynamic updates part of the patch then please test them. > Tested on a Renesas Salvator-XS board (R-Car H3). > > Changes since RFC v1: > * rebased to v4.15-rc8 > * fixed commit abbrev and one of the sysfs paths in commit desc > * removed trailing space and fixed pointer declaration in code > > I consider all the remaining checkpatch issues irrelevant for this patch. I am OK with the line length warnings in this patch. Why can't the macro error be fixed? A file entry needs to be added to MAINTAINERS. > > So what about applying it? > > Kind regards, > > Wolfram > > > Tyrel Datwyler (1): > of: introduce event tracepoints for dynamic device_node lifecyle > > drivers/of/dynamic.c | 32 ++++++---------- > include/trace/events/of.h | 93 +++++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 105 insertions(+), 20 deletions(-) > create mode 100644 include/trace/events/of.h >