From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [PATCH 1/7] clk: gxbb: expose USB clocks Date: Wed, 7 Sep 2016 15:14:03 -0700 Message-ID: <20160907221403.GA13062@codeaurora.org> References: <20160904213152.25837-1-martin.blumenstingl@googlemail.com> <20160904213152.25837-2-martin.blumenstingl@googlemail.com> <20160907003355.GQ12510@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Martin Blumenstingl Cc: linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, jbrunet-rdvid1DuHRBWk0Htik3J/w@public.gmane.org, johnyoun-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, kishon-l0cyMroinI0@public.gmane.org, khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org, carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, mturquette-rdvid1DuHRBWk0Htik3J/w@public.gmane.org, will.deacon-5wv7dgnIgG8@public.gmane.org, catalin.marinas-5wv7dgnIgG8@public.gmane.org, gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org List-Id: devicetree@vger.kernel.org On 09/07, Martin Blumenstingl wrote: > On Wed, Sep 7, 2016 at 2:33 AM, Stephen Boyd wrote: > > On 09/04, Martin Blumenstingl wrote: > >> USB0_DDR_BRIDGE and USB1_DDR_BRIDGE1 are needed for the related > >> dwc2 usb controller. USB, USB0 and USB1 are needed for the PHYs. > >> Expose these clocks to DT and comment out in clk driver. > >> > >> Signed-off-by: Jerome Brunet > >> Signed-off-by: Martin Blumenstingl > > > > Is authorship correct on this patch? Did Jerome author it > > instead? > We (Jerome and I) have both worked on this patch, that's why you have > two signed-off-by's. > Or is this simply about the order (author = from address should be > listed first)? Yes. Typically author is listed first in the signed-off-by chain. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in 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: Date: Wed, 7 Sep 2016 15:14:03 -0700 From: Stephen Boyd To: Martin Blumenstingl Cc: linux-clk@vger.kernel.org, linux-usb@vger.kernel.org, linux-amlogic@lists.infradead.org, jbrunet@baylibre.com, johnyoun@synopsys.com, kishon@ti.com, khilman@baylibre.com, carlo@caione.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, mturquette@baylibre.com, will.deacon@arm.com, catalin.marinas@arm.com, gregkh@linuxfoundation.org, mark.rutland@arm.com, robh+dt@kernel.org Subject: Re: [PATCH 1/7] clk: gxbb: expose USB clocks Message-ID: <20160907221403.GA13062@codeaurora.org> References: <20160904213152.25837-1-martin.blumenstingl@googlemail.com> <20160904213152.25837-2-martin.blumenstingl@googlemail.com> <20160907003355.GQ12510@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: List-ID: On 09/07, Martin Blumenstingl wrote: > On Wed, Sep 7, 2016 at 2:33 AM, Stephen Boyd wrote: > > On 09/04, Martin Blumenstingl wrote: > >> USB0_DDR_BRIDGE and USB1_DDR_BRIDGE1 are needed for the related > >> dwc2 usb controller. USB, USB0 and USB1 are needed for the PHYs. > >> Expose these clocks to DT and comment out in clk driver. > >> > >> Signed-off-by: Jerome Brunet > >> Signed-off-by: Martin Blumenstingl > > > > Is authorship correct on this patch? Did Jerome author it > > instead? > We (Jerome and I) have both worked on this patch, that's why you have > two signed-off-by's. > Or is this simply about the order (author = from address should be > listed first)? Yes. Typically author is listed first in the signed-off-by chain. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project From mboxrd@z Thu Jan 1 00:00:00 1970 From: sboyd@codeaurora.org (Stephen Boyd) Date: Wed, 7 Sep 2016 15:14:03 -0700 Subject: [PATCH 1/7] clk: gxbb: expose USB clocks In-Reply-To: References: <20160904213152.25837-1-martin.blumenstingl@googlemail.com> <20160904213152.25837-2-martin.blumenstingl@googlemail.com> <20160907003355.GQ12510@codeaurora.org> Message-ID: <20160907221403.GA13062@codeaurora.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 09/07, Martin Blumenstingl wrote: > On Wed, Sep 7, 2016 at 2:33 AM, Stephen Boyd wrote: > > On 09/04, Martin Blumenstingl wrote: > >> USB0_DDR_BRIDGE and USB1_DDR_BRIDGE1 are needed for the related > >> dwc2 usb controller. USB, USB0 and USB1 are needed for the PHYs. > >> Expose these clocks to DT and comment out in clk driver. > >> > >> Signed-off-by: Jerome Brunet > >> Signed-off-by: Martin Blumenstingl > > > > Is authorship correct on this patch? Did Jerome author it > > instead? > We (Jerome and I) have both worked on this patch, that's why you have > two signed-off-by's. > Or is this simply about the order (author = from address should be > listed first)? Yes. Typically author is listed first in the signed-off-by chain. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project From mboxrd@z Thu Jan 1 00:00:00 1970 From: sboyd@codeaurora.org (Stephen Boyd) Date: Wed, 7 Sep 2016 15:14:03 -0700 Subject: [PATCH 1/7] clk: gxbb: expose USB clocks In-Reply-To: References: <20160904213152.25837-1-martin.blumenstingl@googlemail.com> <20160904213152.25837-2-martin.blumenstingl@googlemail.com> <20160907003355.GQ12510@codeaurora.org> Message-ID: <20160907221403.GA13062@codeaurora.org> To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org On 09/07, Martin Blumenstingl wrote: > On Wed, Sep 7, 2016 at 2:33 AM, Stephen Boyd wrote: > > On 09/04, Martin Blumenstingl wrote: > >> USB0_DDR_BRIDGE and USB1_DDR_BRIDGE1 are needed for the related > >> dwc2 usb controller. USB, USB0 and USB1 are needed for the PHYs. > >> Expose these clocks to DT and comment out in clk driver. > >> > >> Signed-off-by: Jerome Brunet > >> Signed-off-by: Martin Blumenstingl > > > > Is authorship correct on this patch? Did Jerome author it > > instead? > We (Jerome and I) have both worked on this patch, that's why you have > two signed-off-by's. > Or is this simply about the order (author = from address should be > listed first)? Yes. Typically author is listed first in the signed-off-by chain. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project