Re: [PATCH 0/2] clk: mediatek: add missing ethernet reset definition

Re: [PATCH 0/2] clk: mediatek: add missing ethernet reset definition

[Stephen Boyd]

On 01/23, John Crispin wrote:
> When the clk code for mt2701 was merged, we forgot to add the reset
> controller of the ethernet subsystem. This series adds the missing code
> and header files required to reference the bits from within a devicetree
> file.
> 
> John Crispin (2):
>   clk: mediatek: add mt2701 ethernet reset
>   reset: mediatek: Add MT2701 ethsys reset controller include file
> 
>  drivers/clk/mediatek/clk-mt2701-eth.c     |    2 ++
>  include/dt-bindings/reset/mt2701-resets.h |    7 +++++++

Any planned DT users in the next merge window? Should I put this
into a topic branch? Do you want a clk maintainer ack? At the
least we need someone like James to ack this first.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

Re: [PATCH 0/2] clk: mediatek: add missing ethernet reset definition

[Stephen Boyd]

On 01/27, John Crispin wrote:
> 
> 
> On 27/01/2017 01:14, Stephen Boyd wrote:
> > On 01/23, John Crispin wrote:
> >> When the clk code for mt2701 was merged, we forgot to add the reset
> >> controller of the ethernet subsystem. This series adds the missing code
> >> and header files required to reference the bits from within a devicetree
> >> file.
> >>
> >> John Crispin (2):
> >>   clk: mediatek: add mt2701 ethernet reset
> >>   reset: mediatek: Add MT2701 ethsys reset controller include file
> >>
> >>  drivers/clk/mediatek/clk-mt2701-eth.c     |    2 ++
> >>  include/dt-bindings/reset/mt2701-resets.h |    7 +++++++
> > 
> > Any planned DT users in the next merge window? Should I put this
> > into a topic branch? Do you want a clk maintainer ack? At the
> > least we need someone like James to ack this first.
> > 
> 
> Hi Stephen,
> 
> the addition of the dtsi snippets for the ethernet core is pending due
> to this patch. getting it into v4.11 would be geat so that i can sent
> dtsi changes during the next merge window.
> 

Sorry, I never saw anything from anyone @mediatek so I never did
anything here. I suppose I'll just go and apply this to clk-next
now and then v4.13 can get the ethernet patches?

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

Re: [PATCH 0/2] clk: mediatek: add missing ethernet reset definition

[John Crispin]

On 19/04/17 19:18, Stephen Boyd wrote:
> On 01/27, John Crispin wrote:
>>
>> On 27/01/2017 01:14, Stephen Boyd wrote:
>>> On 01/23, John Crispin wrote:
>>>> When the clk code for mt2701 was merged, we forgot to add the reset
>>>> controller of the ethernet subsystem. This series adds the missing code
>>>> and header files required to reference the bits from within a devicetree
>>>> file.
>>>>
>>>> John Crispin (2):
>>>>    clk: mediatek: add mt2701 ethernet reset
>>>>    reset: mediatek: Add MT2701 ethsys reset controller include file
>>>>
>>>>   drivers/clk/mediatek/clk-mt2701-eth.c     |    2 ++
>>>>   include/dt-bindings/reset/mt2701-resets.h |    7 +++++++
>>> Any planned DT users in the next merge window? Should I put this
>>> into a topic branch? Do you want a clk maintainer ack? At the
>>> least we need someone like James to ack this first.
>>>
>> Hi Stephen,
>>
>> the addition of the dtsi snippets for the ethernet core is pending due
>> to this patch. getting it into v4.11 would be geat so that i can sent
>> dtsi changes during the next merge window.
>>
> Sorry, I never saw anything from anyone @mediatek so I never did
> anything here. I suppose I'll just go and apply this to clk-next
> now and then v4.13 can get the ethernet patches?
>
Hi Stephen,

sounds like a plan. i'll add the series to my local pending tree and 
resend stuff during the next merge window if it was not merged by then.

     John

Re: [PATCH 1/2] clk: mediatek: add mt2701 ethernet reset

[Stephen Boyd]

On 01/23, John Crispin wrote:
> The ethernet clock core has a reset register that is currently not exposed
> to the user. Fix this by adding the missing registration code.
> 
> Signed-off-by: John Crispin <john@phrozen.org>
> ---

Applied to clk-next

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project