2020-01-22 09:26:54

by Jerome Brunet

[permalink] [raw]
Subject: [PATCH] dt-bindings: reset: meson: add gxl internal dac reset

Add the reset line of the internal DAC found on the amlogic gxl SoC family

Signed-off-by: Jerome Brunet <[email protected]>
---
include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
index ea5058618863..883bfd3bcbad 100644
--- a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
+++ b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
@@ -69,7 +69,7 @@
#define RESET_SYS_CPU_L2 58
#define RESET_SYS_CPU_P 59
#define RESET_SYS_CPU_MBIST 60
-/* 61 */
+#define RESET_ACODEC 61
/* 62 */
/* 63 */
/* RESET2 */
--
2.24.1


2020-01-23 09:52:25

by Neil Armstrong

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: reset: meson: add gxl internal dac reset

On 22/01/2020 10:25, Jerome Brunet wrote:
> Add the reset line of the internal DAC found on the amlogic gxl SoC family
>
> Signed-off-by: Jerome Brunet <[email protected]>
> ---
> include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> index ea5058618863..883bfd3bcbad 100644
> --- a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> +++ b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> @@ -69,7 +69,7 @@
> #define RESET_SYS_CPU_L2 58
> #define RESET_SYS_CPU_P 59
> #define RESET_SYS_CPU_MBIST 60
> -/* 61 */
> +#define RESET_ACODEC 61
> /* 62 */
> /* 63 */
> /* RESET2 */
>

Reviewed-by: Neil Armstrong <[email protected]>

2020-01-23 10:15:37

by Philipp Zabel

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: reset: meson: add gxl internal dac reset

On Wed, 2020-01-22 at 10:25 +0100, Jerome Brunet wrote:
> Add the reset line of the internal DAC found on the amlogic gxl SoC family
>
> Signed-off-by: Jerome Brunet <[email protected]>
> ---
> include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> index ea5058618863..883bfd3bcbad 100644
> --- a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> +++ b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> @@ -69,7 +69,7 @@
> #define RESET_SYS_CPU_L2 58
> #define RESET_SYS_CPU_P 59
> #define RESET_SYS_CPU_MBIST 60
> -/* 61 */
> +#define RESET_ACODEC 61
> /* 62 */
> /* 63 */
> /* RESET2 */

Thank you, applied to reset/next.

regards
Philipp

2020-04-14 15:07:10

by Jerome Brunet

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: reset: meson: add gxl internal dac reset


On Thu 23 Jan 2020 at 11:13, Philipp Zabel <[email protected]> wrote:

> On Wed, 2020-01-22 at 10:25 +0100, Jerome Brunet wrote:
>> Add the reset line of the internal DAC found on the amlogic gxl SoC family
>>
>> Signed-off-by: Jerome Brunet <[email protected]>
>> ---
>> include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>> index ea5058618863..883bfd3bcbad 100644
>> --- a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>> +++ b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>> @@ -69,7 +69,7 @@
>> #define RESET_SYS_CPU_L2 58
>> #define RESET_SYS_CPU_P 59
>> #define RESET_SYS_CPU_MBIST 60
>> -/* 61 */
>> +#define RESET_ACODEC 61
>> /* 62 */
>> /* 63 */
>> /* RESET2 */
>
> Thank you, applied to reset/next.

Hi Philip,

It seems reset/next has not made it to v5.7-rc1

Would it be possible to provide an immutable branch with this change, or
maybe let Kevin apply this change through the amlogic tree ?

It would allow us to progress on some DT changes during this new cycle.

Thanks
Jerome

>
> regards
> Philipp

2020-05-06 13:53:04

by Jerome Brunet

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: reset: meson: add gxl internal dac reset


On Tue 14 Apr 2020 at 10:28, Jerome Brunet <[email protected]> wrote:

> On Thu 23 Jan 2020 at 11:13, Philipp Zabel <[email protected]> wrote:
>
>> On Wed, 2020-01-22 at 10:25 +0100, Jerome Brunet wrote:
>>> Add the reset line of the internal DAC found on the amlogic gxl SoC family
>>>
>>> Signed-off-by: Jerome Brunet <[email protected]>
>>> ---
>>> include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>>> index ea5058618863..883bfd3bcbad 100644
>>> --- a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>>> +++ b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>>> @@ -69,7 +69,7 @@
>>> #define RESET_SYS_CPU_L2 58
>>> #define RESET_SYS_CPU_P 59
>>> #define RESET_SYS_CPU_MBIST 60
>>> -/* 61 */
>>> +#define RESET_ACODEC 61
>>> /* 62 */
>>> /* 63 */
>>> /* RESET2 */
>>
>> Thank you, applied to reset/next.
>
> Hi Philip,
>
> It seems reset/next has not made it to v5.7-rc1
>
> Would it be possible to provide an immutable branch with this change, or
> maybe let Kevin apply this change through the amlogic tree ?
>
> It would allow us to progress on some DT changes during this new cycle.
>
> Thanks
> Jerome

