mbox series

[V5,0/2] Fix WLED FSC Sync and brightness Sync settings

Message ID 1616071180-24493-1-git-send-email-kgunda@codeaurora.org
Headers show
Series Fix WLED FSC Sync and brightness Sync settings | expand

Message

Kiran Gunda March 18, 2021, 12:39 p.m. UTC
This patch series has the following two WLED fixes
 1. As per the current implementation, for WLED5, after
    the FSC (Full Scale Current) update the driver is incorrectly
    toggling the MOD_SYNC register instead of toggling the SYNC register.
    The patch 1/2 fixes this by toggling the SYNC register after
    FSC update.

 2. Currently, the sync bits are set-then-cleared after FSC and brightness
    update. As per hardware team recommendation the FSC and brightness sync
    takes place from clear-then-set transition of the sync bits.
    The patch 2/2 fies this issue.

changes from V4:
  1. Rebased this patch series on the below patch.
     "backlight-qcom-wled-Use-sink_addr-for-sync-toggle.patch".

Changes from V3:
  1. Updated the patch description as per Daneil's suggestion.
  2. Added Daniel's "Reviewed-by" tag for patch 2/2.
  3. Updated the cover letter to use "set" and "clear" properly.
 
Changes from V2:
  1. Added Daniel's "Reviewed-by" tag for patch 1/2.
  2. Updated the patch 2/2 description with "set" and "clear"
     terminology instead of "1" and "0".
  3. Updated the cover letter with "set" and "clear" terminology
     instead of "1" and "0".

Changes from V1:
  1. Updated the cover letter.
  2. Updated the description of the patches as per Daniel's suggestion.

Kiran Gunda (2):
  backlight: qcom-wled: Fix FSC update issue for WLED5
  backlight: qcom-wled: Correct the sync_toggle sequence

 drivers/video/backlight/qcom-wled.c | 37 +++++++++++++++++++++++++------------
 1 file changed, 25 insertions(+), 12 deletions(-)

Comments

patchwork-bot+linux-arm-msm@kernel.org May 26, 2021, 7:03 p.m. UTC | #1
Hello:

This series was applied to qcom/linux.git (refs/heads/for-next):

On Thu, 18 Mar 2021 18:09:38 +0530 you wrote:
> This patch series has the following two WLED fixes

>  1. As per the current implementation, for WLED5, after

>     the FSC (Full Scale Current) update the driver is incorrectly

>     toggling the MOD_SYNC register instead of toggling the SYNC register.

>     The patch 1/2 fixes this by toggling the SYNC register after

>     FSC update.

> 

> [...]


Here is the summary with links:
  - [V5,1/2] backlight: qcom-wled: Fix FSC update issue for WLED5
    https://git.kernel.org/qcom/c/4d6e9cdff7fb
  - [V5,2/2] backlight: qcom-wled: Correct the sync_toggle sequence
    https://git.kernel.org/qcom/c/5eb622eec92c

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html