diff mbox series

dt-bindings: reset: meson: add gxl internal dac reset

Message ID 20200122092526.2436421-1-jbrunet@baylibre.com (mailing list archive)
State New, archived
Headers show
Series dt-bindings: reset: meson: add gxl internal dac reset | expand

Commit Message

Jerome Brunet Jan. 22, 2020, 9:25 a.m. UTC
Add the reset line of the internal DAC found on the amlogic gxl SoC family

Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
---
 include/dt-bindings/reset/amlogic,meson-gxbb-reset.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Neil Armstrong Jan. 23, 2020, 9:51 a.m. UTC | #1
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 <jbrunet@baylibre.com>
> ---
>  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 <narmstrong@baylibre.com>
Philipp Zabel Jan. 23, 2020, 10:13 a.m. UTC | #2
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 <jbrunet@baylibre.com>
> ---
>  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
Jerome Brunet April 14, 2020, 8:28 a.m. UTC | #3
On Thu 23 Jan 2020 at 11:13, Philipp Zabel <p.zabel@pengutronix.de> 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 <jbrunet@baylibre.com>
>> ---
>>  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
Jerome Brunet May 6, 2020, 1:50 p.m. UTC | #4
On Tue 14 Apr 2020 at 10:28, Jerome Brunet <jbrunet@baylibre.com> wrote:

> On Thu 23 Jan 2020 at 11:13, Philipp Zabel <p.zabel@pengutronix.de> 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 <jbrunet@baylibre.com>
>>> ---
>>>  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
Philipp Zabel May 6, 2020, 2:54 p.m. UTC | #5
Hi Jerome,

On Wed, 2020-05-06 at 15:50 +0200, Jerome Brunet wrote:
> On Tue 14 Apr 2020 at 10:28, Jerome Brunet <jbrunet@baylibre.com> wrote:
> 
> > On Thu 23 Jan 2020 at 11:13, Philipp Zabel <p.zabel@pengutronix.de> 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 <jbrunet@baylibre.com>
> > > > ---
> > > >  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
Jerome Brunet May 6, 2020, 3:04 p.m. UTC | #6
On Wed 06 May 2020 at 16:54, Philipp Zabel <p.zabel@pengutronix.de> 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 <jbrunet@baylibre.com> wrote:
>> 
>> > On Thu 23 Jan 2020 at 11:13, Philipp Zabel <p.zabel@pengutronix.de> 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 <jbrunet@baylibre.com>
>> > > > ---
>> > > >  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
diff mbox series

Patch

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					*/