From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752676AbcDOGiM (ORCPT ); Fri, 15 Apr 2016 02:38:12 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:49770 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751642AbcDOGiL (ORCPT ); Fri, 15 Apr 2016 02:38:11 -0400 Subject: Re: [PATCH 0/2] da8xx clocks (was part of "da8xx USB clocks") To: David Lechner References: <1460661217-7389-1-git-send-email-david@lechnology.com> <570FEEDB.7000109@lechnology.com> CC: Kevin Hilman , Russell King , , From: Sekhar Nori Message-ID: <57108BFA.4080509@ti.com> Date: Fri, 15 Apr 2016 12:06:42 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <570FEEDB.7000109@lechnology.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday 15 April 2016 12:56 AM, David Lechner wrote: > On 04/14/2016 02:13 PM, David Lechner wrote: >> I have separated these patches from the "da8xx USB clocks" series >> since that >> series no longer depends on the clock init being moved. >> >> Tested working on the linux-davinci/master branch with LEGO MINDSTORMS >> EV3. >> >> David Lechner (2): >> ARM: davinci: Move clock init after ioremap. >> ARM: davinci: da850: use clk->set_parent for async3 >> >> arch/arm/mach-davinci/clock.c | 2 +- >> arch/arm/mach-davinci/common.c | 6 --- >> arch/arm/mach-davinci/da830.c | 2 + >> arch/arm/mach-davinci/da850.c | 90 >> ++++++++++++++++++++---------------------- >> arch/arm/mach-davinci/dm355.c | 1 + >> arch/arm/mach-davinci/dm365.c | 1 + >> arch/arm/mach-davinci/dm644x.c | 1 + >> arch/arm/mach-davinci/dm646x.c | 1 + >> 8 files changed, 49 insertions(+), 55 deletions(-) >> > > Hmm... I was just reviewing my previous emails and you said that you > have already applied these two patches, but they don't seem to have made > it into the linux-davinci/master branch with the others. No, I had commented about removing the new clk init function you had introduced so I was waiting for that to be addressed. I was okay with the second patch in this series, but did not push it to master branch as it depends on clk init movement and breaks boot if applied independently. I will review these two patches. Regards, Sekhar