* Re: [PATCH V5 05/12] Documentation: Add DT bindings for panel-lvds driver [not found] ` <1405629839-12086-6-git-send-email-ajaykumar.rs@samsung.com> @ 2014-07-17 20:50 ` Ajay kumar 2014-07-17 22:48 ` Thierry Reding 0 siblings, 1 reply; 6+ messages in thread From: Ajay kumar @ 2014-07-17 20:50 UTC (permalink / raw) To: devicetree@vger.kernel.org Cc: linux-samsung-soc@vger.kernel.org, Sean Paul, Daniel Vetter, sunil joshi, dri-devel@lists.freedesktop.org, Javier Martinez Canillas, Prashanth G, Ajay Kumar +devicetree@vger.kernel.org On Fri, Jul 18, 2014 at 2:13 AM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: > Add DT binding documentation for panel-lvds driver. > > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > --- > .../devicetree/bindings/panel/panel-lvds.txt | 50 ++++++++++++++++++++ > 1 file changed, 50 insertions(+) > create mode 100644 Documentation/devicetree/bindings/panel/panel-lvds.txt > > diff --git a/Documentation/devicetree/bindings/panel/panel-lvds.txt b/Documentation/devicetree/bindings/panel/panel-lvds.txt > new file mode 100644 > index 0000000..fdf91da2 > --- /dev/null > +++ b/Documentation/devicetree/bindings/panel/panel-lvds.txt > @@ -0,0 +1,50 @@ > +panel interface for eDP/lvds panels > + > +Required properties: > + - compatible: "panel-lvds" > + > +Optional properties: > + -lcd-enable-gpios: > + panel LCD poweron GPIO. > + Indicates which GPIO needs to be powered up as output > + to powerup/enable the switch to the LCD panel. > + -backlight-enable-gpios: > + panel LED enable GPIO. > + Indicates which GPIO needs to be powered up as output > + to enable the backlight. > + -panel-prepare-delay: > + delay value in ms required for panel_prepare process > + Delay in ms needed for the panel LCD unit to > + powerup completely. > + ex: delay needed till eDP panel throws HPD. > + delay needed so that we cans tart reading edid. > + -panel-enable-delay: > + delay value in ms required for panel_enable process > + Delay in ms needed for the panel backlight/LED unit > + to powerup, and delay needed between video_enable and > + backlight_enable. > + -panel-disable-delay: > + delay value in ms required for panel_disable process > + Delay in ms needed for the panel backlight/LED unit > + powerdown, and delay needed between backlight_disable > + and video_disable. > + -panel-unprepare-delay: > + delay value in ms required for panel_post_disable process > + Delay in ms needed for the panel LCD unit to > + to powerdown completely, and the minimum delay needed > + before powering it on again. > + -panel-width-mm: physical panel width [mm] > + -panel-height-mm: physical panel height [mm] > + > +Example: > + > + panel-lvds { > + compatible = "panel-lvds"; > + backlight-enable-gpios = <&gpx3 0 0>; > + panel-prepare-delay = <40>; > + panel-enable-delay = <20>; > + panel-disable-delay = <20>; > + panel-unprepare-delay = <30>; > + panel-width-mm = <256>; > + panel-height-mm = <144>; > + }; > -- > 1.7.9.5 > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V5 05/12] Documentation: Add DT bindings for panel-lvds driver 2014-07-17 20:50 ` [PATCH V5 05/12] Documentation: Add DT bindings for panel-lvds driver Ajay kumar @ 2014-07-17 22:48 ` Thierry Reding 2014-07-18 6:48 ` Ajay kumar 0 siblings, 1 reply; 6+ messages in thread From: Thierry Reding @ 2014-07-17 22:48 UTC (permalink / raw) To: Ajay kumar Cc: devicetree@vger.kernel.org, Ajay Kumar, dri-devel@lists.freedesktop.org, linux-samsung-soc@vger.kernel.org, InKi Dae, Rob Clark, Daniel Vetter, Sean Paul, Jingoo Han, sunil joshi, Prashanth G, Javier Martinez Canillas [-- Attachment #1: Type: text/plain, Size: 3789 bytes --] On Fri, Jul 18, 2014 at 02:20:39AM +0530, Ajay kumar wrote: > +devicetree@vger.kernel.org > > On Fri, Jul 18, 2014 at 2:13 AM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: > > Add DT binding documentation for panel-lvds driver. > > > > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > > --- > > .../devicetree/bindings/panel/panel-lvds.txt | 50 ++++++++++++++++++++ > > 1 file changed, 50 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/panel/panel-lvds.txt > > > > diff --git a/Documentation/devicetree/bindings/panel/panel-lvds.txt b/Documentation/devicetree/bindings/panel/panel-lvds.txt > > new file mode 100644 > > index 0000000..fdf91da2 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/panel/panel-lvds.txt > > @@ -0,0 +1,50 @@ > > +panel interface for eDP/lvds panels > > + > > +Required properties: > > + - compatible: "panel-lvds" I think I've said this before. I oppose the addition of this binding. We need to list a device-specific compatible value here, wildcards like this aren't a good choice. And then if we have that compatible value we can move most of the optional properties below into a driver. > > +Optional properties: If all these properties are optional, then what happens if a device tree node doesn't contain any of them? Doesn't that turn the driver into one big no-op? > > + -lcd-enable-gpios: > > + panel LCD poweron GPIO. > > + Indicates which GPIO needs to be powered up as output > > + to powerup/enable the switch to the LCD panel. > > + -backlight-enable-gpios: > > + panel LED enable GPIO. > > + Indicates which GPIO needs to be powered up as output > > + to enable the backlight. I've also said before that this really belongs in a backlight driver. Chances are that you'll want to have a way to set the brightness of the backlight as well, so simply an enable GPIO won't be good enough. > > + -panel-prepare-delay: > > + delay value in ms required for panel_prepare process > > + Delay in ms needed for the panel LCD unit to > > + powerup completely. > > + ex: delay needed till eDP panel throws HPD. > > + delay needed so that we cans tart reading edid. If the panel signals HPD then we don't need this delay at all and we should just wait for HPD instead. > > + -panel-enable-delay: > > + delay value in ms required for panel_enable process > > + Delay in ms needed for the panel backlight/LED unit > > + to powerup, and delay needed between video_enable and > > + backlight_enable. > > + -panel-disable-delay: > > + delay value in ms required for panel_disable process > > + Delay in ms needed for the panel backlight/LED unit > > + powerdown, and delay needed between backlight_disable > > + and video_disable. > > + -panel-unprepare-delay: > > + delay value in ms required for panel_post_disable process > > + Delay in ms needed for the panel LCD unit to > > + to powerdown completely, and the minimum delay needed > > + before powering it on again. These delays are all panel specific and they don't vary per board, so they shouldn't go into the device tree at all. > > + -panel-width-mm: physical panel width [mm] > > + -panel-height-mm: physical panel height [mm] Same here. Thierry [-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V5 05/12] Documentation: Add DT bindings for panel-lvds driver 2014-07-17 22:48 ` Thierry Reding @ 2014-07-18 6:48 ` Ajay kumar 2014-07-21 7:52 ` Thierry Reding 0 siblings, 1 reply; 6+ messages in thread From: Ajay kumar @ 2014-07-18 6:48 UTC (permalink / raw) To: Thierry Reding Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, Sean Paul, Daniel Vetter, sunil joshi, dri-devel@lists.freedesktop.org, Javier Martinez Canillas, Prashanth G, Ajay Kumar Hi Thierry, Thanks for your comments. On Fri, Jul 18, 2014 at 4:18 AM, Thierry Reding <thierry.reding@gmail.com> wrote: > On Fri, Jul 18, 2014 at 02:20:39AM +0530, Ajay kumar wrote: >> +devicetree@vger.kernel.org >> >> On Fri, Jul 18, 2014 at 2:13 AM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: >> > Add DT binding documentation for panel-lvds driver. >> > >> > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> >> > --- >> > .../devicetree/bindings/panel/panel-lvds.txt | 50 ++++++++++++++++++++ >> > 1 file changed, 50 insertions(+) >> > create mode 100644 Documentation/devicetree/bindings/panel/panel-lvds.txt >> > >> > diff --git a/Documentation/devicetree/bindings/panel/panel-lvds.txt b/Documentation/devicetree/bindings/panel/panel-lvds.txt >> > new file mode 100644 >> > index 0000000..fdf91da2 >> > --- /dev/null >> > +++ b/Documentation/devicetree/bindings/panel/panel-lvds.txt >> > @@ -0,0 +1,50 @@ >> > +panel interface for eDP/lvds panels >> > + >> > +Required properties: >> > + - compatible: "panel-lvds" > > I think I've said this before. I oppose the addition of this binding. We > need to list a device-specific compatible value here, wildcards like > this aren't a good choice. And then if we have that compatible value we > can move most of the optional properties below into a driver. Yes, "panel-lvds" is a wildcard for all lvds panels. And since lvds panels from different vendors have different values for power_up_delay, delay from video_to_backlight etc, I thought its better to pick them up from device tree. >> > +Optional properties: > > If all these properties are optional, then what happens if a device tree > node doesn't contain any of them? Doesn't that turn the driver into one > big no-op? No! We need to provide lcd-supply and backlight-supply. >> > + -lcd-enable-gpios: >> > + panel LCD poweron GPIO. >> > + Indicates which GPIO needs to be powered up as output >> > + to powerup/enable the switch to the LCD panel. >> > + -backlight-enable-gpios: >> > + panel LED enable GPIO. >> > + Indicates which GPIO needs to be powered up as output >> > + to enable the backlight. > > I've also said before that this really belongs in a backlight driver. > Chances are that you'll want to have a way to set the brightness of the > backlight as well, so simply an enable GPIO won't be good enough. Ok. I can handle this in backlight driver itself (with some minor glitches). But, how do I map bridge functions to panel functions now? The bridge supports (pre_enable, enable, disable and post_disable) which I map with (prepare, enable, disable and unprepare) of the panel, using a sample layer called bridge to panel_binder. Moving out the backlight control from panel means I really don't need those extra panel calls(prepare and unprepare)! Then how to distribute 2 panel calls(enable and disable) across 4 bridge calls? >> > + -panel-prepare-delay: >> > + delay value in ms required for panel_prepare process >> > + Delay in ms needed for the panel LCD unit to >> > + powerup completely. >> > + ex: delay needed till eDP panel throws HPD. >> > + delay needed so that we cans tart reading edid. > > If the panel signals HPD then we don't need this delay at all and we > should just wait for HPD instead. Not always for HPD, we need to wait for EDID module as well. >> > + -panel-enable-delay: >> > + delay value in ms required for panel_enable process >> > + Delay in ms needed for the panel backlight/LED unit >> > + to powerup, and delay needed between video_enable and >> > + backlight_enable. >> > + -panel-disable-delay: >> > + delay value in ms required for panel_disable process >> > + Delay in ms needed for the panel backlight/LED unit >> > + powerdown, and delay needed between backlight_disable >> > + and video_disable. >> > + -panel-unprepare-delay: >> > + delay value in ms required for panel_post_disable process >> > + Delay in ms needed for the panel LCD unit to >> > + to powerdown completely, and the minimum delay needed >> > + before powering it on again. > > These delays are all panel specific and they don't vary per board, so > they shouldn't go into the device tree at all. But, fetching them from device tree would allow us to support all lvds panels in this single driver. Thanks and Regards, Ajay Kumar ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V5 05/12] Documentation: Add DT bindings for panel-lvds driver 2014-07-18 6:48 ` Ajay kumar @ 2014-07-21 7:52 ` Thierry Reding 2014-07-21 12:30 ` Ajay kumar 0 siblings, 1 reply; 6+ messages in thread From: Thierry Reding @ 2014-07-21 7:52 UTC (permalink / raw) To: Ajay kumar Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, Sean Paul, Daniel Vetter, sunil joshi, dri-devel@lists.freedesktop.org, Javier Martinez Canillas, Prashanth G, Ajay Kumar [-- Attachment #1.1: Type: text/plain, Size: 6645 bytes --] On Fri, Jul 18, 2014 at 12:18:05PM +0530, Ajay kumar wrote: > Hi Thierry, > > Thanks for your comments. > > On Fri, Jul 18, 2014 at 4:18 AM, Thierry Reding > <thierry.reding@gmail.com> wrote: > > On Fri, Jul 18, 2014 at 02:20:39AM +0530, Ajay kumar wrote: > >> +devicetree@vger.kernel.org > >> > >> On Fri, Jul 18, 2014 at 2:13 AM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: > >> > Add DT binding documentation for panel-lvds driver. > >> > > >> > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > >> > --- > >> > .../devicetree/bindings/panel/panel-lvds.txt | 50 ++++++++++++++++++++ > >> > 1 file changed, 50 insertions(+) > >> > create mode 100644 Documentation/devicetree/bindings/panel/panel-lvds.txt > >> > > >> > diff --git a/Documentation/devicetree/bindings/panel/panel-lvds.txt b/Documentation/devicetree/bindings/panel/panel-lvds.txt > >> > new file mode 100644 > >> > index 0000000..fdf91da2 > >> > --- /dev/null > >> > +++ b/Documentation/devicetree/bindings/panel/panel-lvds.txt > >> > @@ -0,0 +1,50 @@ > >> > +panel interface for eDP/lvds panels > >> > + > >> > +Required properties: > >> > + - compatible: "panel-lvds" > > > > I think I've said this before. I oppose the addition of this binding. We > > need to list a device-specific compatible value here, wildcards like > > this aren't a good choice. And then if we have that compatible value we > > can move most of the optional properties below into a driver. > Yes, "panel-lvds" is a wildcard for all lvds panels. > And since lvds panels from different vendors have different values > for power_up_delay, delay from video_to_backlight etc, I thought its > better to pick them up from device tree. What I'm saying is that we shouldn't be adding this type of wildcard. Instead the compatible property needs to list the specific type of panel and since the driver will match on that specific compatible, the values for the delays etc. are all implicit and don't have to be listed in device tree. > >> > +Optional properties: > > > > If all these properties are optional, then what happens if a device tree > > node doesn't contain any of them? Doesn't that turn the driver into one > > big no-op? > No! We need to provide lcd-supply and backlight-supply. What are those? I don't see them described anywhere. > >> > + -lcd-enable-gpios: > >> > + panel LCD poweron GPIO. > >> > + Indicates which GPIO needs to be powered up as output > >> > + to powerup/enable the switch to the LCD panel. > >> > + -backlight-enable-gpios: > >> > + panel LED enable GPIO. > >> > + Indicates which GPIO needs to be powered up as output > >> > + to enable the backlight. > > > > I've also said before that this really belongs in a backlight driver. > > Chances are that you'll want to have a way to set the brightness of the > > backlight as well, so simply an enable GPIO won't be good enough. > Ok. I can handle this in backlight driver itself (with some minor glitches). You can make it work without glitches as well and we've discussed this before. > But, how do I map bridge functions to panel functions now? > The bridge supports (pre_enable, enable, disable and post_disable) which I map > with (prepare, enable, disable and unprepare) of the panel, using a sample layer > called bridge to panel_binder. > Moving out the backlight control from panel means I really don't need > those extra > panel calls(prepare and unprepare)! > Then how to distribute 2 panel calls(enable and disable) across 4 bridge calls? The .prepare() and .unprepare() functions are useful irrespective of backlight control. What you want to separate is powering up the panel from enabling the backlight. So .prepare() could be what's doing the power up of the panel (and possibly other initialization required to make it work) and .enable() could be as simple as turning on the backlight. What I'm saying is rather than use a backlight-enable-gpios property in the panel, that property should be part of the backlight device and the GPIO can then be toggled by the backlight driver when the backlight is enabled. > >> > + -panel-prepare-delay: > >> > + delay value in ms required for panel_prepare process > >> > + Delay in ms needed for the panel LCD unit to > >> > + powerup completely. > >> > + ex: delay needed till eDP panel throws HPD. > >> > + delay needed so that we cans tart reading edid. > > > > If the panel signals HPD then we don't need this delay at all and we > > should just wait for HPD instead. > Not always for HPD, we need to wait for EDID module as well. I always thought that HPD signalled readiness from the panel to provide EDID, too. Are there really panels that need additional delays after HPD has been signalled until the EDID can be read? Are there maybe other ways to detect that EDID can't be read? > >> > + -panel-enable-delay: > >> > + delay value in ms required for panel_enable process > >> > + Delay in ms needed for the panel backlight/LED unit > >> > + to powerup, and delay needed between video_enable and > >> > + backlight_enable. > >> > + -panel-disable-delay: > >> > + delay value in ms required for panel_disable process > >> > + Delay in ms needed for the panel backlight/LED unit > >> > + powerdown, and delay needed between backlight_disable > >> > + and video_disable. > >> > + -panel-unprepare-delay: > >> > + delay value in ms required for panel_post_disable process > >> > + Delay in ms needed for the panel LCD unit to > >> > + to powerdown completely, and the minimum delay needed > >> > + before powering it on again. > > > > These delays are all panel specific and they don't vary per board, so > > they shouldn't go into the device tree at all. > But, fetching them from device tree would allow us to support all lvds > panels in this single driver. You can do that even without having this data in device tree. My point is that this is redundant information once you know the panel's specific compatible value. Therefore there shouldn't be a need to list this in device tree again. Thierry [-- Attachment #1.2: Type: application/pgp-signature, Size: 819 bytes --] [-- Attachment #2: Type: text/plain, Size: 159 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH V5 05/12] Documentation: Add DT bindings for panel-lvds driver 2014-07-21 7:52 ` Thierry Reding @ 2014-07-21 12:30 ` Ajay kumar 0 siblings, 0 replies; 6+ messages in thread From: Ajay kumar @ 2014-07-21 12:30 UTC (permalink / raw) To: Thierry Reding Cc: devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, Sean Paul, Daniel Vetter, sunil joshi, dri-devel@lists.freedesktop.org, Javier Martinez Canillas, Prashanth G, Ajay Kumar Hi Thierry, On Mon, Jul 21, 2014 at 1:22 PM, Thierry Reding <thierry.reding@gmail.com> wrote: > On Fri, Jul 18, 2014 at 12:18:05PM +0530, Ajay kumar wrote: >> Hi Thierry, >> >> Thanks for your comments. >> >> On Fri, Jul 18, 2014 at 4:18 AM, Thierry Reding >> <thierry.reding@gmail.com> wrote: >> > On Fri, Jul 18, 2014 at 02:20:39AM +0530, Ajay kumar wrote: >> >> +devicetree@vger.kernel.org >> >> >> >> On Fri, Jul 18, 2014 at 2:13 AM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: >> >> > Add DT binding documentation for panel-lvds driver. >> >> > >> >> > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> >> >> > --- >> >> > .../devicetree/bindings/panel/panel-lvds.txt | 50 ++++++++++++++++++++ >> >> > 1 file changed, 50 insertions(+) >> >> > create mode 100644 Documentation/devicetree/bindings/panel/panel-lvds.txt >> >> > >> >> > diff --git a/Documentation/devicetree/bindings/panel/panel-lvds.txt b/Documentation/devicetree/bindings/panel/panel-lvds.txt >> >> > new file mode 100644 >> >> > index 0000000..fdf91da2 >> >> > --- /dev/null >> >> > +++ b/Documentation/devicetree/bindings/panel/panel-lvds.txt >> >> > @@ -0,0 +1,50 @@ >> >> > +panel interface for eDP/lvds panels >> >> > + >> >> > +Required properties: >> >> > + - compatible: "panel-lvds" >> > >> > I think I've said this before. I oppose the addition of this binding. We >> > need to list a device-specific compatible value here, wildcards like >> > this aren't a good choice. And then if we have that compatible value we >> > can move most of the optional properties below into a driver. >> Yes, "panel-lvds" is a wildcard for all lvds panels. >> And since lvds panels from different vendors have different values >> for power_up_delay, delay from video_to_backlight etc, I thought its >> better to pick them up from device tree. > > What I'm saying is that we shouldn't be adding this type of wildcard. > Instead the compatible property needs to list the specific type of panel > and since the driver will match on that specific compatible, the values > for the delays etc. are all implicit and don't have to be listed in > device tree. > >> >> > +Optional properties: >> > >> > If all these properties are optional, then what happens if a device tree >> > node doesn't contain any of them? Doesn't that turn the driver into one >> > big no-op? >> No! We need to provide lcd-supply and backlight-supply. > > What are those? I don't see them described anywhere. > >> >> > + -lcd-enable-gpios: >> >> > + panel LCD poweron GPIO. >> >> > + Indicates which GPIO needs to be powered up as output >> >> > + to powerup/enable the switch to the LCD panel. >> >> > + -backlight-enable-gpios: >> >> > + panel LED enable GPIO. >> >> > + Indicates which GPIO needs to be powered up as output >> >> > + to enable the backlight. >> > >> > I've also said before that this really belongs in a backlight driver. >> > Chances are that you'll want to have a way to set the brightness of the >> > backlight as well, so simply an enable GPIO won't be good enough. >> Ok. I can handle this in backlight driver itself (with some minor glitches). > > You can make it work without glitches as well and we've discussed this > before. > >> But, how do I map bridge functions to panel functions now? >> The bridge supports (pre_enable, enable, disable and post_disable) which I map >> with (prepare, enable, disable and unprepare) of the panel, using a sample layer >> called bridge to panel_binder. >> Moving out the backlight control from panel means I really don't need >> those extra >> panel calls(prepare and unprepare)! >> Then how to distribute 2 panel calls(enable and disable) across 4 bridge calls? > > The .prepare() and .unprepare() functions are useful irrespective of > backlight control. What you want to separate is powering up the panel > from enabling the backlight. So .prepare() could be what's doing the > power up of the panel (and possibly other initialization required to > make it work) and .enable() could be as simple as turning on the > backlight. > > What I'm saying is rather than use a backlight-enable-gpios property in > the panel, that property should be part of the backlight device and the > GPIO can then be toggled by the backlight driver when the backlight is > enabled. > >> >> > + -panel-prepare-delay: >> >> > + delay value in ms required for panel_prepare process >> >> > + Delay in ms needed for the panel LCD unit to >> >> > + powerup completely. >> >> > + ex: delay needed till eDP panel throws HPD. >> >> > + delay needed so that we cans tart reading edid. >> > >> > If the panel signals HPD then we don't need this delay at all and we >> > should just wait for HPD instead. >> Not always for HPD, we need to wait for EDID module as well. > > I always thought that HPD signalled readiness from the panel to provide > EDID, too. Are there really panels that need additional delays after HPD > has been signalled until the EDID can be read? Are there maybe other > ways to detect that EDID can't be read? > >> >> > + -panel-enable-delay: >> >> > + delay value in ms required for panel_enable process >> >> > + Delay in ms needed for the panel backlight/LED unit >> >> > + to powerup, and delay needed between video_enable and >> >> > + backlight_enable. >> >> > + -panel-disable-delay: >> >> > + delay value in ms required for panel_disable process >> >> > + Delay in ms needed for the panel backlight/LED unit >> >> > + powerdown, and delay needed between backlight_disable >> >> > + and video_disable. >> >> > + -panel-unprepare-delay: >> >> > + delay value in ms required for panel_post_disable process >> >> > + Delay in ms needed for the panel LCD unit to >> >> > + to powerdown completely, and the minimum delay needed >> >> > + before powering it on again. >> > >> > These delays are all panel specific and they don't vary per board, so >> > they shouldn't go into the device tree at all. >> But, fetching them from device tree would allow us to support all lvds >> panels in this single driver. > > You can do that even without having this data in device tree. My point > is that this is redundant information once you know the panel's specific > compatible value. Therefore there shouldn't be a need to list this in > device tree again. > > Thierry I am planning to use panel-simple driver for the lvds panels. Let me test using that and if things workout well, then will drop this entire driver. Ajay ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <1405629839-12086-12-git-send-email-ajaykumar.rs@samsung.com>]
* Re: [PATCH V5 11/12] Documentation: Add DT bindings for ps8622/ps8625 bridge driver [not found] ` <1405629839-12086-12-git-send-email-ajaykumar.rs@samsung.com> @ 2014-07-17 20:51 ` Ajay kumar 0 siblings, 0 replies; 6+ messages in thread From: Ajay kumar @ 2014-07-17 20:51 UTC (permalink / raw) To: devicetree@vger.kernel.org Cc: linux-samsung-soc@vger.kernel.org, Vincent Palatin, Sean Paul, Daniel Vetter, sunil joshi, dri-devel@lists.freedesktop.org, Javier Martinez Canillas, Prashanth G, Ajay Kumar +devicetree@vger.kernel.org On Fri, Jul 18, 2014 at 2:13 AM, Ajay Kumar <ajaykumar.rs@samsung.com> wrote: > From: Vincent Palatin <vpalatin@chromium.org> > > Add DT binding documentation for ps8622/ps8625 bridge driver. > > Signed-off-by: Vincent Palatin <vpalatin@chromium.org> > Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com> > --- > .../devicetree/bindings/drm/bridge/ps8622.txt | 21 ++++++++++++++++++++ > 1 file changed, 21 insertions(+) > create mode 100644 Documentation/devicetree/bindings/drm/bridge/ps8622.txt > > diff --git a/Documentation/devicetree/bindings/drm/bridge/ps8622.txt b/Documentation/devicetree/bindings/drm/bridge/ps8622.txt > new file mode 100644 > index 0000000..1d154ac > --- /dev/null > +++ b/Documentation/devicetree/bindings/drm/bridge/ps8622.txt > @@ -0,0 +1,21 @@ > +ps8622-bridge bindings > + > +Required properties: > + - compatible: "parade,ps8622" or "parade,ps8625" > + - reg: first i2c address of the bridge > + - sleep-gpios: OF device-tree gpio specification > + - reset-gpios: OF device-tree gpio specification > + > +Optional properties: > + - lane-count: number of DP lanes to use > + - use-external-pwm: backlight will be controlled by an external PWM > + > +Example: > + ps8622-bridge@48 { > + compatible = "parade,ps8622"; > + reg = <0x48>; > + sleep-gpios = <&gpc3 6 1 0 0>; > + reset-gpios = <&gpc3 1 1 0 0>; > + display-timings = <&lcd_display_timings>; > + lane-count = <1> > + }; > -- > 1.7.9.5 > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-07-21 12:30 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1405629839-12086-1-git-send-email-ajaykumar.rs@samsung.com>
[not found] ` <1405629839-12086-6-git-send-email-ajaykumar.rs@samsung.com>
2014-07-17 20:50 ` [PATCH V5 05/12] Documentation: Add DT bindings for panel-lvds driver Ajay kumar
2014-07-17 22:48 ` Thierry Reding
2014-07-18 6:48 ` Ajay kumar
2014-07-21 7:52 ` Thierry Reding
2014-07-21 12:30 ` Ajay kumar
[not found] ` <1405629839-12086-12-git-send-email-ajaykumar.rs@samsung.com>
2014-07-17 20:51 ` [PATCH V5 11/12] Documentation: Add DT bindings for ps8622/ps8625 bridge driver Ajay kumar
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).