From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753222AbcBIMpe (ORCPT ); Tue, 9 Feb 2016 07:45:34 -0500 Received: from arrakis.dune.hu ([78.24.191.176]:50244 "EHLO arrakis.dune.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751089AbcBIMpd (ORCPT ); Tue, 9 Feb 2016 07:45:33 -0500 Subject: Re: [PATCH V5 1/2] regulator: Add document for MT6323 regulator To: Mark Brown References: <1454874037-38905-1-git-send-email-blogic@openwrt.org> <20160209124133.GE13270@sirena.org.uk> Cc: Liam Girdwood , Matthias Brugger , chen.zhong@mediatek.com, HenryC.Chen@mediatek.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org From: John Crispin Message-ID: <56B9DF5D.2010501@openwrt.org> Date: Tue, 9 Feb 2016 13:45:17 +0100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <20160209124133.GE13270@sirena.org.uk> 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 On 09/02/2016 13:41, Mark Brown wrote: > On Sun, Feb 07, 2016 at 08:40:36PM +0100, John Crispin wrote: > >> +Required properties: >> +- compatible: "mediatek,mt6323-regulator" > > I'm really not happy with MFD subfunctions like this which add no > information over the parent device name appearing directly in the > binding, it results in us putting Linux specifics about how we split the > device up into the driver. It's not like this could realistically be > used separately to the parent device. Just have the MFD create the > device directly or if the individual regulators are reusable IPs which > can share a driver then describe them as such in the DT. > Hi Mark, How would you like it to be. i am failing to understand that from your comment. John