From mboxrd@z Thu Jan 1 00:00:00 1970 From: cw00.choi@samsung.com (Chanwoo Choi) Date: Wed, 04 Dec 2013 20:09:18 +0900 Subject: [PATCH v6 3/3] mfd: max14577: Add device tree bindings document In-Reply-To: <1386155011.25938.4.camel@AMDC1943> References: <1386153615-25456-1-git-send-email-k.kozlowski@samsung.com> <1386153615-25456-4-git-send-email-k.kozlowski@samsung.com> <529F08DD.2090005@samsung.com> <1386154579.25938.2.camel@AMDC1943> <529F0B75.5040004@samsung.com> <1386155011.25938.4.camel@AMDC1943> Message-ID: <529F0D5E.4060202@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 12/04/2013 08:03 PM, Krzysztof Kozlowski wrote: > On Wed, 2013-12-04 at 20:01 +0900, Chanwoo Choi wrote: >> On 12/04/2013 07:56 PM, Krzysztof Kozlowski wrote: >>> On Wed, 2013-12-04 at 19:50 +0900, Chanwoo Choi wrote: > (...) >>>> >>>> I prefer to add dt data about max14577-muic on following: >>>> If extcon consumer driver need to use muic device, dts should provide following dt data. >>>> >>>> muic: max14577-muic { >>>> compatible = "maxim, max14577-muic"; >>>> }; >>>>> + }; >>>>> >>>> For example, >>>> If fuel-gauge want to use extcon subsystem, dt data of fuel-gauge add 'extcon' property as following: >>>> >>>> fuel-gague@ { >>>> compatible = "..."; >>>> extcon = <&muic>; >>>> } >>>> >>> Hi, >>> >>> The extcon driver itself does not use the binding. It is only needed for >>> consumers. Is it a common practice to add to documentation such notice >>> for consumers only? >> >> As you comment, It is only for extcon consumer. >> But extcon provider driver always is operated with extcon consumer driver. >> If dts file don't support upper dt data about extcon provider driver, >> all of extcon consumer drivers could not use extcon subsystem. >> >> I thinks it is common dt data for muic device. > > OK, I'll add it. > Thanks, Best Regards, Chanwoo Choi