Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752675AbbETBC4 (ORCPT ); Tue, 19 May 2015 21:02:56 -0400 Received: from mga14.intel.com ([192.55.52.115]:41029 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752542AbbETBCy convert rfc822-to-8bit (ORCPT ); Tue, 19 May 2015 21:02:54 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,461,1427785200"; d="scan'208";a="573974044" From: "Zha, Qipeng" To: Mark Brown CC: "linux-kernel@vger.kernel.org" , "Yang, Fei" , "Zhong, Huiquan" , "Chen, Jason CJ" , "Zheng, Qi" Subject: RE: [PATCH] regmap: add virtual PMIC IPC bus support Thread-Topic: [PATCH] regmap: add virtual PMIC IPC bus support Thread-Index: AQHQe3+/Jgx/xoYcK0Ow1OcZtR3wtp2Bi7ng//+wpYCAAanGkP//9f+AgAFayWA= Date: Wed, 20 May 2015 01:02:49 +0000 Message-ID: References: <1429573138-68002-1-git-send-email-qipeng.zha@intel.com> <20150518112109.GZ2761@sirena.org.uk> <20150519120915.GJ2761@sirena.org.uk> In-Reply-To: <20150519120915.GJ2761@sirena.org.uk> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2077 Lines: 40 Hi Not sure what's your mean using regmap api without bus suport for this case Confirm many device driver will use this api. The detail is Beside pmic core driver itself, there are many pmic function device dirvers, such as gpio, theremal, charger, bcu ..., will use this regmap api to access registers. Using regmap, we can keep transparent for register access in these sub pmic device drivers , Since we do also have different pmic hw which is controlled by i2c, not ipc here If don't add such regmap core support here, we need wrap a similar layer as regmap core for these sub device drivers. Best wishes Qipeng -----Original Message----- From: Mark Brown [mailto:broonie@kernel.org] Sent: Tuesday, May 19, 2015 8:09 PM To: Zha, Qipeng Cc: linux-kernel@vger.kernel.org; Yang, Fei; Zhong, Huiquan; Chen, Jason CJ; Zheng, Qi Subject: Re: [PATCH] regmap: add virtual PMIC IPC bus support On Tue, May 19, 2015 at 04:57:00AM +0000, Zha, Qipeng wrote: > >Why is this being added to the regmap core rather than being > >implemented in using the reg_write() and reg_read() callbacks in the > >core driver for the PMIC? > Thanks. I don't receive this message before in eamil. > Current regmap only support register access using several bus: i2c, spi, ac97 ... > But register access for intel pmic are done by hardware ipc, so we add > such support in regmap core, To make pmic subdevice drivers can use generic regmap r/w API to access registers. That doesn't really address my concern - we already have the reg_read() and reg_write() interface so devices can use regmap without a bus in the core. The question is if this is something that's going to be used by many devices so there's code to share or if it's something that's only going to be used by this PMIC in which case keeping it in the driver should be easier. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/