Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751200AbdFBQqb (ORCPT ); Fri, 2 Jun 2017 12:46:31 -0400 Received: from lelnx194.ext.ti.com ([198.47.27.80]:53408 "EHLO lelnx194.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751140AbdFBQq3 (ORCPT ); Fri, 2 Jun 2017 12:46:29 -0400 Subject: Re: [PATCH v5 0/2] Add TI SCI Reset Driver To: Philipp Zabel , Rob Herring , Santosh Shilimkar References: <20170524180930.9290-1-s-anna@ti.com> CC: , , , "Andrew F. Davis" , Tero Kristo , Nishanth Menon , Dave Gerlach From: Suman Anna Message-ID: <04a25503-b675-8832-0323-eb7ecbaaefeb@ti.com> Date: Fri, 2 Jun 2017 11:45:53 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <20170524180930.9290-1-s-anna@ti.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2250 Lines: 63 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