Re: [PATCH V1 5/5] backlight: qcom-wled: Add auto string detection logic

2018-05-14 Thread kgunda
On 2018-05-14 22:32, Bjorn Andersson wrote: On Wed 09 May 00:14 PDT 2018, kgu...@codeaurora.org wrote: On 2018-05-07 23:40, Bjorn Andersson wrote: > On Thu 03 May 02:57 PDT 2018, Kiran Gunda wrote: > > [..] > > + > > +#define WLED_AUTO_DETECT_OVP_COUNT 5 > > +#define

Re: [PATCH V1 5/5] backlight: qcom-wled: Add auto string detection logic

2018-05-14 Thread Bjorn Andersson
On Wed 09 May 00:14 PDT 2018, kgu...@codeaurora.org wrote: > On 2018-05-07 23:40, Bjorn Andersson wrote: > > On Thu 03 May 02:57 PDT 2018, Kiran Gunda wrote: > > > > [..] > > > + > > > +#define WLED_AUTO_DETECT_OVP_COUNT 5 > > > +#define WLED_AUTO_DETECT_CNT_DLY_US HZ

Re: [PATCH V1 5/5] backlight: qcom-wled: Add auto string detection logic

2018-05-09 Thread kgunda
On 2018-05-07 23:40, Bjorn Andersson wrote: On Thu 03 May 02:57 PDT 2018, Kiran Gunda wrote: [..] + +#define WLED_AUTO_DETECT_OVP_COUNT 5 +#define WLED_AUTO_DETECT_CNT_DLY_USHZ /* 1 second */ +static bool wled_auto_detection_required(struct wled *wled) So

Re: [PATCH V1 5/5] backlight: qcom-wled: Add auto string detection logic

2018-05-07 Thread Bjorn Andersson
On Thu 03 May 02:57 PDT 2018, Kiran Gunda wrote: [..] > + > +#define WLED_AUTO_DETECT_OVP_COUNT 5 > +#define WLED_AUTO_DETECT_CNT_DLY_US HZ /* 1 second */ > +static bool wled_auto_detection_required(struct wled *wled) So cfg.auto_detection_enabled is set, but we didn't have a

[PATCH V1 5/5] backlight: qcom-wled: Add auto string detection logic

2018-05-03 Thread Kiran Gunda
The auto string detection algorithm checks if the current WLED sink configuration is valid. It tries enabling every sink and checks if the OVP fault is observed. Based on this information it detects and enables the valid sink configuration. Auto calibration will be triggered when the OVP fault