From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.1 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_PASS,T_DKIMWL_WL_HIGH,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 01A3CC070C3 for ; Wed, 12 Sep 2018 22:49:07 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 91F922133F for ; Wed, 12 Sep 2018 22:49:06 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="1RHln0+Q" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 91F922133F Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728178AbeIMDzl (ORCPT ); Wed, 12 Sep 2018 23:55:41 -0400 Received: from mail.kernel.org ([198.145.29.99]:44860 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727803AbeIMDzl (ORCPT ); Wed, 12 Sep 2018 23:55:41 -0400 Received: from mail-qt0-f171.google.com (mail-qt0-f171.google.com [209.85.216.171]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id BD28821476; Wed, 12 Sep 2018 22:49:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1536792542; bh=dZed1JfvodCQ+q46LOdYB0CZ2OQj4wjBms2zAAlz3rc=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=1RHln0+QB7Ul8w9eWi3e3RPi4UxKZhsIoA3HLYYeZGl5qABggsEbt7aMVFFbM40VN knxSf6IrOEIVD8jXfCVKFxzP+YP86/74hce5ZZFeJLjYUPtA0x+N4zyHo0TlvvV28e cRCKo5Qx/Y9hLHtTwWZrMfVhYprkw5ZFzAkrdvDY= Received: by mail-qt0-f171.google.com with SMTP id x7-v6so3621516qtk.5; Wed, 12 Sep 2018 15:49:02 -0700 (PDT) X-Gm-Message-State: APzg51A87qKhn8umGX5wUScHz6zTUE0/i1azzKhBgoUnAUpBnfP97d1y TO1DRZlrYgrz8JBk+OIBKpIvRCK5TB/PuTDWJQ== X-Google-Smtp-Source: ANB0VdbOnxH03XrbtA6ZEwdd6PWtZGtB7BSL+0rvxrBsbor5/T3hME5mZMnAFv6+zADaIu7iiPlgWo1qL5vTkMZeG0E= X-Received: by 2002:a0c:db87:: with SMTP id m7-v6mr3274626qvk.90.1536792541922; Wed, 12 Sep 2018 15:49:01 -0700 (PDT) MIME-Version: 1.0 References: <20180828154433.5693-1-robh@kernel.org> <20180828154433.5693-7-robh@kernel.org> <20180912121705.010a999d@coco.lan> <20180912180734.37dfafb2@coco.lan> In-Reply-To: <20180912180734.37dfafb2@coco.lan> From: Rob Herring Date: Wed, 12 Sep 2018 17:48:50 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2] staging: Convert to using %pOFn instead of device_node.name To: Mauro Carvalho Chehab Cc: Joe Perches , "linux-kernel@vger.kernel.org" , Ian Arkver , Steve Longerbeam , Philipp Zabel , Mauro Carvalho Chehab , Greg Kroah-Hartman , Linux Media Mailing List , devel@driverdev.osuosl.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 12, 2018 at 4:07 PM Mauro Carvalho Chehab wrote: > > Em Wed, 12 Sep 2018 15:26:48 -0500 > Rob Herring escreveu: > > > +Joe P > > > > On Wed, Sep 12, 2018 at 10:17 AM Mauro Carvalho Chehab > > wrote: > > > > > > Em Tue, 28 Aug 2018 10:44:33 -0500 > > > Rob Herring escreveu: > > > > > > > In preparation to remove the node name pointer from struct device_node, > > > > convert printf users to use the %pOFn format specifier. > > > > > > > > Cc: Steve Longerbeam > > > > Cc: Philipp Zabel > > > > Cc: Mauro Carvalho Chehab > > > > Cc: Greg Kroah-Hartman > > > > Cc: linux-media@vger.kernel.org > > > > Cc: devel@driverdev.osuosl.org > > > > Signed-off-by: Rob Herring > > > > --- > > > > v2: > > > > - fix conditional use of node name vs devname for imx > > > > > > > > drivers/staging/media/imx/imx-media-dev.c | 15 ++++++++++----- > > > > drivers/staging/media/imx/imx-media-of.c | 4 ++-- > > > > drivers/staging/mt7621-eth/mdio.c | 4 ++-- > > > > > > It would be better if you had submitted the staging/media stuff > > > on a separate patch, as they usually go via the media tree. > > > > Sorry, I thought Greg took all of staging. > > No, I usually take media patches on staging. It seems that at least > the IIO subsystem does the same: > > IIO SUBSYSTEM AND DRIVERS > M: Jonathan Cameron > R: Hartmut Knaack > R: Lars-Peter Clausen > R: Peter Meerwald-Stadler > L: linux-iio@vger.kernel.org > T: git git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git > S: Maintained > F: Documentation/ABI/testing/configfs-iio* > F: Documentation/ABI/testing/sysfs-bus-iio* > F: Documentation/devicetree/bindings/iio/ > F: drivers/iio/ > F: drivers/staging/iio/ > F: include/linux/iio/ > F: tools/iio/ Yes, the information is there, but needs human processing. > Anyway, as I said before, I don't have any issues if Greg takes > this specific patch. I know. I'm just thinking about how to improve things. Ideally, what I'd like is a script that spits out a To list for who applies the patch and a Cc list of reviewers. > > A problem with MAINTAINERS is there is no way to tell who applies > > patches for a given path vs. anyone else listed. This frequently > > happens when the maintainer organization doesn't match the directory > > org. If we distinguished this, then it would be quite easy to see when > > you've created a patch that needs to be split to different maintainers > > (or an explanation why it isn't). Whatever happened with splitting up > > MAINTAINERS? If there was a file for each maintainer tree, then it > > would be easier to extract that information. > > Yes, but, on the other hand, get_maintainers.pl would likely > take more time to process, if a patch touches multiple subsystems. > > > > > Or maybe we just need to be stricter with the 'M' vs. 'R' tag and 'M' > > means that is the person who applies the patch. I don't think many > > drivers have their own tree and maintainer except for a few big ones. > > Hmm... just getting a random file under staging/media: > > ./scripts/get_maintainer.pl -f drivers/staging/media/imx/imx-ic.h > Steve Longerbeam (maintainer:MEDIA DRIVERS FOR FREESCALE IMX) > Philipp Zabel (maintainer:MEDIA DRIVERS FOR FREESCALE IMX) > Mauro Carvalho Chehab (maintainer:MEDIA INPUT INFRASTRUCTURE (V4L/DVB)) > Greg Kroah-Hartman (supporter:STAGING SUBSYSTEM) > linux-media@vger.kernel.org (open list:MEDIA DRIVERS FOR FREESCALE IMX) > devel@driverdev.osuosl.org (open list:STAGING SUBSYSTEM) > linux-kernel@vger.kernel.org (open list) > > It seems that the maintainers are already ordered by the tree > depth (placing the most relevant results first). Most relevant in the sense of who reviews and cares about the changes, but not to answer who applies this patch. I could say I just need to ignore supporters and look at the last maintainer, but then that breaks for the case below. I don't think 'maintainer' vs. 'supporter' is very well maintained either. $ scripts/get_maintainer.pl -f drivers/staging/olpc_dcon/ Jens Frederich (maintainer:STAGING - OLPC SECONDARY DISPLAY CONTROLLER (DCON)) Daniel Drake (maintainer:STAGING - OLPC SECONDARY DISPLAY CONTROLLER (DCON)) Jon Nettleton (maintainer:STAGING - OLPC SECONDARY DISPLAY CONTROLLER (DCON)) Greg Kroah-Hartman (supporter:STAGING SUBSYSTEM) devel@driverdev.osuosl.org (open list:STAGING SUBSYSTEM) linux-kernel@vger.kernel.org (open list) Maybe Jon has a tree and applies it. Who knows... You either have to know who are "real" maintainers (the ones applying patches) or go look at git history to see Greg is the only non-author S-o-B. > So, both driver maintainers appear first, then me and finally > Greg (as supporter). > > Mailing lists are also ordered by relevance: media ML, then staging > ML and finally LKML. Another clue, yes, but something you can script? Rob