Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755871AbcCDAPB (ORCPT ); Thu, 3 Mar 2016 19:15:01 -0500 Received: from mail.kernel.org ([198.145.29.136]:46824 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751010AbcCDAPA (ORCPT ); Thu, 3 Mar 2016 19:15:00 -0500 MIME-Version: 1.0 In-Reply-To: <20160303171640.6a156643.drivshin.allworx@gmail.com> References: <1456954551-31742-1-git-send-email-drivshin.allworx@gmail.com> <20160303171640.6a156643.drivshin.allworx@gmail.com> From: Rob Herring Date: Thu, 3 Mar 2016 18:14:37 -0600 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] of: add 'const' for of_property_*_string*() parameter '*np' To: "David Rivshin (Allworx)" Cc: "devicetree@vger.kernel.org" , Frank Rowand , Grant Likely , "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 Content-Length: 1988 Lines: 46 On Thu, Mar 3, 2016 at 4:16 PM, David Rivshin (Allworx) wrote: > On Thu, 3 Mar 2016 07:52:51 -0600 > Rob Herring wrote: > >> On Wed, Mar 2, 2016 at 3:35 PM, David Rivshin (Allworx) >> wrote: >> > From: David Rivshin >> > >> > The of_property_{read,count,match}_string* family of functions never >> > modify the struct device_node pointer that is passed in, so there is no >> > reason for it to be non-const. Equivalent functions for all other types >> > already take a 'const struct device_node *np'. >> > >> > Signed-off-by: David Rivshin >> > --- >> > >> > MAINTAINTERS says that the appropriate tree is >> > git://git.kernel.org/pub/scm/linux/kernel/git/glikely/linux.git >> >> Yes, we probably need to update that. >> >> > but it looks like that hasn't been updated in a while. So this patch >> > is based off the "for-next" branch of >> > git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git >> > instead. Let me know if you need me to respin from another tree/branch. >> >> You should base off of Linus' tree unless you have some dependency. I > > I was under the impression that the general rule would be to base off > whichever tree it is likely to go through, to make it easier for the > maintainer if nothing else. Is that an incorrect impression, or do you > mean that just for OF/DT changes? This applies to all of the kernel. You should base your work off of Linus' tree unless you know you have a dependency. If you do, then the maintainer needs to know that. It is nice though to test your patches on a maintainer's tree and check if there are conflicts. If trivial, then it's okay to let the maintainer fix up the patch. If not trivial, then you should base on the maintainer's tree. All this matters less with patches than pull requests. Different maintainers have different rules for the stability of their branches also. Rob