[PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Valentine Barshak]

This fixes a number of issues with CPG MSTP clock array handling.

Changes in V2:
* removed brackets from the one-line for loop

Valentine Barshak (2):
  clk: shmobile: Fix MSTP clock index
  clk: shmobile: Fix MSTP clock array initialization

 drivers/clk/shmobile/clk-mstp.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

-- 
1.8.3.1

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Mike Turquette]

Quoting Valentine Barshak (2013-12-28 04:09:07)
> This fixes a number of issues with CPG MSTP clock array handling.

Thanks for the quick respin. Taken into clk-next.

Regards,
Mike

> 
> Changes in V2:
> * removed brackets from the one-line for loop
> 
> Valentine Barshak (2):
>   clk: shmobile: Fix MSTP clock index
>   clk: shmobile: Fix MSTP clock array initialization
> 
>  drivers/clk/shmobile/clk-mstp.c | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)
> 
> -- 
> 1.8.3.1
> 

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Valentine]

On 12/30/2013 01:54 AM, Mike Turquette wrote:
> Quoting Valentine Barshak (2013-12-28 04:09:07)
>> This fixes a number of issues with CPG MSTP clock array handling.
>
> Thanks for the quick respin. Taken into clk-next.

Sorry, I don't see it in https://git.linaro.org/people/mike.turquette/linux.git/shortlog/refs/heads/clk-next
Is this the right place to look for?

>
> Regards,
> Mike

Thanks,
Val.

>
>>
>> Changes in V2:
>> * removed brackets from the one-line for loop
>>
>> Valentine Barshak (2):
>>    clk: shmobile: Fix MSTP clock index
>>    clk: shmobile: Fix MSTP clock array initialization
>>
>>   drivers/clk/shmobile/clk-mstp.c | 12 ++++++++----
>>   1 file changed, 8 insertions(+), 4 deletions(-)
>>
>> --
>> 1.8.3.1
>>

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Simon Horman]

[ CC Ben Dooks ]

On Mon, Jan 06, 2014 at 08:33:17PM +0400, Valentine wrote:
> On 12/30/2013 01:54 AM, Mike Turquette wrote:
> >Quoting Valentine Barshak (2013-12-28 04:09:07)
> >>This fixes a number of issues with CPG MSTP clock array handling.
> >
> >Thanks for the quick respin. Taken into clk-next.
> 
> Sorry, I don't see it in https://git.linaro.org/people/mike.turquette/linux.git/shortlog/refs/heads/clk-next
> Is this the right place to look for?

Hi Mike,

I'm not seeing them there either.

Also, I would like to pull these changes into my "devel" branch
as they fix problems that developers are hitting. If I could use
an updated clk-next-shmobile branch that includes these patches
and commit ids that won't change I would be most grateful.

Thanks.

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Ben Dooks]

On 29/12/13 21:54, Mike Turquette wrote:
> Quoting Valentine Barshak (2013-12-28 04:09:07)
>> This fixes a number of issues with CPG MSTP clock array handling.
>
> Thanks for the quick respin. Taken into clk-next.

Hmm, couldn't see them in your tree.
I've test both of these and they fix issues I've seen on the lager.

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Ben Dooks]

On 28/12/13 12:09, Valentine Barshak wrote:
> This fixes a number of issues with CPG MSTP clock array handling.
>
> Changes in V2:
> * removed brackets from the one-line for loop
>
> Valentine Barshak (2):
>    clk: shmobile: Fix MSTP clock index
>    clk: shmobile: Fix MSTP clock array initialization
>
>   drivers/clk/shmobile/clk-mstp.c | 12 ++++++++----
>   1 file changed, 8 insertions(+), 4 deletions(-)

Both look fine to me, both can have:
Tested-by: Ben Dooks <ben.dooks@codethink.co.uk>


-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Mike Turquette]

