From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751934AbcGFHgM (ORCPT ); Wed, 6 Jul 2016 03:36:12 -0400 Received: from mout.kundenserver.de ([212.227.126.131]:57530 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750760AbcGFHgK (ORCPT ); Wed, 6 Jul 2016 03:36:10 -0400 From: Arnd Bergmann To: John Stultz Cc: Olof Johansson , lkml , "arm@kernel.org" , Michael Turquette , Stephen Boyd , Rob Herring , Pawel Moll , Wei Xu , Guodong Xu , Zhangfei Gao Subject: Re: [PATCH 0/2 v3] Add pl031 RTC support for Hi6220 Date: Wed, 06 Jul 2016 09:38:59 +0200 Message-ID: <3989688.6Bh7v9havA@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-22-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: References: <1467247725-3665-1-git-send-email-john.stultz@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:fEdrHnLy8EM9I1ywSMdu80f66iA5Y/NNEfpGSXZSAl7XGGim+a3 j/w9nh4U3S1vop3bgeJFvC66XX7yTFRBEWyU2dky/H2pn0MMnaQGt3qLgCDAqd3oRJl5qAD nZTbcDT+E0/7xfgmp/M8JRUBKklb2SiHZHiN8ZpYpaCCxApeD8R0t0qwwVBkGfuNt1VyKBB 9JzbR4xTdyuf5FcBtaAeA== X-UI-Out-Filterresults: notjunk:1;V01:K0:NtgZn4b97XA=:KFGFAzYfbxvS4PVJ7M90hk C5U4iEmbUme+kZuUOUjhE2x6/Bkm6OqPD9YTHXNYiLkNNMeR8HScPBPldib8sd/uzM7uCNn7c CZEq939CWWjdjd+/Pp1+dYJhYWBOLgGZlX4mWhgeqL1k9Uvjq5CBFO+8o89KZPY1EGNrEE32m cvgXj9ksaGXTpQbgC06j1FDzkarcmEUBOvs3DzWVLM8cYWyIXhWqQsDPIWVBWj43lN4lU5Wor jx6LC82/L1qXAi6Ql8FM1CKNB8ZBYgD2tGwGCB1htjEhRuN2pxpLv+x9D4O2CsA76c2y+D5Ik VAc3s74UEhhVcBhVN6EMaxXUX7GXrm+p2AXTbmxiGyjJGETztbWOm82ZbqPdF+kTWtdxdN3pc 4Z1sF9Kl/pWG/xTO5L4we/ne3AiHHc7O+abwKt8lCc8CMA+REESX7BfHxN6JIPdWjRgt6ToBe x3vNGvDNC0GEgEJqgZ8XrtKTRb2ioU2U+ODuopmJghvcz21P/yDLr5oBbduTFqkXL3zeA5h6v BnDP/2dUa0JcurOE5PcibTyIjh0d2kRF6Z55dput0vpJz+jq41I2SnWngUUxKM1B5auk6kBfT Wt1ykwYngu9gm3JOddBe5ot6i/Yup7Jr5HQtIVQoPTiaUV4oF0clexn9kjfdTZO999tFm7R4K 6Q7HS2HSo7nzNTq5k7oPaAYKusN4e4o4QOHoQrjWM4hj0XzfE4lYdJ8xFwuGNxpe53ry1Mt5k R9HtWfU4YaPck3FE Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday, July 6, 2016 12:20:15 AM CEST John Stultz wrote: > On Wed, Jul 6, 2016 at 12:04 AM, Olof Johansson wrote: > > On Tue, Jul 5, 2016 at 11:55 PM, John Stultz wrote: > >> On Tue, Jul 5, 2016 at 10:22 PM, Olof Johansson wrote: > >>> On Wed, Jun 29, 2016 at 05:48:43PM -0700, John Stultz wrote: > >>>> This patchset enables the pl031 RTC on the Hi6220 SoC. > >>>> > >>>> I'd like to submit it to be merged. > >>>> > >>>> Wei has acked the second patch (modulo a whitespace fix which > >>>> I've included in this v3), so it seems like both could go > >>>> through the clk tree. > >>>> > >>>> But Wei also seemed open to pulling in a clk tree branch > >>>> as it goes through arm-soc. > >>>> > >>>> Michael/Stephen: If there's no other objections, could you > >>>> queue the first patch and make it avilable via the branch for > >>>> Wei, or just take both patches? > >>> > >>> I happen to dread these kind of patchsets these days. There's added > >>> dependencies across trees just because a defined name for the clock > >>> number is added to a header file. > >>> > >>> I much prefer to use numerical clocks for one release, and then once > >>> everything is in, switch over to the defines in the DTS. > >>> > >>> That way there are no dependencies, no need to setup a shared branch > >>> for a simple 3-line patch, etc. > >>> > >>> So, mind respinning the DTS piece? > >> > >> Huh.. > > > > Sorry if it appeared random, I've complained about it for a while to > > submaintainers. > > No.. I get it, the cross-maintainer shared branch is complex enough to > want to avoid. I figured it would be easier to just take a maintainer > acked patch in via the clk tree, but its not my tree, so I'll leave it > to you maintainers to resolve. The question this raises is why that clock was missed the first time around. I'd suggest whoever owns the clock driver can go through the documentation again and look for others that may have been missed, then send a patch to the driver to add *all* the missing ones for the merge window, and one release later we add the driver depending on previously unknown clocks. Arnd