2014-11-10 10:22:17

by Paul Bolle

[permalink] [raw]
Subject: ARM: sti: SOC_STIH407 and STIH407_RESET?

Your commit c27321141915 ("ARM: sti: Add STiH407 Kconfig entry to select
STIH407_RESET") was included in today's linux-next (ie, next-20141110).
It adds a Kconfig entry for SOC_STIH407. That entry selects
STIH407_RESET.

There's no Kconfig symbol STIH407_RESET. And there's nothing in
next-20141110 that cares about SOC_STIH407. So this entry is currently a
nop.

I assume that there are one or more patches queued that make this entry
do something. Is that correct?


Paul Bolle


2014-11-10 10:28:08

by Maxime Coquelin

[permalink] [raw]
Subject: Re: ARM: sti: SOC_STIH407 and STIH407_RESET?

Hi Paul,
On 11/10/2014 11:22 AM, Paul Bolle wrote:
> Your commit c27321141915 ("ARM: sti: Add STiH407 Kconfig entry to select
> STIH407_RESET") was included in today's linux-next (ie, next-20141110).
> It adds a Kconfig entry for SOC_STIH407. That entry selects
> STIH407_RESET.
>
> There's no Kconfig symbol STIH407_RESET. And there's nothing in
> next-20141110 that cares about SOC_STIH407. So this entry is currently a
> nop.
>
> I assume that there are one or more patches queued that make this entry
> do something. Is that correct?
Exactly.

The STIH407_RESET symbol is introduced in Reset sub-system.

The STiH407 reset has been merged in Philipp Zabel Reset tree for v3.19:
http://git.pengutronix.de/?p=pza/linux.git;a=shortlog;h=refs/heads/reset/for_v3.19

Commit: 3ba9204308eb51fe98fb1ab352c17d5319f26724

Best regards,
Maxime

>
>
> Paul Bolle
>

2014-11-10 10:49:36

by Paul Bolle

[permalink] [raw]
Subject: Re: ARM: sti: SOC_STIH407 and STIH407_RESET?

Hi Maxime,

On Mon, 2014-11-10 at 11:27 +0100, Maxime Coquelin wrote:
> The STiH407 reset has been merged in Philipp Zabel Reset tree for v3.19:
> http://git.pengutronix.de/?p=pza/linux.git;a=shortlog;h=refs/heads/reset/for_v3.19

That tree isn't part of linux-next, is it? Is it included in a tree that
itself is included in linux-next, so that it will end up in linux-next
one of these days?


Paul Bolle

2014-11-10 10:56:58

by Lee Jones

[permalink] [raw]
Subject: Re: ARM: sti: SOC_STIH407 and STIH407_RESET?

On Mon, 10 Nov 2014, Paul Bolle wrote:
> Your commit c27321141915 ("ARM: sti: Add STiH407 Kconfig entry to select
> STIH407_RESET") was included in today's linux-next (ie, next-20141110).
> It adds a Kconfig entry for SOC_STIH407. That entry selects
> STIH407_RESET.
>
> There's no Kconfig symbol STIH407_RESET. And there's nothing in
> next-20141110 that cares about SOC_STIH407. So this entry is currently a
> nop.

You're right, thanks for your diligence.

> I assume that there are one or more patches queued that make this entry
> do something. Is that correct?

That is also true. Patch is en route.

It should be coming through the reset tree and arm-soc.

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

2014-11-10 11:10:48

by Maxime Coquelin

[permalink] [raw]
Subject: Re: ARM: sti: SOC_STIH407 and STIH407_RESET?


On 11/10/2014 11:49 AM, Paul Bolle wrote:
> Hi Maxime,
>
> On Mon, 2014-11-10 at 11:27 +0100, Maxime Coquelin wrote:
>> The STiH407 reset has been merged in Philipp Zabel Reset tree for v3.19:
>> http://git.pengutronix.de/?p=pza/linux.git;a=shortlog;h=refs/heads/reset/for_v3.19
> That tree isn't part of linux-next, is it? Is it included in a tree that
> itself is included in linux-next, so that it will end up in linux-next
> one of these days?
I don't know if reset tree is regularly merged into linux-next.
I let Philipp, its maintainer, answer.

Kind regards,
Maxime

>
> Paul Bolle
>