From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752107AbdHHLov (ORCPT ); Tue, 8 Aug 2017 07:44:51 -0400 Received: from mail-wm0-f48.google.com ([74.125.82.48]:34888 "EHLO mail-wm0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751993AbdHHLos (ORCPT ); Tue, 8 Aug 2017 07:44:48 -0400 Subject: Re: [PATCH v2 3/5] thermal: imx: Add support for reading OCOTP through nvmem To: Leonard Crestez , Zhang Rui , Shawn Guo Cc: Rob Herring , Eduardo Valentin , Mark Rutland , =?UTF-8?Q?Lothar_Wa=c3=9fmann?= , Dong Aisheng , Bai Ping , Anson Huang , Octavian Purdila , Fabio Estevam , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20170725080835.GD20064@dragon> <1502176915.4296.1.camel@intel.com> <26407dd5-a9b1-67c0-c4ab-d97fa0b88b79@linaro.org> <1502192287.24350.9.camel@nxp.com> From: Srinivas Kandagatla Message-ID: <9732c524-fbdc-d1be-cd03-089dbb2266c4@linaro.org> Date: Tue, 8 Aug 2017 12:44:45 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <1502192287.24350.9.camel@nxp.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/08/17 12:38, Leonard Crestez wrote: > On Tue, 2017-08-08 at 12:00 +0100, Srinivas Kandagatla wrote: >> On 08/08/17 08:21, Zhang Rui wrote: >>> On Tue, 2017-07-25 at 16:08 +0800, Shawn Guo wrote: >>>> On Fri, Jul 14, 2017 at 05:11:08PM +0300, Leonard Crestez wrote: > >>>>> On newer imx SOCs accessing OCOTP directly is wrong because the >>>>> ocotp clock >>>>> needs to be enabled first. Add support for reading those same >>>>> values through >>>>> the nvmem API instead. >>>>> >>>>> The older path is preserved for compatibility with older dts and >>>>> because it >>>>> works correctly on imx6qdl chips. >>>>> >>>>> Signed-off-by: Leonard Crestez >>>> Acked-by: Shawn Guo > >>> I'm okay with the thermal change. >>> We still need ACK for the nvmem changes in this patch series. > >> NVMEM changes are already sent to Greg K H with other patches >> (https://lkml.org/lkml/2017/7/26/164), should appear in next. > > These patches have a compile-time dependency on each other. Wouldn't it > make more sense for the whole series to go through a single maintainer > tree, atomically? Most of the changes are in driver/thermal. I was expecting that the nvmem change go in as fix in a rc release so that you could apply the other patches after that. Let me ping Greg about this!! > > I'm really very confused about how series that touch multiple areas are > applied. It seems to be a mostly ad-hoc process. > > -- > Regards, > Leonard >