drm/msm/hdmi: Simplify the code

[PATCH 0/7] drm/msm/hdmi: Simplify the code

Krzysztof Kozlowski posted 7 patches 3 weeks, 5 days ago

Download series mbox

drivers/gpu/drm/msm/hdmi/hdmi.c          |  32 +++++-----
drivers/gpu/drm/msm/hdmi/hdmi.h          |  25 +++++++-
drivers/gpu/drm/msm/hdmi/hdmi_audio.c    |   5 +-
drivers/gpu/drm/msm/hdmi/hdmi_bridge.c   | 102 +++++++++++++------------------
drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c     |  85 +++++++++-----------------
drivers/gpu/drm/msm/hdmi/hdmi_hpd.c      |   4 +-
drivers/gpu/drm/msm/hdmi/hdmi_i2c.c      |  12 ++--
drivers/gpu/drm/msm/hdmi/hdmi_phy.c      |   2 +-
drivers/gpu/drm/msm/hdmi/hdmi_phy_8960.c |   2 +-
drivers/gpu/drm/msm/hdmi/hdmi_phy_8x60.c |   2 +-
drivers/gpu/drm/msm/hdmi/hdmi_phy_8x74.c |   2 +-
11 files changed, 121 insertions(+), 152 deletions(-)

Expand all Fold all

[PATCH 0/7] drm/msm/hdmi: Simplify the code

Posted by Krzysztof Kozlowski 3 weeks, 5 days ago

Few code simplifications, without functional impact.

Context based (dependency) on:
https://lore.kernel.org/all/20260311191620.245394-2-krzysztof.kozlowski@oss.qualcomm.com/

Best regards,
Krzysztof

---
Krzysztof Kozlowski (7):
      drm/msm/hdmi: Simplify with dev_of_node()
      drm/msm/hdmi: Avoid double error print on msm_hdmi_get_phy() failure
      drm/msm/hdmi: Simplify with local 'dev' variable
      drm/msm/hdmi: Consistently use u32 instead of uint32_t
      drm/msm/hdmi: Drop redundant 'int' for longs
      drm/msm/hdmi_bridge: Simplify register bit updates
      drm/msm/hdmi_hdcp: Simplify register bit updates

 drivers/gpu/drm/msm/hdmi/hdmi.c          |  32 +++++-----
 drivers/gpu/drm/msm/hdmi/hdmi.h          |  25 +++++++-
 drivers/gpu/drm/msm/hdmi/hdmi_audio.c    |   5 +-
 drivers/gpu/drm/msm/hdmi/hdmi_bridge.c   | 102 +++++++++++++------------------
 drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c     |  85 +++++++++-----------------
 drivers/gpu/drm/msm/hdmi/hdmi_hpd.c      |   4 +-
 drivers/gpu/drm/msm/hdmi/hdmi_i2c.c      |  12 ++--
 drivers/gpu/drm/msm/hdmi/hdmi_phy.c      |   2 +-
 drivers/gpu/drm/msm/hdmi/hdmi_phy_8960.c |   2 +-
 drivers/gpu/drm/msm/hdmi/hdmi_phy_8x60.c |   2 +-
 drivers/gpu/drm/msm/hdmi/hdmi_phy_8x74.c |   2 +-
 11 files changed, 121 insertions(+), 152 deletions(-)
---
base-commit: 5138081b838d92e1bfcddc7b72b9215cca6e83f0
change-id: 20260311-drm-msm-hdmi-cleanup-035d0ca56c9f
prerequisite-message-id: 20260311191620.245394-2-krzysztof.kozlowski@oss.qualcomm.com
prerequisite-patch-id: 2720d6338c556d78897568eba7c7f3a457f690f4

Best regards,
-- 
Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>

Re: [PATCH 0/7] drm/msm/hdmi: Simplify the code

Posted by Dmitry Baryshkov 1 week, 3 days ago

On Wed, 11 Mar 2026 21:17:24 +0100, Krzysztof Kozlowski wrote:
> Few code simplifications, without functional impact.
> 
> Context based (dependency) on:
> https://lore.kernel.org/all/20260311191620.245394-2-krzysztof.kozlowski@oss.qualcomm.com/
> 
> Best regards,
> Krzysztof
> 
> [...]

Applied to msm-next, thanks!

[1/7] drm/msm/hdmi: Simplify with dev_of_node()
      https://gitlab.freedesktop.org/lumag/msm/-/commit/5a9a712b0b00
[2/7] drm/msm/hdmi: Avoid double error print on msm_hdmi_get_phy() failure
      https://gitlab.freedesktop.org/lumag/msm/-/commit/ae505afd62f3
[3/7] drm/msm/hdmi: Simplify with local 'dev' variable
      https://gitlab.freedesktop.org/lumag/msm/-/commit/69c68ab38d67
[4/7] drm/msm/hdmi: Consistently use u32 instead of uint32_t
      https://gitlab.freedesktop.org/lumag/msm/-/commit/bc11794cfe00
[5/7] drm/msm/hdmi: Drop redundant 'int' for longs
      https://gitlab.freedesktop.org/lumag/msm/-/commit/536d2eb2bf87

Best regards,
-- 
With best wishes
Dmitry