Hi Philip, how can we move forward on this ?

>
>>
>> regards
>> Philipp

2020-05-06 14:56:41

by Philipp Zabel

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: reset: meson: add gxl internal dac reset

Hi Jerome,

On Wed, 2020-05-06 at 15:50 +0200, Jerome Brunet wrote:
> On Tue 14 Apr 2020 at 10:28, Jerome Brunet <[email protected]> wrote:
>
> > On Thu 23 Jan 2020 at 11:13, Philipp Zabel <[email protected]> wrote:
> >
> > > On Wed, 2020-01-22 at 10:25 +0100, Jerome Brunet wrote:
> > > > Add the reset line of the internal DAC found on the amlogic gxl SoC family
> > > >
> > > > Signed-off-by: Jerome Brunet <[email protected]>
> > > > ---
> > > > include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > >
> > > > diff --git a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> > > > index ea5058618863..883bfd3bcbad 100644
> > > > --- a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> > > > +++ b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
> > > > @@ -69,7 +69,7 @@
> > > > #define RESET_SYS_CPU_L2 58
> > > > #define RESET_SYS_CPU_P 59
> > > > #define RESET_SYS_CPU_MBIST 60
> > > > -/* 61 */
> > > > +#define RESET_ACODEC 61
> > > > /* 62 */
> > > > /* 63 */
> > > > /* RESET2 */
> > >
> > > Thank you, applied to reset/next.
> >
> > Hi Philip,
> >
> > It seems reset/next has not made it to v5.7-rc1
> >
> > Would it be possible to provide an immutable branch with this change, or
> > maybe let Kevin apply this change through the amlogic tree ?
> >
> > It would allow us to progress on some DT changes during this new cycle.
> >
> > Thanks
> > Jerome
>
> Hi Philip, how can we move forward on this ?

Sorry for the delay, I have missed the last window. I've now created an
immutable branch:

git://git.pengutronix.de/pza/linux.git reset/meson-gxl-dac

which I will be included in the next reset pull request.

regards
Philipp

2020-05-06 15:08:04

by Jerome Brunet

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: reset: meson: add gxl internal dac reset


On Wed 06 May 2020 at 16:54, Philipp Zabel <[email protected]> wrote:

> Hi Jerome,
>
> On Wed, 2020-05-06 at 15:50 +0200, Jerome Brunet wrote:
>> On Tue 14 Apr 2020 at 10:28, Jerome Brunet <[email protected]> wrote:
>>
>> > On Thu 23 Jan 2020 at 11:13, Philipp Zabel <[email protected]> wrote:
>> >
>> > > On Wed, 2020-01-22 at 10:25 +0100, Jerome Brunet wrote:
>> > > > Add the reset line of the internal DAC found on the amlogic gxl SoC family
>> > > >
>> > > > Signed-off-by: Jerome Brunet <[email protected]>
>> > > > ---
>> > > > include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
>> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
>> > > >
>> > > > diff --git a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>> > > > index ea5058618863..883bfd3bcbad 100644
>> > > > --- a/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>> > > > +++ b/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h
>> > > > @@ -69,7 +69,7 @@
>> > > > #define RESET_SYS_CPU_L2 58
>> > > > #define RESET_SYS_CPU_P 59
>> > > > #define RESET_SYS_CPU_MBIST 60
>> > > > -/* 61 */
>> > > > +#define RESET_ACODEC 61
>> > > > /* 62 */
>> > > > /* 63 */
>> > > > /* RESET2 */
>> > >
>> > > Thank you, applied to reset/next.
>> >
>> > Hi Philip,
>> >
>> > It seems reset/next has not made it to v5.7-rc1
>> >
>> > Would it be possible to provide an immutable branch with this change, or
>> > maybe let Kevin apply this change through the amlogic tree ?
>> >
>> > It would allow us to progress on some DT changes during this new cycle.
>> >
>> > Thanks
>> > Jerome
>>
>> Hi Philip, how can we move forward on this ?
>
> Sorry for the delay, I have missed the last window. I've now created an
> immutable branch:
>
> git://git.pengutronix.de/pza/linux.git reset/meson-gxl-dac

Thx !

>
> which I will be included in the next reset pull request.
>
> regards
> Philipp