Quoting Simon Horman (2014-01-09 17:07:01)
> [ CC Ben Dooks ]
> 
> On Mon, Jan 06, 2014 at 08:33:17PM +0400, Valentine wrote:
> > On 12/30/2013 01:54 AM, Mike Turquette wrote:
> > >Quoting Valentine Barshak (2013-12-28 04:09:07)
> > >>This fixes a number of issues with CPG MSTP clock array handling.
> > >
> > >Thanks for the quick respin. Taken into clk-next.
> > 
> > Sorry, I don't see it in https://git.linaro.org/people/mike.turquette/linux.git/shortlog/refs/heads/clk-next
> > Is this the right place to look for?
> 
> Hi Mike,
> 
> I'm not seeing them there either.
> 
> Also, I would like to pull these changes into my "devel" branch
> as they fix problems that developers are hitting. If I could use
> an updated clk-next-shmobile branch that includes these patches
> and commit ids that won't change I would be most grateful.

Hi all,

Thanks for your persistence. This was sitting in a local branch for a
while and I forgot to merge it into clk-next. I've updated the
clk-next-shmobile branch to include these patches and I have applied all
Acks and Tested-bys. That branch includes some other patches as well but
it shouldn't be a problem for a development branch.

I've also merged clk-next-shmobile into clk-next and pushed out to my
Linaro mirror.

Regards,
Mike

> 
> Thanks.

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Ben Dooks]

On 14/01/14 19:44, Mike Turquette wrote:
> Quoting Simon Horman (2014-01-09 17:07:01)
>> [ CC Ben Dooks ]
>>
>> On Mon, Jan 06, 2014 at 08:33:17PM +0400, Valentine wrote:
>>> On 12/30/2013 01:54 AM, Mike Turquette wrote:
>>>> Quoting Valentine Barshak (2013-12-28 04:09:07)
>>>>> This fixes a number of issues with CPG MSTP clock array handling.
>>>>
>>>> Thanks for the quick respin. Taken into clk-next.
>>>
>>> Sorry, I don't see it in https://git.linaro.org/people/mike.turquette/linux.git/shortlog/refs/heads/clk-next
>>> Is this the right place to look for?
>>
>> Hi Mike,
>>
>> I'm not seeing them there either.
>>
>> Also, I would like to pull these changes into my "devel" branch
>> as they fix problems that developers are hitting. If I could use
>> an updated clk-next-shmobile branch that includes these patches
>> and commit ids that won't change I would be most grateful.
>
> Hi all,
>
> Thanks for your persistence. This was sitting in a local branch for a
> while and I forgot to merge it into clk-next. I've updated the
> clk-next-shmobile branch to include these patches and I have applied all
> Acks and Tested-bys. That branch includes some other patches as well but
> it shouldn't be a problem for a development branch.
>
> I've also merged clk-next-shmobile into clk-next and pushed out to my
> Linaro mirror.

Great to hear, I will drop my local versions of these.

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

Re: [PATCH V2 0/2] clk: shmobile: clk-mstp fixes

[Simon Horman]

On Tue, Jan 14, 2014 at 11:44:41AM -0800, Mike Turquette wrote:
> Quoting Simon Horman (2014-01-09 17:07:01)
> > [ CC Ben Dooks ]
> > 
> > On Mon, Jan 06, 2014 at 08:33:17PM +0400, Valentine wrote:
> > > On 12/30/2013 01:54 AM, Mike Turquette wrote:
> > > >Quoting Valentine Barshak (2013-12-28 04:09:07)
> > > >>This fixes a number of issues with CPG MSTP clock array handling.
> > > >
> > > >Thanks for the quick respin. Taken into clk-next.
> > > 
> > > Sorry, I don't see it in https://git.linaro.org/people/mike.turquette/linux.git/shortlog/refs/heads/clk-next
> > > Is this the right place to look for?
> > 
> > Hi Mike,
> > 
> > I'm not seeing them there either.
> > 
> > Also, I would like to pull these changes into my "devel" branch
> > as they fix problems that developers are hitting. If I could use
> > an updated clk-next-shmobile branch that includes these patches
> > and commit ids that won't change I would be most grateful.
> 
> Hi all,
> 
> Thanks for your persistence. This was sitting in a local branch for a
> while and I forgot to merge it into clk-next. I've updated the
> clk-next-shmobile branch to include these patches and I have applied all
> Acks and Tested-bys. That branch includes some other patches as well but
> it shouldn't be a problem for a development branch.
> 
> I've also merged clk-next-shmobile into clk-next and pushed out to my
> Linaro mirror.

Thanks.

As a convenience to Renesas developers I will merge this into
the devel branch of the renesas tree. I am not planning
to submit any pull requests based on it at this time.