From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Zhang Date: Sun, 20 Jan 2013 05:51:53 +0000 Subject: Re: [PATCH 0/3] pwm-backlight: add subdrivers & Tegra support Message-Id: <50FB85F9.4090007@nvidia.com> List-Id: References: <1358591420-7790-1-git-send-email-acourbot@nvidia.com> <50FB669C.3020704@nvidia.com> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Alexandre Courbot Cc: Thierry Reding , Stephen Warren , "linux-fbdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-tegra@vger.kernel.org" , Alexandre Courbot On 01/20/2013 01:26 PM, Alexandre Courbot wrote: > On Sun, Jan 20, 2013 at 12:38 PM, Mark Zhang wrote: >> So this is a non power sequence version of backlight & panel enabling, >> isn't it? I remember we talked about this several days ago and you >> mentioned kernel guys want an ad-hoc version(power sequence logics >> inside driver, not in DT) and I believe this is it, right? > > Basically, yes - I still think power-seqs could be useful here > (especially after seeing the size of these sub-drivers if you want to > do error checking properly) and plan to give it another shot without > DT, but this will not happen soon since we need to do some GPIO > redesign before. You can see what's wrong in the init() function of > the subdriver: we call a device tree function to obtain the GPIO as > there is no get function. > Okay. I think I got the picture. I'll read the codes when I'm free and I think I'll understand this better after that. >> I think finally I can enable Tegra30 cardhu's display after this patch >> merged. > > Yes, feel free to write a subdriver for Cardhu if you like - I'd like > to see all T20 and T30 boards supported by the time this gets merged. > Yep, I can try to do that. I'll let you know if I have problems. Mark > Thanks, > Alex. >