From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sylwester Nawrocki Subject: Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC Date: Mon, 27 Oct 2014 17:47:29 +0100 Message-ID: <544E7721.3040108@samsung.com> References: <1411361228-18279-1-git-send-email-a.kesavan@samsung.com> <541FE357.5090000@gmail.com> <542AD2AE.6060109@gmail.com> <01a501cfe1e5$759fc8c0$60df5a40$@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-reply-to: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Abhilash Kesavan Cc: devicetree , linux-samsung-soc , Catalin Marinas , Tomasz Figa , Kukjin Kim , Naveen Krishna , linux-arm-kernel List-Id: linux-samsung-soc@vger.kernel.org Hi Abhilash, On 24/10/14 16:32, Abhilash Kesavan wrote: > On Wed, Oct 8, 2014 at 11:18 AM, Abhilash Kesavan > wrote: >> On Tue, Oct 7, 2014 at 9:46 AM, Kukjin Kim wrote: >>> Tomasz Figa wrote: >>>> On 30.09.2014 17:12, Abhilash Kesavan wrote: >>>>> On Mon, Sep 22, 2014 at 2:22 PM, Tomasz Figa wrote: >>>>>> >>>>>> The clock patches look good to me, but since they are doing quite a lot >>>>>> of code moving I'd prefer to take them through clk tree. Based on the >>>>>> fact that there are no code dependencies between clock patches and >>>>>> remaining ones and Exynos7 is a new material for 3.18, I'm inclined to >>>>>> apply them to my tree if nobody minds. >>>>> >>>>> Will you be picking up the clock changes soon ? >>>> >>>> I'd like to do so. Kukjin, since clock changes are a part of this >>>> series, might I have your Ack for them to be applied separately? >>>> >>> Sure, I'm fine so please go ahead with my ack on the changes. >>> >>> Acked-by: Kukjin Kim >>> >>> Thanks for your asking. >> >> Can you please pick up the clock changes now that Kukjin has ack'ed it. > > Sylwester, you mentioned in another thread that Tomasz might not be > available for the next few days. Could you please consider applying > the clock changes in this series to samsung clk-next if they look OK ? > > Please take a look at clock changes at > http://www.spinics.net/lists/arm-kernel/msg370673.html too. I've picked up patches from the $subject series. But please see my comment the the second series you've pointed out with the link above. If you decide to rearrange changes to exynos7-clk.h header file among those two series I will drop that 4 patches. Perhaps you could gather all pending exynos7 clk patches, fix any issues and send to me in one series ? I'd like to avoid modifying indices assigned to specific clocks in subsequent patches, DT binding should be defined in advance and ideally be specifying exact indices for each possible clock. -- Thanks, Sylwester From mboxrd@z Thu Jan 1 00:00:00 1970 From: s.nawrocki@samsung.com (Sylwester Nawrocki) Date: Mon, 27 Oct 2014 17:47:29 +0100 Subject: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC In-Reply-To: References: <1411361228-18279-1-git-send-email-a.kesavan@samsung.com> <541FE357.5090000@gmail.com> <542AD2AE.6060109@gmail.com> <01a501cfe1e5$759fc8c0$60df5a40$@kernel.org> Message-ID: <544E7721.3040108@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Abhilash, On 24/10/14 16:32, Abhilash Kesavan wrote: > On Wed, Oct 8, 2014 at 11:18 AM, Abhilash Kesavan > wrote: >> On Tue, Oct 7, 2014 at 9:46 AM, Kukjin Kim wrote: >>> Tomasz Figa wrote: >>>> On 30.09.2014 17:12, Abhilash Kesavan wrote: >>>>> On Mon, Sep 22, 2014 at 2:22 PM, Tomasz Figa wrote: >>>>>> >>>>>> The clock patches look good to me, but since they are doing quite a lot >>>>>> of code moving I'd prefer to take them through clk tree. Based on the >>>>>> fact that there are no code dependencies between clock patches and >>>>>> remaining ones and Exynos7 is a new material for 3.18, I'm inclined to >>>>>> apply them to my tree if nobody minds. >>>>> >>>>> Will you be picking up the clock changes soon ? >>>> >>>> I'd like to do so. Kukjin, since clock changes are a part of this >>>> series, might I have your Ack for them to be applied separately? >>>> >>> Sure, I'm fine so please go ahead with my ack on the changes. >>> >>> Acked-by: Kukjin Kim >>> >>> Thanks for your asking. >> >> Can you please pick up the clock changes now that Kukjin has ack'ed it. > > Sylwester, you mentioned in another thread that Tomasz might not be > available for the next few days. Could you please consider applying > the clock changes in this series to samsung clk-next if they look OK ? > > Please take a look at clock changes at > http://www.spinics.net/lists/arm-kernel/msg370673.html too. I've picked up patches from the $subject series. But please see my comment the the second series you've pointed out with the link above. If you decide to rearrange changes to exynos7-clk.h header file among those two series I will drop that 4 patches. Perhaps you could gather all pending exynos7 clk patches, fix any issues and send to me in one series ? I'd like to avoid modifying indices assigned to specific clocks in subsequent patches, DT binding should be defined in advance and ideally be specifying exact indices for each possible clock. -- Thanks, Sylwester