From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v7 0/7] Introduce STPMIC1 PMIC Driver Date: Wed, 16 Jan 2019 08:13:56 +0000 Message-ID: <20190116081356.GA23399@dell> References: <1543571742-17995-1-git-send-email-p.paillet@st.com> <20181214124701.GZ4497@dell> <20190115004856.GL117329@dtor-ws> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <20190115004856.GL117329@dtor-ws> Sender: linux-kernel-owner@vger.kernel.org To: "dmitry.torokhov@gmail.com" Cc: Pascal PAILLET-LME , "robh+dt@kernel.org" , "mark.rutland@arm.com" , "lgirdwood@gmail.com" , "broonie@kernel.org" , "wim@linux-watchdog.org" , "linux@roeck-us.net" , "linux-input@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-watchdog@vger.kernel.org" , "benjamin.gaignard@linaro.org" , "eballetbo@gmail.com" , "axel.lin@ingics.com" List-Id: linux-input@vger.kernel.org On Mon, 14 Jan 2019, dmitry.torokhov@gmail.com wrote: > On Fri, Dec 14, 2018 at 12:47:01PM +0000, Lee Jones wrote: > > On Fri, 30 Nov 2018, Pascal PAILLET-LME wrote: > > > > > The goal of this patch-set is to propose a driver for the STPMIC1 PMIC from > > > STMicroelectronics. > > > The STPMIC1 regulators supply power to an application processor as well as > > > to external system peripherals such as DDR, Flash memories and system > > > devices. It also features onkey button input and an hardware watchdog. > > > The STPMIC1 is controlled via I2C. > > > > > > Main driver is drivers/mfd/stpmic1 that handle I2C regmap configuration and > > > irqchip. stpmic1_regulator, stpmic1_onkey and stpmic1_wdt need stpmic1 mfd > > > as parent. > > > > > > STPMIC1 MFD and regulator drivers maybe mandatory at boot time. > > > > > > Pascal Paillet (7): > > > changes in v7: > > > * rebase on regul/for-next > > > > > > dt-bindings: mfd: document stpmic1 > > > mfd: stpmic1: add stpmic1 driver > > > dt-bindings: input: document stpmic1 pmic onkey > > > input: stpmic1: add stpmic1 onkey driver > > > dt-bindings: watchdog: document stpmic1 pmic watchdog > > > watchdog: stpmic1: add stpmic1 watchdog driver > > > regulator: stpmic1: fix regulator_lock usage > > > > Could you please remove any patches which have been applied and > > [RESEND]? > > > > Also, is Dmitry planning on Acking: > > > > dt-bindings: input: document stpmic1 pmic onkey > > > > ... or is Rob's Ack enough? > > For bindings I normally defer to Rob (unless I see something that really > bugs me and then I'll speak up). I usually do the opposite. Seeing as Rob is pretty heavily laden with DT patches, I tend to review/apply patches which do not add new DT properties and/or are obvious/trivial and leave Rob to the rest. Your train-set though of course, your decision. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog