From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Rutland Subject: Re: [RFC v4 2/2] backlight: device tree: add new tps611xx backlight binding Date: Fri, 4 Jul 2014 17:31:19 +0100 Message-ID: <20140704163119.GB7106@leverpostej> References: <1404367277-12003-1-git-send-email-gshark.jeong@gmail.com> <1404367277-12003-3-git-send-email-gshark.jeong@gmail.com> <20140703090447.GC29837@leverpostej> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-doc-owner@vger.kernel.org To: Daniel Jeong Cc: Jingoo Han , Bryan Wu , Lee Jones , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , "grant.likely@linaro.org" , Rob Herring , Randy Dunlap , Daniel Jeong , "linux-kernel@vger.kernel.org" , "linux-fbdev@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-doc@vger.kernel.org" List-Id: devicetree@vger.kernel.org > > +- rfa-enable: enable request for acknowledge. > > + =C2=A0If RFA is enabled, the data byte includes the RFA bit a= nd device will > wait > > + =C2=A0and check acknowledge from device. >=20 > You didn't answer my question as to why this should be in the DT. >=20 >=20 > According to the RFA enable, the easy scale pin works differently. > This value should be set before the first data transfer. Sure, things works differently if this is set. That I understood. What I haven't heard is a rationale as to why this configuration option shuold be in the DT. Can I enable this on all implementations, or not? When would I enable this and when would I not? The property reads like a switch to turn a feature on, rather than the description of the presence of a feature. Mark.