All of lore.kernel.org
 help / color / mirror / Atom feed
From: nsekhar@ti.com (Sekhar Nori)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] da8xx clocks (was part of "da8xx USB clocks")
Date: Fri, 15 Apr 2016 15:39:42 +0530	[thread overview]
Message-ID: <5710BDE6.6000901@ti.com> (raw)
In-Reply-To: <57108BFA.4080509@ti.com>

On Friday 15 April 2016 12:06 PM, Sekhar Nori wrote:
> 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.

Looks good to me. I will wait couple more days before applying for any
more review.

Regards,
Sekhar

WARNING: multiple messages have this Message-ID (diff)
From: Sekhar Nori <nsekhar@ti.com>
To: David Lechner <david@lechnology.com>
Cc: Russell King <linux@arm.linux.org.uk>,
	<linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	Kevin Hilman <khilman@kernel.org>
Subject: Re: [PATCH 0/2] da8xx clocks (was part of "da8xx USB clocks")
Date: Fri, 15 Apr 2016 15:39:42 +0530	[thread overview]
Message-ID: <5710BDE6.6000901@ti.com> (raw)
In-Reply-To: <57108BFA.4080509@ti.com>

On Friday 15 April 2016 12:06 PM, Sekhar Nori wrote:
> 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.

Looks good to me. I will wait couple more days before applying for any
more review.

Regards,
Sekhar

  reply	other threads:[~2016-04-15 10:09 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-14 19:13 [PATCH 0/2] da8xx clocks (was part of "da8xx USB clocks") David Lechner
2016-04-14 19:13 ` David Lechner
2016-04-14 19:13 ` [PATCH 1/2] ARM: davinci: Move clock init after ioremap David Lechner
2016-04-14 19:13   ` David Lechner
2016-04-14 19:13 ` [PATCH 2/2] ARM: davinci: da850: use clk->set_parent for async3 David Lechner
2016-04-14 19:13   ` David Lechner
2016-04-16 19:31   ` David Lechner
2016-04-16 19:31     ` David Lechner
2016-04-18  4:55     ` Sekhar Nori
2016-04-18  4:55       ` Sekhar Nori
2016-04-14 19:26 ` [PATCH 0/2] da8xx clocks (was part of "da8xx USB clocks") David Lechner
2016-04-14 19:26   ` David Lechner
2016-04-15  6:36   ` Sekhar Nori
2016-04-15  6:36     ` Sekhar Nori
2016-04-15 10:09     ` Sekhar Nori [this message]
2016-04-15 10:09       ` Sekhar Nori
2016-04-15 16:21     ` David Lechner
2016-04-15 16:21       ` David Lechner
2016-04-27 13:39 ` Sekhar Nori
2016-04-27 13:39   ` Sekhar Nori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5710BDE6.6000901@ti.com \
    --to=nsekhar@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.