Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1656391 > unrolled thread

Re: [PATCH v5 0/2] Add TI SCI Reset Driver

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v5 0/2] Add TI SCI Reset Driver Suman Anna <s-anna@ti.com> - 2017-06-02 18:50 +0200
    Re: [PATCH v5 0/2] Add TI SCI Reset Driver Philipp Zabel <p.zabel@pengutronix.de> - 2017-06-06 10:00 +0200

#1656391 — Re: [PATCH v5 0/2] Add TI SCI Reset Driver

Hi Philipp,

On 05/24/2017 01:09 PM, Suman Anna wrote:
> Hi Philipp,
> 
> This series adds a reset controller driver that uses the TI SCI protocol
> to manage resets on some TI SoCs. It is a repost of the previous v4 [1]
> submission that has already been acked by Rob and Santosh, but with some
> minor changes. This series has been waiting for the ti-sci-pm-domains
> series to make it to mainline and because of the dependencies against
> a DT header file including the device id values. The ti-sci-pm-domains
> series is now merged and available in 4.12-rc1.
> 
> The main changes in v5 is to drop the include/dt-bindings/reset/k2g.h
> file based on some comments received from Arnd on a similar header usage
> in the ti-sci-pm-domains series. The ti-sci-pm-domains usage is being
> addressed in [2] and this reset series is refreshed to follow the same
> rule and drop the reset header file.
> 
> Patches are on top of 4.12-rc1 plus the recent Kconfig name change on
> the TI SYSCON reset driver [3].
> 
> Rob, Santosh,
> I have retained the Acks received from you on v4, but I am not sure
> since there are some minor changes to the series. Please provide any
> comments if you do not agree with the changes.
> 
> regards
> Suman
> 
> Changes from v4 [1]:
>  - Removed the include/dt-bindings/reset/k2g.h, and updated the
>    bindings document
>  - Minor checkpatch fixes to the driver patch
> 
> Changes from v3:
>  - Addressed Rob Herring's comments
>  - Added Santosh Shilimkar's Ack
> 
> Changes from v2:
>  - Merged DT binding patch and reset header patch
>  - Added locking for reset bit mask
> 
> Changes from v1:
>  - Revised dt binding
>  - CC Linux ARM list
> 
> [1] http://marc.info/?l=devicetree&m=147976458332492&w=2
> [2] http://marc.info/?l=linux-arm-kernel&m=149547831708124&w=2
> [3] https://patchwork.kernel.org/patch/9744909/
> 
> Andrew F. Davis (2):
>   Documentation: dt: reset: Add TI SCI reset binding
>   reset: Add the TI SCI reset driver
> 

Can you pick up this series for v4.13 if there are no comments. Btw, [2]
is acked by Rob. Will appreciate if you can fixup the patch header from
"Documentation: dt: reset: ..." to "dt-bindings: reset: ..." on the
first patch when applying. Thanks in advance.

regards
Suman

[toc] | [next] | [standalone]

#1658464

Hi Suman,

On Fri, 2017-06-02 at 11:45 -0500, Suman Anna wrote:
[...]
> > [1] http://marc.info/?l=devicetree&m=147976458332492&w=2
> > [2] http://marc.info/?l=linux-arm-kernel&m=149547831708124&w=2
> > [3] https://patchwork.kernel.org/patch/9744909/
> > 
> > Andrew F. Davis (2):
> >   Documentation: dt: reset: Add TI SCI reset binding
> >   reset: Add the TI SCI reset driver
> > 
> 
> Can you pick up this series for v4.13 if there are no comments. Btw, [2]
> is acked by Rob. Will appreciate if you can fixup the patch header from
> "Documentation: dt: reset: ..." to "dt-bindings: reset: ..." on the
> first patch when applying. Thanks in advance.

Thank you for the reminder, I've now applied both to the reset/next
branch. I've fixed the dt-bindings patch commit message as you suggested
and made the struct reset_control_ops const in the driver patch.

regards
Philipp

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web