From mboxrd@z Thu Jan 1 00:00:00 1970 From: heiko@sntech.de (Heiko =?ISO-8859-1?Q?St=FCbner?=) Date: Wed, 26 Nov 2014 18:43:40 +0100 Subject: [PATCH 1/2] clk: rockchip: add binding ID for DMC (memory controller) clocks on rk3288 In-Reply-To: References: <1416960783-25119-1-git-send-email-dianders@chromium.org> <1717801.sWHeWsmqUi@diego> Message-ID: <3545025.lFcuKWk0o9@diego> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Am Dienstag, 25. November 2014, 16:49:08 schrieb Doug Anderson: > Heiko, > > On Tue, Nov 25, 2014 at 4:45 PM, Heiko St?bner wrote: > > Am Dienstag, 25. November 2014, 16:13:02 schrieb Doug Anderson: > >> From: Jeff Chen > >> > >> The DMC clocks need to be turned off at runtime, so we should have IDs > >> so we can export them. > >> > >> Signed-off-by: Jeff Chen > >> [dianders: split into two patches; adjusted commit msg] > >> Signed-off-by: Doug Anderson > > > > both patches look ok to me and I'll probably pick them up tomorrow. > > OK, thanks! I've now added both patches to my clk branch. > > We have currently three clock patches/series adding new clock ids [this > > one, the mmc phases and Sonny's i2s_clkout] and I'm trying to dertermine > > if I need a shared branch for those. > > > > Currently it doesn't look like the others will provide dts patches using > > the clocks in time, so it looks like I can skip the shared branch and we > > can use the clock ids regularly after the merge-window. Some true for > > your clocks? > Yup, no problem for me. I provided this as two patches to you just in > case it was convenient, since I saw that as feedback in previous > submissions. Yep, I think it is the sanest of doing it in one way all the time, so we have the option of easily creating shared branches depending on what is needed. Heiko From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko =?ISO-8859-1?Q?St=FCbner?= Subject: Re: [PATCH 1/2] clk: rockchip: add binding ID for DMC (memory controller) clocks on rk3288 Date: Wed, 26 Nov 2014 18:43:40 +0100 Message-ID: <3545025.lFcuKWk0o9@diego> References: <1416960783-25119-1-git-send-email-dianders@chromium.org> <1717801.sWHeWsmqUi@diego> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Doug Anderson Cc: jeff chen , Mike Turquette , Derek Basehore , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "open list:ARM/Rockchip SoC..." , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Kever Yang , Mark yao , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: devicetree@vger.kernel.org Am Dienstag, 25. November 2014, 16:49:08 schrieb Doug Anderson: > Heiko, >=20 > On Tue, Nov 25, 2014 at 4:45 PM, Heiko St=FCbner wr= ote: > > Am Dienstag, 25. November 2014, 16:13:02 schrieb Doug Anderson: > >> From: Jeff Chen > >>=20 > >> The DMC clocks need to be turned off at runtime, so we should have= IDs > >> so we can export them. > >>=20 > >> Signed-off-by: Jeff Chen > >> [dianders: split into two patches; adjusted commit msg] > >> Signed-off-by: Doug Anderson > >=20 > > both patches look ok to me and I'll probably pick them up tomorrow. >=20 > OK, thanks! I've now added both patches to my clk branch. > > We have currently three clock patches/series adding new clock ids [= this > > one, the mmc phases and Sonny's i2s_clkout] and I'm trying to derte= rmine > > if I need a shared branch for those. > >=20 > > Currently it doesn't look like the others will provide dts patches = using > > the clocks in time, so it looks like I can skip the shared branch a= nd we > > can use the clock ids regularly after the merge-window. Some true f= or > > your clocks? > Yup, no problem for me. I provided this as two patches to you just i= n > case it was convenient, since I saw that as feedback in previous > submissions. Yep, I think it is the sanest of doing it in one way all the time, so w= e have=20 the option of easily creating shared branches depending on what is need= ed. Heiko -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753174AbaKZRkM (ORCPT ); Wed, 26 Nov 2014 12:40:12 -0500 Received: from gloria.sntech.de ([95.129.55.99]:46520 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751688AbaKZRkJ convert rfc822-to-8bit (ORCPT ); Wed, 26 Nov 2014 12:40:09 -0500 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: Doug Anderson Cc: jeff chen , Mike Turquette , Derek Basehore , "linux-arm-kernel@lists.infradead.org" , "open list:ARM/Rockchip SoC..." , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Kever Yang , Mark yao , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/2] clk: rockchip: add binding ID for DMC (memory controller) clocks on rk3288 Date: Wed, 26 Nov 2014 18:43:40 +0100 Message-ID: <3545025.lFcuKWk0o9@diego> User-Agent: KMail/4.14.1 (Linux/3.16-3-amd64; KDE/4.14.2; x86_64; ; ) In-Reply-To: References: <1416960783-25119-1-git-send-email-dianders@chromium.org> <1717801.sWHeWsmqUi@diego> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="iso-8859-1" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Dienstag, 25. November 2014, 16:49:08 schrieb Doug Anderson: > Heiko, > > On Tue, Nov 25, 2014 at 4:45 PM, Heiko Stübner wrote: > > Am Dienstag, 25. November 2014, 16:13:02 schrieb Doug Anderson: > >> From: Jeff Chen > >> > >> The DMC clocks need to be turned off at runtime, so we should have IDs > >> so we can export them. > >> > >> Signed-off-by: Jeff Chen > >> [dianders: split into two patches; adjusted commit msg] > >> Signed-off-by: Doug Anderson > > > > both patches look ok to me and I'll probably pick them up tomorrow. > > OK, thanks! I've now added both patches to my clk branch. > > We have currently three clock patches/series adding new clock ids [this > > one, the mmc phases and Sonny's i2s_clkout] and I'm trying to dertermine > > if I need a shared branch for those. > > > > Currently it doesn't look like the others will provide dts patches using > > the clocks in time, so it looks like I can skip the shared branch and we > > can use the clock ids regularly after the merge-window. Some true for > > your clocks? > Yup, no problem for me. I provided this as two patches to you just in > case it was convenient, since I saw that as feedback in previous > submissions. Yep, I think it is the sanest of doing it in one way all the time, so we have the option of easily creating shared branches depending on what is needed. Heiko