Skip to contentSkip to conversation

Initial support Cadence MHDP8501(HDMI/DP) for i.MX8MQ

PATCH Superseded Sandor Yu 2 participants 14 messages Feb 20, 2024 – Feb 21, 2024

Subsystems (6)

v14 of 20 Superseded Latest is v20 →

lore
Share & tools
lore

Sandor Yu

author
#0 lore raw

v14 cover Initial support Cadence MHDP8501(HDMI/DP) for i.MX8MQ

The patch set initial support Cadence MHDP8501(HDMI/DP) DRM bridge driver and Cadence HDP-TX PHY(HDMI/DP) drivers for Freescale i.MX8MQ.

The patch set compose of DRM bridge drivers and PHY drivers.

Both of them need by patch #1 and #2 to pass build.

DRM bridges driver patches:
#1: drm: bridge: Cadence: Creat mhdp helper driver #2: phy: Add HDMI configuration options
#3: dt-bindings: display: bridge: Add Cadence MHDP8501 #4: drm: bridge: Cadence: Add MHDP8501 DP/HDMI driver

PHY driver patches:
#1: drm: bridge: Cadence: Creat mhdp helper driver #2: phy: Add HDMI configuration options
#5: dt-bindings: phy: Add Freescale iMX8MQ DP and HDMI PHY #6: phy: freescale: Add DisplayPort PHY driver for i.MX8MQ #7: phy: freescale: Add HDMI PHY driver for i.MX8MQ

v13->v14:
Patch #4:
- Rebase to next-20240219, replace get_edid function by edid_read function as commits d807ad80d811b ("drm/bridge: add ->edid_read hook and drm_bridge_edid_read()") and 27b8f91c08d99 ("drm/bridge:
remove ->get_edid callback") had change the API.

v12->v13:
Patch #4:
- Explicitly include linux/platform_device.h for cdns-mhdp8501-core.c
- Fix build warning
- Order bit bpc and color_space in descending shit.
Patch #7:
- Fix build warning

v11->v12:
Patch #1:
- Move status initialize out of mbox_mutex.
- Reorder API functions in alphabetical.
- Add notes for malibox access functions.
- Add year 2024 to copyright.
Patch #4:
- Replace DRM_INFO with dev_info or dev_warn.
- Replace DRM_ERROR with dev_err.
- Return ret when cdns_mhdp_dpcd_read failed in function cdns_dp_aux_transferi().
- Remove unused parmeter in function cdns_dp_get_msa_misc
and use two separate variables for color space and bpc.
- Add year 2024 to copyright.
Patch #6:
- Return error code to replace -1 for function wait_for_ack().
- Set cdns_phy->power_up = false in phy_power_down function.
- Remove "RATE_8_1 = 810000", it is not used in driver.
- Add year 2024 to copyright.
Patch #7:
- Adjust clk disable order.
- Return error code to replace -1 for function wait_for_ack().
- Use bool for variable pclk_in.
- Add year 2024 to copyright.

v10->v11:
- rewrite cdns_mhdp_set_firmware_active() in mhdp8546 core driver, use cdns_mhdp_mailbox_send() to replace cdns_mhdp_mailbox_write() same as the other mailbox access functions.
- use static for cdns_mhdp_mailbox_write() and cdns_mhdp_mailbox_read() and remove them from EXPORT_SYMBOL_GPL().
- remove MODULE_ALIAS() from mhdp8501 driver.

v9->v10:
- Create mhdp helper driver to replace macro functions, move all mhdp mailbox access functions and common functions into the helper driver.
Patch #1:drm: bridge: Cadence: Creat mhdp helper driver it is totaly different with v9.

v8->v9:
- Remove compatible string "cdns,mhdp8501" that had removed from dt-bindings file in v8.
- Add Dmitry's R-b tag to patch #2
- Add Krzysztof's R-b tag to patch #3

v7->v8:
MHDP8501 HDMI/DP:
- Correct DT node name to "display-bridge".
- Remove "cdns,mhdp8501" from mhdp8501 dt-binding doc.

HDMI/DP PHY:
- Introduced functions wait_for_ack and wait_for_ack_clear to handle waiting with acknowledgment bits set and cleared respectively.
- Use FIELD_PRE() to set bitfields for both HDMI and DP PHY.

v6->v7:
MHDP8501 HDMI/DP:
- Combine HDMI and DP driver into one mhdp8501 driver. Use the connector type to load the corresponding functions.
- Remove connector init functions.
- Add <linux/hdmi.h> in phy_hdmi.h to reuse ‘enum hdmi_colorspace’.

HDMI/DP PHY:
- Lowercase hex values
- Fix parameters indent issue on some functions
- Replace ‘udelay’ with ‘usleep_range’

v5->v6:
HDMI/DP bridge driver
- 8501 is the part number of Cadence MHDP on i.MX8MQ.
Use MHDP8501 to name hdmi/dp drivers and files.
- Add compatible "fsl,imx8mq-mhdp8501-dp" for i.MX8MQ DP driver
- Add compatible "fsl,imx8mq-mhdp8501-hdmi" for i.MX8MQ HDMI driver
- Combine HDMI and DP dt-bindings into one file cdns,mhdp8501.yaml
- Fix HDMI scrambling is not enable issue when driver working in 4Kp60 mode.
- Add HDMI/DP PHY API mailbox protect.

HDMI/DP PHY driver:
- Rename DP and HDMI PHY files and move to folder phy/freescale/
- Remove properties num_lanes and link_rate from DP PHY driver.
- Combine HDMI and DP dt-bindings into one file fsl,imx8mq-dp-hdmi-phy.yaml
- Update compatible string to "fsl,imx8mq-dp-phy".
- Update compatible string to "fsl,imx8mq-hdmi-phy".

v4->v5:
- Drop "clk" suffix in clock name.
- Add output port property in the example of hdmi/dp.

v3->v4:
dt-bindings:
- Correct dt-bindings coding style and address review comments.
- Add apb_clk description.
- Add output port for HDMI/DP connector
PHY:
- Alphabetically sorted in Kconfig and Makefile for DP and HDMI PHY
- Remove unused registers define from HDMI and DP PHY drivers.
- More description in phy_hdmi.h.
- Add apb_clk to HDMI and DP phy driver.
HDMI/DP:
- Use get_unaligned_le32() to replace hardcode type conversion in HDMI AVI infoframe data fill function.
- Add mailbox mutex lock in HDMI/DP driver for phy functions to reslove race conditions between HDMI/DP and PHY drivers.
- Add apb_clk to both HDMI and DP driver.
- Rename some function names and add prefix with "cdns_hdmi/cdns_dp".
- Remove bpc 12 and 16 optional that not supported.

v2->v3:
Address comments for dt-bindings files.
- Correct dts-bindings file names
Rename phy-cadence-hdptx-dp.yaml to cdns,mhdp-imx8mq-dp.yaml Rename phy-cadence-hdptx-hdmi.yaml to cdns,mhdp-imx8mq-hdmi.yaml
- Drop redundant words and descriptions.
- Correct hdmi/dp node name.

v2 is a completely different version compared to v1. Previous v1 can be available here [1].

v1->v2:
- Reuse Cadence mailbox access functions from mhdp8546 instead of rockchip DP.
- Mailbox access functions be convert to marco functions that will be referenced by HDP-TX PHY(HDMI/DP) driver too.
- Plain bridge instead of component driver.
- Standalone Cadence HDP-TX PHY(HDMI/DP) driver.
- Audio driver are removed from the patch set, it will be add in another patch set later.

[1] https://patchwork.kernel.org/project/linux-rockchip/cover/[email protected]/

Sandor Yu (7):
drm: bridge: Cadence: Create mhdp helper driver
phy: Add HDMI configuration options
dt-bindings: display: bridge: Add Cadence MHDP8501
drm: bridge: Cadence: Add MHDP8501 DP/HDMI driver
dt-bindings: phy: Add Freescale iMX8MQ DP and HDMI PHY
phy: freescale: Add DisplayPort PHY driver for i.MX8MQ
phy: freescale: Add HDMI PHY driver for i.MX8MQ

 .../display/bridge/cdns,mhdp8501.yaml         | 104 ++
 .../bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml  |  53 +
 drivers/gpu/drm/bridge/cadence/Kconfig        |  20 +
 drivers/gpu/drm/bridge/cadence/Makefile       |   3 +
 .../gpu/drm/bridge/cadence/cdns-mhdp-helper.c | 304 ++++++
 .../drm/bridge/cadence/cdns-mhdp8501-core.c   | 316 ++++++
 .../drm/bridge/cadence/cdns-mhdp8501-core.h   | 365 +++++++
 .../gpu/drm/bridge/cadence/cdns-mhdp8501-dp.c | 700 +++++++++++++
 .../drm/bridge/cadence/cdns-mhdp8501-hdmi.c   | 680 +++++++++++++
 .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 403 ++------
 .../drm/bridge/cadence/cdns-mhdp8546-core.h   |  44 +-
 drivers/phy/freescale/Kconfig                 |  20 +
 drivers/phy/freescale/Makefile                |   2 +
 drivers/phy/freescale/phy-fsl-imx8mq-dp.c     | 726 +++++++++++++
 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c   | 960 ++++++++++++++++++
 include/drm/bridge/cdns-mhdp-helper.h         |  97 ++
 include/linux/phy/phy-hdmi.h                  |  24 +
 include/linux/phy/phy.h                       |   7 +-
 18 files changed, 4453 insertions(+), 375 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/display/bridge/cdns,mhdp8501.yaml
 create mode 100644 Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp-helper.c
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-core.c
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-core.h
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-dp.c
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-hdmi.c
 create mode 100644 drivers/phy/freescale/phy-fsl-imx8mq-dp.c
 create mode 100644 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c
 create mode 100644 include/drm/bridge/cdns-mhdp-helper.h
 create mode 100644 include/linux/phy/phy-hdmi.h
Signature · 1 line
2.34.1
author
#1 lore raw

PATCH v14 1/7 drm: bridge: Cadence: Create mhdp helper driver

MHDP8546 mailbox access functions will be share to other mhdp driver and Cadence HDP-TX HDMI/DP PHY drivers.
Create a new mhdp helper driver and move all those functions into.

cdns_mhdp_reg_write() is renamed to cdns_mhdp_dp_reg_write(), because it use the DPTX command ID DPTX_WRITE_REGISTER.

New cdns_mhdp_reg_write() is created with the general command ID GENERAL_REGISTER_WRITE.

rewrite cdns_mhdp_set_firmware_active() in mhdp8546 core driver, use cdns_mhdp_mailbox_send() to replace cdns_mhdp_mailbox_write() same as the other mailbox access functions.

---
v12->v14:
*No change.

V11->v12:
- Move status initialize out of mbox_mutex.
- Reorder API functions in alphabetical.
- Add notes for malibox access functions.
- Add year 2024 to copyright.

 drivers/gpu/drm/bridge/cadence/Kconfig        |   4 +
 drivers/gpu/drm/bridge/cadence/Makefile       |   1 +
 .../gpu/drm/bridge/cadence/cdns-mhdp-helper.c | 304 +++++++++++++
 .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 403 +++---------------
 .../drm/bridge/cadence/cdns-mhdp8546-core.h   |  44 +-
 include/drm/bridge/cdns-mhdp-helper.h         |  97 +++++
 6 files changed, 479 insertions(+), 374 deletions(-)
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp-helper.c
 create mode 100644 include/drm/bridge/cdns-mhdp-helper.h
drivers/gpu/drm/bridge/cadence/Kconfig +4 -0
index cced81633ddcd..e0973339e9e33 100644
@@ -21,6 +21,9 @@ config DRM_CDNS_DSI_J721E
 	  the routing of the DSS DPI signal to the Cadence DSI.
 endif
 
+config CDNS_MHDP_HELPER
+	tristate
+
 config DRM_CDNS_MHDP8546
 	tristate "Cadence DPI/DP bridge"
 	select DRM_DISPLAY_DP_HELPER
@@ -28,6 +31,7 @@ config DRM_CDNS_MHDP8546
 	select DRM_DISPLAY_HELPER
 	select DRM_KMS_HELPER
 	select DRM_PANEL_BRIDGE
+	select CDNS_MHDP_HELPER
 	depends on OF
 	help
 	  Support Cadence DPI to DP bridge. This is an internal
drivers/gpu/drm/bridge/cadence/Makefile +1 -0
index c95fd5b81d137..087dc074820d7 100644
@@ -2,6 +2,7 @@
 obj-$(CONFIG_DRM_CDNS_DSI) += cdns-dsi.o
 cdns-dsi-y := cdns-dsi-core.o
 cdns-dsi-$(CONFIG_DRM_CDNS_DSI_J721E) += cdns-dsi-j721e.o
+obj-$(CONFIG_CDNS_MHDP_HELPER) += cdns-mhdp-helper.o
 obj-$(CONFIG_DRM_CDNS_MHDP8546) += cdns-mhdp8546.o
 cdns-mhdp8546-y := cdns-mhdp8546-core.o cdns-mhdp8546-hdcp.o
 cdns-mhdp8546-$(CONFIG_DRM_CDNS_MHDP8546_J721E) += cdns-mhdp8546-j721e.o
drivers/gpu/drm/bridge/cadence/cdns-mhdp-helper.c new file +304 -0

304 lines not on this page · raw message

drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c +69 -334

670 lines not on this page · raw message

drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.h +4 -40

71 lines not on this page · raw message

include/drm/bridge/cdns-mhdp-helper.h new file +97 -0

97 lines not on this page · raw message

Signature · 1 line
2.34.1
author
#2 lore raw

PATCH v14 2/7 phy: Add HDMI configuration options

In reply to Sandor Yu

Allow HDMI PHYs to be configured through the generic functions through a custom structure added to the generic union.

The parameters added here are based on HDMI PHY implementation practices. The current set of parameters should cover the potential users.

---
v9->v14:
*No change.

 include/linux/phy/phy-hdmi.h | 24 ++++++++++++++++++++++++
 include/linux/phy/phy.h      |  7 ++++++-
 2 files changed, 30 insertions(+), 1 deletion(-)
 create mode 100644 include/linux/phy/phy-hdmi.h
include/linux/phy/phy-hdmi.h new file +24 -0
new file mode 100644
index 0000000000000..b7de88e9090f0
@@ -0,0 +1,24 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright 2022 NXP
+ */
+
+#ifndef __PHY_HDMI_H_
+#define __PHY_HDMI_H_
+
+#include <linux/hdmi.h>
+/**
+ * struct phy_configure_opts_hdmi - HDMI configuration set
+ * @pixel_clk_rate: Pixel clock of video modes in KHz.
+ * @bpc: Maximum bits per color channel.
+ * @color_space: Colorspace in enum hdmi_colorspace.
+ *
+ * This structure is used to represent the configuration state of a HDMI phy.
+ */
+struct phy_configure_opts_hdmi {
+	unsigned int pixel_clk_rate;
+	unsigned int bpc;
+	enum hdmi_colorspace color_space;
+};
+
+#endif /* __PHY_HDMI_H_ */
include/linux/phy/phy.h +6 -1
index aa76609ba2580..0731b8b707f7c 100644
@@ -17,6 +17,7 @@
 #include <linux/regulator/consumer.h>
 
 #include <linux/phy/phy-dp.h>
+#include <linux/phy/phy-hdmi.h>
 #include <linux/phy/phy-lvds.h>
 #include <linux/phy/phy-mipi-dphy.h>
 
@@ -42,7 +43,8 @@ enum phy_mode {
 	PHY_MODE_MIPI_DPHY,
 	PHY_MODE_SATA,
 	PHY_MODE_LVDS,
-	PHY_MODE_DP
+	PHY_MODE_DP,
+	PHY_MODE_HDMI,
 };
 
 enum phy_media {
@@ -60,11 +62,14 @@ enum phy_media {
  *		the DisplayPort protocol.
  * @lvds:	Configuration set applicable for phys supporting
  *		the LVDS phy mode.
+ * @hdmi:	Configuration set applicable for phys supporting
+ *		the HDMI phy mode.
  */
 union phy_configure_opts {
 	struct phy_configure_opts_mipi_dphy	mipi_dphy;
 	struct phy_configure_opts_dp		dp;
 	struct phy_configure_opts_lvds		lvds;
+	struct phy_configure_opts_hdmi		hdmi;
 };
 
 /**
Signature · 1 line
2.34.1
author
#3 lore raw

PATCH v14 3/7 dt-bindings: display: bridge: Add Cadence MHDP8501

In reply to Sandor Yu

Add bindings for Cadence MHDP8501 DisplayPort/HDMI bridge.

---
v9->v14:
*No change.

 .../display/bridge/cdns,mhdp8501.yaml         | 104 ++++++++++++++++++
 1 file changed, 104 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/display/bridge/cdns,mhdp8501.yaml
Documentation/devicetree/bindings/display/bridge/cdns,mhdp8501.yaml new file +104 -0
new file mode 100644
index 0000000000000..3ae643845cfee
@@ -0,0 +1,104 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/display/bridge/cdns,mhdp8501.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Cadence MHDP8501 DP/HDMI bridge
+
+maintainers:
+  - Sandor Yu <[email protected]>
+
+description:
+  Cadence MHDP8501 DisplayPort/HDMI interface.
+
+properties:
+  compatible:
+    enum:
+      - fsl,imx8mq-mhdp8501
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    maxItems: 1
+    description: MHDP8501 DP/HDMI APB clock.
+
+  phys:
+    maxItems: 1
+    description:
+      phandle to the DisplayPort or HDMI PHY
+
+  interrupts:
+    items:
+      - description: Hotplug cable plugin.
+      - description: Hotplug cable plugout.
+
+  interrupt-names:
+    items:
+      - const: plug_in
+      - const: plug_out
+
+  ports:
+    $ref: /schemas/graph.yaml#/properties/ports
+
+    properties:
+      port@0:
+        $ref: /schemas/graph.yaml#/properties/port
+        description:
+          Input port from display controller output.
+      port@1:
+        $ref: /schemas/graph.yaml#/properties/port
+        description:
+          Output port to DisplayPort or HDMI connector.
+
+    required:
+      - port@0
+      - port@1
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - interrupts
+  - interrupt-names
+  - phys
+  - ports
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/clock/imx8mq-clock.h>
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+    mhdp: display-bridge@32c00000 {
+        compatible = "fsl,imx8mq-mhdp8501";
+        reg = <0x32c00000 0x100000>;
+        interrupts = <GIC_SPI 16 IRQ_TYPE_LEVEL_HIGH>,
+                     <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>;
+        interrupt-names = "plug_in", "plug_out";
+        clocks = <&clk IMX8MQ_CLK_DISP_APB_ROOT>;
+        phys = <&dp_phy>;
+
+        ports {
+            #address-cells = <1>;
+            #size-cells = <0>;
+
+            port@0 {
+                reg = <0>;
+
+                mhdp_in: endpoint {
+                    remote-endpoint = <&dcss_out>;
+                };
+            };
+
+            port@1 {
+                reg = <1>;
+
+                mhdp_out: endpoint {
+                    remote-endpoint = <&dp_connector>;
+                };
+            };
+        };
+    };
Signature · 1 line
2.34.1
author
#4 lore raw

PATCH v14 4/7 drm: bridge: Cadence: Add MHDP8501 DP/HDMI driver

In reply to Sandor Yu

Add a new DRM DisplayPort and HDMI bridge driver for Candence MHDP8501 used in i.MX8MQ SOC. MHDP8501 could support HDMI or DisplayPort standards according embedded Firmware running in the uCPU.

For iMX8MQ SOC, the DisplayPort/HDMI FW was loaded and activated by SOC's ROM code. Bootload binary included respective specific firmware is required.

Driver will check display connector type and
then load the corresponding driver.

---
v13->v14:
- Rebase to next-20240219, replace get_edid function by edid_read function.

v12->v13:
- Explicitly include linux/platform_device.h for cdns-mhdp8501-core.c
- Fix build warning
- Order bit bpc and color_space in descending shit.

v11->v12:
- Replace DRM_INFO with dev_info or dev_warn.
- Replace DRM_ERROR with dev_err.
- Return ret when cdns_mhdp_dpcd_read failed in function cdns_dp_aux_transferi().
- Remove unused parmeter in function cdns_dp_get_msa_misc
and use two separate variables for color space and bpc.
- Add year 2024 to copyright.

 drivers/gpu/drm/bridge/cadence/Kconfig        |  16 +
 drivers/gpu/drm/bridge/cadence/Makefile       |   2 +
 .../drm/bridge/cadence/cdns-mhdp8501-core.c   | 316 ++++++++
 .../drm/bridge/cadence/cdns-mhdp8501-core.h   | 365 +++++++++
 .../gpu/drm/bridge/cadence/cdns-mhdp8501-dp.c | 700 ++++++++++++++++++
 .../drm/bridge/cadence/cdns-mhdp8501-hdmi.c   | 680 +++++++++++++++++
 6 files changed, 2079 insertions(+)
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-core.c
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-core.h
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-dp.c
 create mode 100644 drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-hdmi.c
drivers/gpu/drm/bridge/cadence/Kconfig +16 -0
index e0973339e9e33..45848e741f5f4 100644
@@ -51,3 +51,19 @@ config DRM_CDNS_MHDP8546_J721E
 	  initializes the J721E Display Port and sets up the
 	  clock and data muxes.
 endif
+
+config DRM_CDNS_MHDP8501
+	tristate "Cadence MHDP8501 DP/HDMI bridge"
+	select DRM_KMS_HELPER
+	select DRM_PANEL_BRIDGE
+	select DRM_DISPLAY_DP_HELPER
+	select DRM_DISPLAY_HELPER
+	select CDNS_MHDP_HELPER
+	select DRM_CDNS_AUDIO
+	depends on OF
+	help
+	  Support Cadence MHDP8501 DisplayPort/HDMI bridge.
+	  Cadence MHDP8501 support one or more protocols,
+	  including DisplayPort and HDMI.
+	  To use the DP and HDMI drivers, their respective
+	  specific firmware is required.
drivers/gpu/drm/bridge/cadence/Makefile +2 -0
index 087dc074820d7..02c1a9f3cf6fc 100644
@@ -6,3 +6,5 @@ obj-$(CONFIG_CDNS_MHDP_HELPER) += cdns-mhdp-helper.o
 obj-$(CONFIG_DRM_CDNS_MHDP8546) += cdns-mhdp8546.o
 cdns-mhdp8546-y := cdns-mhdp8546-core.o cdns-mhdp8546-hdcp.o
 cdns-mhdp8546-$(CONFIG_DRM_CDNS_MHDP8546_J721E) += cdns-mhdp8546-j721e.o
+obj-$(CONFIG_DRM_CDNS_MHDP8501) += cdns-mhdp8501.o
+cdns-mhdp8501-y := cdns-mhdp8501-core.o cdns-mhdp8501-dp.o cdns-mhdp8501-hdmi.o
drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-core.c new file +316 -0

316 lines not on this page · raw message

drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-core.h new file +365 -0

365 lines not on this page · raw message

drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-dp.c new file +700 -0

700 lines not on this page · raw message

drivers/gpu/drm/bridge/cadence/cdns-mhdp8501-hdmi.c new file +680 -0

680 lines not on this page · raw message

Signature · 1 line
2.34.1
author
#5 lore raw

PATCH v14 5/7 dt-bindings: phy: Add Freescale iMX8MQ DP and HDMI PHY

In reply to Sandor Yu

Add bindings for Freescale iMX8MQ DP and HDMI PHY.

---
v9->v14:
*No change.

 .../bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml  | 53 +++++++++++++++++++
 1 file changed, 53 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +53 -0
new file mode 100644
index 0000000000000..917f113503dca
@@ -0,0 +1,53 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/fsl,imx8mq-dp-hdmi-phy.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Cadence HDP-TX DP/HDMI PHY for Freescale i.MX8MQ SoC
+
+maintainers:
+  - Sandor Yu <[email protected]>
+
+properties:
+  compatible:
+    enum:
+      - fsl,imx8mq-dp-phy
+      - fsl,imx8mq-hdmi-phy
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    items:
+      - description: PHY reference clock.
+      - description: APB clock.
+
+  clock-names:
+    items:
+      - const: ref
+      - const: apb
+
+  "#phy-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - clock-names
+  - "#phy-cells"
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/clock/imx8mq-clock.h>
+    #include <dt-bindings/phy/phy.h>
+    dp_phy: phy@32c00000 {
+        compatible = "fsl,imx8mq-dp-phy";
+        reg = <0x32c00000 0x100000>;
+        #phy-cells = <0>;
+        clocks = <&hdmi_phy_27m>, <&clk IMX8MQ_CLK_DISP_APB_ROOT>;
+        clock-names = "ref", "apb";
+    };
Signature · 1 line
2.34.1
#6 lore raw

Re: [PATCH v14 5/7] dt-bindings: phy: Add Freescale iMX8MQ DP and HDMI PHY

Hi,

thanks for the update.

Am Dienstag, 20. Februar 2024, 04:23:53 CET schrieb Sandor Yu:

Quoted 24 lines

Add bindings for Freescale iMX8MQ DP and HDMI PHY.

---
v9->v14:
*No change.

 .../bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml  | 53 +++++++++++++++++++
 1 file changed, 53 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +6 -0
new file mode 100644
index 0000000000000..917f113503dca
@@ -0,0 +1,53 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/phy/fsl,imx8mq-dp-hdmi-phy.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml line 16

Quoted 10 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +10 -0
+title: Cadence HDP-TX DP/HDMI PHY for Freescale i.MX8MQ SoC
+
+maintainers:
+  - Sandor Yu <[email protected]>
+
+properties:
+  compatible:
+    enum:
+      - fsl,imx8mq-dp-phy
+      - fsl,imx8mq-hdmi-phy

While reading cdns-mhdp8501-core.c I'm not so sure about this. There is only a single PHY which can be configured for either DP or HDMI. Using separate compatibles for that somehow bugs me.
Maybe the DT maintainers can add some input if this should be single or double compatibles.

Thanks and best regards,
Alexander

Quoted 38 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +37 -0
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    items:
+      - description: PHY reference clock.
+      - description: APB clock.
+
+  clock-names:
+    items:
+      - const: ref
+      - const: apb
+
+  "#phy-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - clock-names
+  - "#phy-cells"
+
+additionalProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/clock/imx8mq-clock.h>
+    #include <dt-bindings/phy/phy.h>
+    dp_phy: phy@32c00000 {
+        compatible = "fsl,imx8mq-dp-phy";
+        reg = <0x32c00000 0x100000>;
+        #phy-cells = <0>;
+        clocks = <&hdmi_phy_27m>, <&clk IMX8MQ_CLK_DISP_APB_ROOT>;
+        clock-names = "ref", "apb";
+    };
Signature · 4 lines
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/
author
#7 lore raw

Re: [PATCH v14 5/7] dt-bindings: phy: Add Freescale iMX8MQ DP and HDMI PHY

Hi Alexander,

Thanks for your comments,

Quoted 77 lines

Hi,

thanks for the update.

Am Dienstag, 20. Februar 2024, 04:23:53 CET schrieb Sandor Yu:

Quoted 14 lines

Add bindings for Freescale iMX8MQ DP and HDMI PHY.

---
v9->v14:
*No change.

 .../bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml  | 53
+++++++++++++++++++
 1 file changed, 53 insertions(+)
 create mode 100644

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml

Quoted 7 lines

diff --git a/Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml b/Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file mode 100644
index 0000000000000..917f113503dca
--- /dev/null
+++

b/Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yam

Review line 4

Quoted 7 lines

+++ l

(quoted diff) +4 -0
@@ -0,0 +1,53 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) %YAML 1.2
+---
+$id:
+http://devi/

+cetree.org%2Fschemas%2Fphy%2Ffsl%2Cimx8mq-dp-hdmi-phy.yaml%23&da ta=05

Quoted 1 line

+%7C02%7CSandor.yu%40nxp.com%7Ce79b4d15c204494963c508dc31fbab5d %7C686e

Quoted 1 line

+a1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638440204190687801%7C Unknown%7

Quoted 1 line

+CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWw iLCJX

Quoted 1 line

+VCI6Mn0%3D%7C0%7C%7C%7C&sdata=rKWiYc1wbOvKMO%2BWnvT6agxo 9V%2B1ndZVTxh

Review

Quoted 4 lines
(quoted diff) +3 -0
+gLT0g7h8%3D&reserved=0
+$schema:
+http://devi/

+cetree.org%2Fmeta-schemas%2Fcore.yaml%23&data=05%7C02%7CSandor.y u%40n

Quoted 1 line

+xp.com%7Ce79b4d15c204494963c508dc31fbab5d%7C686ea1d3bc2b4c6fa9 2cd99c5

Quoted 1 line

+c301635%7C0%7C0%7C638440204190709341%7CUnknown%7CTWFpbGZsb 3d8eyJWIjoi

Quoted 1 line

+MC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0 %7C%7C%

Quoted 1 line

+7C&sdata=%2FuCSz0aVVsRLorOqrorbZIyT7iU5BavPKCbA9qL9qDI%3D&reserv ed=0

Quoted 9 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +9 -0
+
+title: Cadence HDP-TX DP/HDMI PHY for Freescale i.MX8MQ SoC
+
+maintainers:
+  - Sandor Yu <[email protected]>
+
+properties:
+  compatible:
+    enum:
Quoted 9 lines

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml line 16

Quoted 2 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +2 -0
+      - fsl,imx8mq-dp-phy
+      - fsl,imx8mq-hdmi-phy

While reading cdns-mhdp8501-core.c I'm not so sure about this. There is only a single PHY which can be configured for either DP or HDMI. Using separate compatibles for that somehow bugs me.
Maybe the DT maintainers can add some input if this should be single or double compatibles.

When user enable MHDP8501 HDMI or DP, he should clearly know which type he want to enable, From board type, flash.bin(firmware) to dts(connector and phy type), they are all need align to HDMI or DP.

B.R
Sandor

Quoted 56 lines

Thanks and best regards,
Alexander

Quoted 25 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +25 -0
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    items:
+      - description: PHY reference clock.
+      - description: APB clock.
+
+  clock-names:
+    items:
+      - const: ref
+      - const: apb
+
+  "#phy-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - clock-names
+  - "#phy-cells"
+
+additionalProperties: false

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml

Quoted 10 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +10 -0
+
+examples:
+  - |
+    #include <dt-bindings/clock/imx8mq-clock.h>
+    #include <dt-bindings/phy/phy.h>
+    dp_phy: phy@32c00000 {
+        compatible = "fsl,imx8mq-dp-phy";
+        reg = <0x32c00000 0x100000>;
+        #phy-cells = <0>;
+        clocks = <&hdmi_phy_27m>, <&clk

IMX8MQ_CLK_DISP_APB_ROOT>;

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml line 53

Quoted 3 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +2 -0
+        clock-names = "ref", "apb";
+    };

--
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-/
group.com%2F&data=05%7C02%7CSandor.yu%40nxp.com%7Ce79b4d15c2044 94963c508dc31fbab5d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0 %7C638440204190726471%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7 C&sdata=7xs1%2FC%2BK1cSFDc3rlBEZdNBsYw6Gc8AR6CWr2Djz4s0%3D&res erved=0

#8 lore raw

Re: [PATCH v14 5/7] dt-bindings: phy: Add Freescale iMX8MQ DP and HDMI PHY

Hi,

Am Mittwoch, 21. Februar 2024, 08:46:46 CET schrieb Sandor Yu:

Quoted 90 lines

Hi Alexander,

Thanks for your comments,

Quoted 77 lines

Hi,

thanks for the update.

Am Dienstag, 20. Februar 2024, 04:23:53 CET schrieb Sandor Yu:

Quoted 14 lines

Add bindings for Freescale iMX8MQ DP and HDMI PHY.

---
v9->v14:
*No change.

 .../bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml  | 53
+++++++++++++++++++
 1 file changed, 53 insertions(+)
 create mode 100644

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml

Quoted 7 lines

diff --git a/Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml b/Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file mode 100644
index 0000000000000..917f113503dca
--- /dev/null
+++

b/Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yam

Review line 4

Quoted 7 lines

+++ l

(quoted diff) +4 -0
@@ -0,0 +1,53 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) %YAML 1.2
+---
+$id:
+http://devi/

+cetree.org%2Fschemas%2Fphy%2Ffsl%2Cimx8mq-dp-hdmi-phy.yaml%23&da ta=05

Quoted 1 line

+%7C02%7CSandor.yu%40nxp.com%7Ce79b4d15c204494963c508dc31fbab5d %7C686e

Quoted 1 line

+a1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C638440204190687801%7C Unknown%7

Quoted 1 line

+CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWw iLCJX

Quoted 1 line

+VCI6Mn0%3D%7C0%7C%7C%7C&sdata=rKWiYc1wbOvKMO%2BWnvT6agxo 9V%2B1ndZVTxh

Review

Quoted 4 lines
(quoted diff) +3 -0
+gLT0g7h8%3D&reserved=0
+$schema:
+http://devi/

+cetree.org%2Fmeta-schemas%2Fcore.yaml%23&data=05%7C02%7CSandor.y u%40n

Quoted 1 line

+xp.com%7Ce79b4d15c204494963c508dc31fbab5d%7C686ea1d3bc2b4c6fa9 2cd99c5

Quoted 1 line

+c301635%7C0%7C0%7C638440204190709341%7CUnknown%7CTWFpbGZsb 3d8eyJWIjoi

Quoted 1 line

+MC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0 %7C%7C%

Quoted 1 line

+7C&sdata=%2FuCSz0aVVsRLorOqrorbZIyT7iU5BavPKCbA9qL9qDI%3D&reserv ed=0

Quoted 9 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +9 -0
+
+title: Cadence HDP-TX DP/HDMI PHY for Freescale i.MX8MQ SoC
+
+maintainers:
+  - Sandor Yu <[email protected]>
+
+properties:
+  compatible:
+    enum:
Quoted 9 lines

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml line 16

Quoted 2 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +2 -0
+      - fsl,imx8mq-dp-phy
+      - fsl,imx8mq-hdmi-phy

While reading cdns-mhdp8501-core.c I'm not so sure about this. There is only a single PHY which can be configured for either DP or HDMI. Using separate compatibles for that somehow bugs me.
Maybe the DT maintainers can add some input if this should be single or double compatibles.

Quoted 2 lines

When user enable MHDP8501 HDMI or DP, he should clearly know which type he want to enable, From board type, flash.bin(firmware) to dts(connector and phy type), they are all need align to HDMI or DP.

I understand your concerns. On the other hand cdns_mhdp8501_dt_parse() from patch 4 already parses DT setup to decide whether the output is DP or HDMI. mhdp->connector_type can be used to configure the PHY accordingly.

Best regards,
Alexander

Quoted 62 lines

B.R
Sandor

Quoted 56 lines

Thanks and best regards,
Alexander

Quoted 25 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +25 -0
+
+  reg:
+    maxItems: 1
+
+  clocks:
+    items:
+      - description: PHY reference clock.
+      - description: APB clock.
+
+  clock-names:
+    items:
+      - const: ref
+      - const: apb
+
+  "#phy-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+  - clocks
+  - clock-names
+  - "#phy-cells"
+
+additionalProperties: false

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml

Quoted 10 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +10 -0
+
+examples:
+  - |
+    #include <dt-bindings/clock/imx8mq-clock.h>
+    #include <dt-bindings/phy/phy.h>
+    dp_phy: phy@32c00000 {
+        compatible = "fsl,imx8mq-dp-phy";
+        reg = <0x32c00000 0x100000>;
+        #phy-cells = <0>;
+        clocks = <&hdmi_phy_27m>, <&clk

IMX8MQ_CLK_DISP_APB_ROOT>;

Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml line 53

Quoted 3 lines
Documentation/devicetree/bindings/phy/fsl,imx8mq-dp-hdmi-phy.yaml new file +2 -0
+        clock-names = "ref", "apb";
+    };

--
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-/
group.com%2F&data=05%7C02%7CSandor.yu%40nxp.com%7Ce79b4d15c2044 94963c508dc31fbab5d%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0 %7C638440204190726471%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjA wMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C%7 C&sdata=7xs1%2FC%2BK1cSFDc3rlBEZdNBsYw6Gc8AR6CWr2Djz4s0%3D&res erved=0

Signature · 4 lines
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/
author
#9 lore raw

PATCH v14 6/7 phy: freescale: Add DisplayPort PHY driver for i.MX8MQ

In reply to Sandor Yu

Add Cadence HDP-TX DisplayPort PHY driver for i.MX8MQ

Cadence HDP-TX PHY could be put in either DP mode or HDMI mode base on the configuration chosen. DisplayPort PHY mode is configurated in the driver.

---
v12->v14:
*No change.

v11->v12:
- Return error code to replace -1 for function wait_for_ack().
- Set cdns_phy->power_up = false in phy_power_down function.
- Remove "RATE_8_1 = 810000", it is not used in driver.
- Add year 2024 to copyright.

 drivers/phy/freescale/Kconfig             |  10 +
 drivers/phy/freescale/Makefile            |   1 +
 drivers/phy/freescale/phy-fsl-imx8mq-dp.c | 726 ++++++++++++++++++++++
 3 files changed, 737 insertions(+)
 create mode 100644 drivers/phy/freescale/phy-fsl-imx8mq-dp.c
drivers/phy/freescale/Kconfig +10 -0
index 853958fb2c063..c39709fd700ac 100644
@@ -35,6 +35,16 @@ config PHY_FSL_IMX8M_PCIE
 	  Enable this to add support for the PCIE PHY as found on
 	  i.MX8M family of SOCs.
 
+config PHY_FSL_IMX8MQ_DP
+	tristate "Freescale i.MX8MQ DP PHY support"
+	depends on OF && HAS_IOMEM
+	depends on COMMON_CLK
+	select GENERIC_PHY
+	select CDNS_MHDP_HELPER
+	help
+	  Enable this to support the Cadence HDPTX DP PHY driver
+	  on i.MX8MQ SOC.
+
 endif
 
 config PHY_FSL_LYNX_28G
drivers/phy/freescale/Makefile +1 -0
index cedb328bc4d28..47e5285209fa8 100644
@@ -1,4 +1,5 @@
 # SPDX-License-Identifier: GPL-2.0-only
+obj-$(CONFIG_PHY_FSL_IMX8MQ_DP)		+= phy-fsl-imx8mq-dp.o
 obj-$(CONFIG_PHY_FSL_IMX8MQ_USB)	+= phy-fsl-imx8mq-usb.o
 obj-$(CONFIG_PHY_MIXEL_LVDS_PHY)	+= phy-fsl-imx8qm-lvds-phy.o
 obj-$(CONFIG_PHY_MIXEL_MIPI_DPHY)	+= phy-fsl-imx8-mipi-dphy.o
drivers/phy/freescale/phy-fsl-imx8mq-dp.c new file +726 -0

726 lines not on this page · raw message

Signature · 1 line
2.34.1
author
#10 lore raw

PATCH v14 7/7 phy: freescale: Add HDMI PHY driver for i.MX8MQ

In reply to Sandor Yu

Add Cadence HDP-TX HDMI PHY driver for i.MX8MQ.

Cadence HDP-TX PHY could be put in either DP mode or HDMI mode base on the configuration chosen. HDMI PHY mode is configurated in the driver.

---
v13->v14:
*No change.

v12->v13:
- Fix build warning

v11->v12:
- Adjust clk disable order.
- Return error code to replace -1 for function wait_for_ack().
- Use bool for variable pclk_in.
- Add year 2024 to copyright.

 drivers/phy/freescale/Kconfig               |  10 +
 drivers/phy/freescale/Makefile              |   1 +
 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c | 960 ++++++++++++++++++++
 3 files changed, 971 insertions(+)
 create mode 100644 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c
drivers/phy/freescale/Kconfig +10 -0
index c39709fd700ac..14f47b7cc77ab 100644
@@ -45,6 +45,16 @@ config PHY_FSL_IMX8MQ_DP
 	  Enable this to support the Cadence HDPTX DP PHY driver
 	  on i.MX8MQ SOC.
 
+config PHY_FSL_IMX8MQ_HDMI
+	tristate "Freescale i.MX8MQ HDMI PHY support"
+	depends on OF && HAS_IOMEM
+	depends on COMMON_CLK
+	select GENERIC_PHY
+	select CDNS_MHDP_HELPER
+	help
+	  Enable this to support the Cadence HDPTX HDMI PHY driver
+	  on i.MX8MQ SOC.
+
 endif
 
 config PHY_FSL_LYNX_28G
drivers/phy/freescale/Makefile +1 -0
index 47e5285209fa8..1380ac31c2ead 100644
@@ -1,5 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
 obj-$(CONFIG_PHY_FSL_IMX8MQ_DP)		+= phy-fsl-imx8mq-dp.o
+obj-$(CONFIG_PHY_FSL_IMX8MQ_HDMI)	+= phy-fsl-imx8mq-hdmi.o
 obj-$(CONFIG_PHY_FSL_IMX8MQ_USB)	+= phy-fsl-imx8mq-usb.o
 obj-$(CONFIG_PHY_MIXEL_LVDS_PHY)	+= phy-fsl-imx8qm-lvds-phy.o
 obj-$(CONFIG_PHY_MIXEL_MIPI_DPHY)	+= phy-fsl-imx8-mipi-dphy.o
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +960 -0

960 lines not on this page · raw message

Signature · 1 line
2.34.1
#11 lore raw

Re: [PATCH v14 7/7] phy: freescale: Add HDMI PHY driver for i.MX8MQ

Hi,

thanks for the update.

Am Dienstag, 20. Februar 2024, 04:23:55 CET schrieb Sandor Yu:

Quoted 8 lines

Add Cadence HDP-TX HDMI PHY driver for i.MX8MQ.

Cadence HDP-TX PHY could be put in either DP mode or HDMI mode base on the configuration chosen. HDMI PHY mode is configurated in the driver.

This still works as before. I noticed there is a lot of code duplication with patch 6. IMHO these PHY drivers should be merged into a single one where the mode is configured using phy_set_mode() from cdns-mhdp8501-core.c. This nicely matches my concerns regarding patch 5.

Best regards,
Alexander

Quoted 1,018 lines

---
v13->v14:
*No change.

v12->v13:
- Fix build warning

v11->v12:
- Adjust clk disable order.
- Return error code to replace -1 for function wait_for_ack().
- Use bool for variable pclk_in.
- Add year 2024 to copyright.

 drivers/phy/freescale/Kconfig               |  10 +
 drivers/phy/freescale/Makefile              |   1 +
 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c | 960 ++++++++++++++++++++
 3 files changed, 971 insertions(+)
 create mode 100644 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c
drivers/phy/freescale/Kconfig +10 -0
index c39709fd700ac..14f47b7cc77ab 100644
@@ -45,6 +45,16 @@ config PHY_FSL_IMX8MQ_DP
 	  Enable this to support the Cadence HDPTX DP PHY driver
 	  on i.MX8MQ SOC.
 
+config PHY_FSL_IMX8MQ_HDMI
+	tristate "Freescale i.MX8MQ HDMI PHY support"
+	depends on OF && HAS_IOMEM
+	depends on COMMON_CLK
+	select GENERIC_PHY
+	select CDNS_MHDP_HELPER
+	help
+	  Enable this to support the Cadence HDPTX HDMI PHY driver
+	  on i.MX8MQ SOC.
+
 endif
 
 config PHY_FSL_LYNX_28G
drivers/phy/freescale/Makefile +1 -0
index 47e5285209fa8..1380ac31c2ead 100644
@@ -1,5 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
 obj-$(CONFIG_PHY_FSL_IMX8MQ_DP)		+= phy-fsl-imx8mq-dp.o
+obj-$(CONFIG_PHY_FSL_IMX8MQ_HDMI)	+= phy-fsl-imx8mq-hdmi.o
 obj-$(CONFIG_PHY_FSL_IMX8MQ_USB)	+= phy-fsl-imx8mq-usb.o
 obj-$(CONFIG_PHY_MIXEL_LVDS_PHY)	+= phy-fsl-imx8qm-lvds-phy.o
 obj-$(CONFIG_PHY_MIXEL_MIPI_DPHY)	+= phy-fsl-imx8-mipi-dphy.o
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +960 -0
new file mode 100644
index 0000000000000..537b1f45c91cc
@@ -0,0 +1,960 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Cadence High-Definition Multimedia Interface (HDMI) PHY driver
+ *
+ * Copyright (C) 2022-2024 NXP Semiconductor, Inc.
+ */
+#include <asm/unaligned.h>
+#include <drm/bridge/cdns-mhdp-helper.h>
+#include <linux/clk.h>
+#include <linux/kernel.h>
+#include <linux/phy/phy.h>
+#include <linux/platform_device.h>
+#include <linux/io.h>
+
+#define ADDR_PHY_AFE	0x80000
+
+/* PHY registers */
+#define CMN_SSM_BIAS_TMR			0x0022
+#define CMN_PLLSM0_USER_DEF_CTRL		0x002f
+#define CMN_PSM_CLK_CTRL			0x0061
+#define CMN_CDIAG_REFCLK_CTRL			0x0062
+#define CMN_PLL0_VCOCAL_START			0x0081
+#define CMN_PLL0_VCOCAL_INIT_TMR		0x0084
+#define CMN_PLL0_VCOCAL_ITER_TMR		0x0085
+#define CMN_TXPUCAL_CTRL			0x00e0
+#define CMN_TXPDCAL_CTRL			0x00f0
+#define CMN_TXPU_ADJ_CTRL			0x0108
+#define CMN_TXPD_ADJ_CTRL			0x010c
+#define CMN_DIAG_PLL0_FBH_OVRD			0x01c0
+#define CMN_DIAG_PLL0_FBL_OVRD			0x01c1
+#define CMN_DIAG_PLL0_OVRD			0x01c2
+#define CMN_DIAG_PLL0_TEST_MODE			0x01c4
+#define CMN_DIAG_PLL0_V2I_TUNE			0x01c5
+#define CMN_DIAG_PLL0_CP_TUNE			0x01c6
+#define CMN_DIAG_PLL0_LF_PROG			0x01c7
+#define CMN_DIAG_PLL0_PTATIS_TUNE1		0x01c8
+#define CMN_DIAG_PLL0_PTATIS_TUNE2		0x01c9
+#define CMN_DIAG_PLL0_INCLK_CTRL		0x01ca
+#define CMN_DIAG_PLL0_PXL_DIVH			0x01cb
+#define CMN_DIAG_PLL0_PXL_DIVL			0x01cc
+#define CMN_DIAG_HSCLK_SEL			0x01e0
+#define XCVR_PSM_RCTRL				0x4001
+#define TX_TXCC_CAL_SCLR_MULT_0			0x4047
+#define TX_TXCC_CPOST_MULT_00_0			0x404c
+#define XCVR_DIAG_PLLDRC_CTRL			0x40e0
+#define XCVR_DIAG_PLLDRC_CTRL			0x40e0
+#define XCVR_DIAG_HSCLK_SEL			0x40e1
+#define XCVR_DIAG_BIDI_CTRL			0x40e8
+#define TX_PSC_A0				0x4100
+#define TX_PSC_A1				0x4101
+#define TX_PSC_A2				0x4102
+#define TX_PSC_A3				0x4103
+#define TX_DIAG_TX_CTRL				0x41e0
+#define TX_DIAG_TX_DRV				0x41e1
+#define TX_DIAG_BGREF_PREDRV_DELAY		0x41e7
+#define TX_DIAG_ACYA_0				0x41ff
+#define TX_DIAG_ACYA_1				0x43ff
+#define TX_DIAG_ACYA_2				0x45ff
+#define TX_DIAG_ACYA_3				0x47ff
+#define TX_ANA_CTRL_REG_1			0x5020
+#define TX_ANA_CTRL_REG_2			0x5021
+#define TX_DIG_CTRL_REG_2			0x5024
+#define TXDA_CYA_AUXDA_CYA			0x5025
+#define TX_ANA_CTRL_REG_3			0x5026
+#define TX_ANA_CTRL_REG_4			0x5027
+#define TX_ANA_CTRL_REG_5			0x5029
+#define RX_PSC_A0				0x8000
+#define RX_PSC_CAL				0x8006
+#define PHY_HDP_MODE_CTRL			0xc008
+#define PHY_HDP_CLK_CTL				0xc009
+#define PHY_ISO_CMN_CTRL			0xc010
+#define PHY_PMA_CMN_CTRL1			0xc800
+#define PHY_PMA_ISO_CMN_CTRL			0xc810
+#define PHY_PMA_ISO_PLL_CTRL1			0xc812
+#define PHY_PMA_ISOLATION_CTRL			0xc81f
+
+/* PHY_HDP_CLK_CTL */
+#define PLL_DATA_RATE_CLK_DIV_MASK		GENMASK(15, 8)
+#define PLL_DATA_RATE_CLK_DIV_HBR		0x24
+#define PLL_DATA_RATE_CLK_DIV_HBR2		0x12
+#define PLL_CLK_EN_ACK_EN			BIT(3)
+#define PLL_CLK_EN				BIT(2)
+#define PLL_READY				BIT(1)
+#define PLL_EN					BIT(0)
+
+/* PHY_PMA_CMN_CTRL1 */
+#define CMA_REF_CLK_DIG_DIV_MASK		GENMASK(13, 12)
+#define CMA_REF_CLK_SEL_MASK			GENMASK(6, 4)
+#define CMA_REF_CLK_RCV_EN_MASK			BIT(3)
+#define CMA_REF_CLK_RCV_EN			1
+#define CMN_READY				BIT(0)
+
+/* PHY_PMA_ISO_PLL_CTRL1 */
+#define CMN_PLL0_CLK_DATART_DIV_MASK		GENMASK(7, 0)
+
+/* TX_DIAG_TX_DRV */
+#define TX_DRIVER_PROG_BOOST_ENABLE		BIT(10)
+#define TX_DRIVER_PROG_BOOST_LEVEL_MASK		GENMASK(9, 8)
+#define TX_DRIVER_LDO_BG_DEPENDENT_REF_ENABLE	BIT(7)
+#define TX_DRIVER_LDO_BANDGAP_REF_ENABLE	BIT(6)
+
+/* TX_TXCC_CAL_SCLR_MULT_0 */
+#define SCALED_RESISTOR_CALIBRATION_CODE_ADD	BIT(8)
+#define RESISTOR_CAL_MULT_VAL_32_128		BIT(5)
+
+/* CMN_CDIAG_REFCLK_CTRL */
+#define DIG_REF_CLK_DIV_SCALER_MASK		GENMASK(14, 12)
+#define REFCLK_TERMINATION_EN_OVERRIDE_EN	BIT(7)
+#define REFCLK_TERMINATION_EN_OVERRIDE		BIT(6)
+
+/* CMN_DIAG_HSCLK_SEL */
+#define HSCLK1_SEL_MASK				GENMASK(5, 4)
+#define HSCLK0_SEL_MASK				GENMASK(1, 0)
+
+/* XCVR_DIAG_HSCLK_SEL */
+#define HSCLK_SEL_MODE3_MASK			GENMASK(13, 12)
+#define HSCLK_SEL_MODE3_HSCLK1			1
+
+/* CMN_PLL0_VCOCAL_START */
+#define VCO_CALIB_CODE_START_POINT_VAL_MASK	GENMASK(8, 0)
+
+/* CMN_DIAG_PLL0_FBH_OVRD */
+#define PLL_FEEDBACK_DIV_HI_OVERRIDE_EN		BIT(15)
+
+/* CMN_DIAG_PLL0_FBL_OVRD */
+#define PLL_FEEDBACK_DIV_LO_OVERRIDE_EN		BIT(15)
+
+/* CMN_DIAG_PLL0_PXL_DIVH */
+#define PLL_PCLK_DIV_EN				BIT(15)
+
+/* XCVR_DIAG_PLLDRC_CTRL */
+#define DPLL_CLK_SEL_MODE3			BIT(14)
+
+/* TX_DIAG_TX_CTRL */
+#define TX_IF_SUBRATE_MODE3_MASK		GENMASK(7, 6)
+
+/* PHY_HDP_MODE_CTRL */
+#define POWER_STATE_A3_ACK			BIT(7)
+#define POWER_STATE_A2_ACK			BIT(6)
+#define POWER_STATE_A1_ACK			BIT(5)
+#define POWER_STATE_A0_ACK			BIT(4)
+#define POWER_STATE_A3				BIT(3)
+#define POWER_STATE_A2				BIT(2)
+#define POWER_STATE_A1				BIT(1)
+#define POWER_STATE_A0				BIT(0)
+
+/* PHY_PMA_ISO_CMN_CTRL */
+#define CMN_MACRO_PWR_EN_ACK			BIT(5)
+
+#define KEEP_ALIVE		0x18
+
+#define REF_CLK_27MHZ		27000000
+
+/* HDMI TX clock control settings */
+struct hdptx_hdmi_ctrl {
+	u32 pixel_clk_freq_min;
+	u32 pixel_clk_freq_max;
+	u32 feedback_factor;
+	u32 data_range_kbps_min;
+	u32 data_range_kbps_max;
+	u32 cmnda_pll0_ip_div;
+	u32 cmn_ref_clk_dig_div;
+	u32 ref_clk_divider_scaler;
+	u32 pll_fb_div_total;
+	u32 cmnda_pll0_fb_div_low;
+	u32 cmnda_pll0_fb_div_high;
+	u32 pixel_div_total;
+	u32 cmnda_pll0_pxdiv_low;
+	u32 cmnda_pll0_pxdiv_high;
+	u32 vco_freq_min;
+	u32 vco_freq_max;
+	u32 vco_ring_select;
+	u32 cmnda_hs_clk_0_sel;
+	u32 cmnda_hs_clk_1_sel;
+	u32 hsclk_div_at_xcvr;
+	u32 hsclk_div_tx_sub_rate;
+	u32 cmnda_pll0_hs_sym_div_sel;
+	u32 cmnda_pll0_clk_freq_min;
+	u32 cmnda_pll0_clk_freq_max;
+};
+
+struct cdns_hdptx_hdmi_phy {
+	struct cdns_mhdp_base base;
+
+	void __iomem *regs;	/* DPTX registers base */
+	struct mutex mbox_mutex; /* mutex to protect mailbox */
+	struct device *dev;
+	struct phy *phy;
+	struct clk *ref_clk, *apb_clk;
+	u32 ref_clk_rate;
+	u32 pixel_clk_rate;
+	enum hdmi_colorspace color_space;
+	u32 bpc;
+};
+
+/* HDMI TX clock control settings, pixel clock is output */
+static const struct hdptx_hdmi_ctrl pixel_clk_output_ctrl_table[] = {
+/*Minclk  Maxclk Fdbak  DR_min   DR_max  ip_d  dig  DS    Totl */
+{ 27000,  27000, 1000,  270000,  270000, 0x03, 0x1, 0x1,  240, 0x0bc, 0x030,  80, 0x026, 0x026, 2160000, 2160000, 0, 2, 2, 2, 4, 0x3,  27000,  27000},
+{ 27000,  27000, 1250,  337500,  337500, 0x03, 0x1, 0x1,  300, 0x0ec, 0x03c, 100, 0x030, 0x030, 2700000, 2700000, 0, 2, 2, 2, 4, 0x3,  33750,  33750},
+{ 27000,  27000, 1500,  405000,  405000, 0x03, 0x1, 0x1,  360, 0x11c, 0x048, 120, 0x03a, 0x03a, 3240000, 3240000, 0, 2, 2, 2, 4, 0x3,  40500,  40500},
+{ 27000,  27000, 2000,  540000,  540000, 0x03, 0x1, 0x1,  240, 0x0bc, 0x030,  80, 0x026, 0x026, 2160000, 2160000, 0, 2, 2, 2, 4, 0x2,  54000,  54000},
+{ 54000,  54000, 1000,  540000,  540000, 0x03, 0x1, 0x1,  480, 0x17c, 0x060,  80, 0x026, 0x026, 4320000, 4320000, 1, 2, 2, 2, 4, 0x3,  54000,  54000},
+{ 54000,  54000, 1250,  675000,  675000, 0x04, 0x1, 0x1,  400, 0x13c, 0x050,  50, 0x017, 0x017, 2700000, 2700000, 0, 1, 1, 2, 4, 0x2,  67500,  67500},
+{ 54000,  54000, 1500,  810000,  810000, 0x04, 0x1, 0x1,  480, 0x17c, 0x060,  60, 0x01c, 0x01c, 3240000, 3240000, 0, 2, 2, 2, 2, 0x2,  81000,  81000},
+{ 54000,  54000, 2000, 1080000, 1080000, 0x03, 0x1, 0x1,  240, 0x0bc, 0x030,  40, 0x012, 0x012, 2160000, 2160000, 0, 2, 2, 2, 1, 0x1, 108000, 108000},
+{ 74250,  74250, 1000,  742500,  742500, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  80, 0x026, 0x026, 5940000, 5940000, 1, 2, 2, 2, 4, 0x3,  74250,  74250},
+{ 74250,  74250, 1250,  928125,  928125, 0x04, 0x1, 0x1,  550, 0x1b4, 0x06e,  50, 0x017, 0x017, 3712500, 3712500, 1, 1, 1, 2, 4, 0x2,  92812,  92812},
+{ 74250,  74250, 1500, 1113750, 1113750, 0x04, 0x1, 0x1,  660, 0x20c, 0x084,  60, 0x01c, 0x01c, 4455000, 4455000, 1, 2, 2, 2, 2, 0x2, 111375, 111375},
+{ 74250,  74250, 2000, 1485000, 1485000, 0x03, 0x1, 0x1,  330, 0x104, 0x042,  40, 0x012, 0x012, 2970000, 2970000, 0, 2, 2, 2, 1, 0x1, 148500, 148500},
+{ 99000,  99000, 1000,  990000,  990000, 0x03, 0x1, 0x1,  440, 0x15c, 0x058,  40, 0x012, 0x012, 3960000, 3960000, 1, 2, 2, 2, 2, 0x2,  99000,  99000},
+{ 99000,  99000, 1250, 1237500, 1237500, 0x03, 0x1, 0x1,  275, 0x0d8, 0x037,  25, 0x00b, 0x00a, 2475000, 2475000, 0, 1, 1, 2, 2, 0x1, 123750, 123750},
+{ 99000,  99000, 1500, 1485000, 1485000, 0x03, 0x1, 0x1,  330, 0x104, 0x042,  30, 0x00d, 0x00d, 2970000, 2970000, 0, 2, 2, 2, 1, 0x1, 148500, 148500},
+{ 99000,  99000, 2000, 1980000, 1980000, 0x03, 0x1, 0x1,  440, 0x15c, 0x058,  40, 0x012, 0x012, 3960000, 3960000, 1, 2, 2, 2, 1, 0x1, 198000, 198000},
+{148500, 148500, 1000, 1485000, 1485000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  40, 0x012, 0x012, 5940000, 5940000, 1, 2, 2, 2, 2, 0x2, 148500, 148500},
+{148500, 148500, 1250, 1856250, 1856250, 0x04, 0x1, 0x1,  550, 0x1b4, 0x06e,  25, 0x00b, 0x00a, 3712500, 3712500, 1, 1, 1, 2, 2, 0x1, 185625, 185625},
+{148500, 148500, 1500, 2227500, 2227500, 0x03, 0x1, 0x1,  495, 0x188, 0x063,  30, 0x00d, 0x00d, 4455000, 4455000, 1, 1, 1, 2, 2, 0x1, 222750, 222750},
+{148500, 148500, 2000, 2970000, 2970000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  40, 0x012, 0x012, 5940000, 5940000, 1, 2, 2, 2, 1, 0x1, 297000, 297000},
+{198000, 198000, 1000, 1980000, 1980000, 0x03, 0x1, 0x1,  220, 0x0ac, 0x02c,  10, 0x003, 0x003, 1980000, 1980000, 0, 1, 1, 2, 1, 0x0, 198000, 198000},
+{198000, 198000, 1250, 2475000, 2475000, 0x03, 0x1, 0x1,  550, 0x1b4, 0x06e,  25, 0x00b, 0x00a, 4950000, 4950000, 1, 1, 1, 2, 2, 0x1, 247500, 247500},
+{198000, 198000, 1500, 2970000, 2970000, 0x03, 0x1, 0x1,  330, 0x104, 0x042,  15, 0x006, 0x005, 2970000, 2970000, 0, 1, 1, 2, 1, 0x0, 297000, 297000},
+{198000, 198000, 2000, 3960000, 3960000, 0x03, 0x1, 0x1,  440, 0x15c, 0x058,  20, 0x008, 0x008, 3960000, 3960000, 1, 1, 1, 2, 1, 0x0, 396000, 396000},
+{297000, 297000, 1000, 2970000, 2970000, 0x03, 0x1, 0x1,  330, 0x104, 0x042,  10, 0x003, 0x003, 2970000, 2970000, 0, 1, 1, 2, 1, 0x0, 297000, 297000},
+{297000, 297000, 1500, 4455000, 4455000, 0x03, 0x1, 0x1,  495, 0x188, 0x063,  15, 0x006, 0x005, 4455000, 4455000, 1, 1, 1, 2, 1, 0x0, 445500, 445500},
+{297000, 297000, 2000, 5940000, 5940000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  20, 0x008, 0x008, 5940000, 5940000, 1, 1, 1, 2, 1, 0x0, 594000, 594000},
+{594000, 594000, 1000, 5940000, 5940000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  10, 0x003, 0x003, 5940000, 5940000, 1, 1, 1, 2, 1, 0x0, 594000, 594000},
+{594000, 594000,  750, 4455000, 4455000, 0x03, 0x1, 0x1,  495, 0x188, 0x063,  10, 0x003, 0x003, 4455000, 4455000, 1, 1, 1, 2, 1, 0x0, 445500, 445500},
+{594000, 594000,  625, 3712500, 3712500, 0x04, 0x1, 0x1,  550, 0x1b4, 0x06e,  10, 0x003, 0x003, 3712500, 3712500, 1, 1, 1, 2, 1, 0x0, 371250, 371250},
+{594000, 594000,  500, 2970000, 2970000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  10, 0x003, 0x003, 5940000, 5940000, 1, 1, 1, 2, 2, 0x1, 297000, 297000},
+};
+
+/* HDMI TX PLL tuning settings */
+struct hdptx_hdmi_pll_tuning {
+	u32 vco_freq_bin;
+	u32 vco_freq_min;
+	u32 vco_freq_max;
+	u32 volt_to_current_coarse;
+	u32 volt_to_current;
+	u32 ndac_ctrl;
+	u32 pmos_ctrl;
+	u32 ptat_ndac_ctrl;
+	u32 feedback_div_total;
+	u32 charge_pump_gain;
+	u32 coarse_code;
+	u32 v2i_code;
+	u32 vco_cal_code;
+};
+
+/* HDMI TX PLL tuning settings, pixel clock is output */
+static const struct hdptx_hdmi_pll_tuning pixel_clk_output_pll_table[] = {
+/*bin VCO_freq min/max  coar  cod NDAC  PMOS PTAT div-T P-Gain Coa V2I CAL */
+{  1, 1980000, 1980000, 0x4, 0x3, 0x0, 0x09, 0x09, 220, 0x42, 160, 5, 183 },
+{  2, 2160000, 2160000, 0x4, 0x3, 0x0, 0x09, 0x09, 240, 0x42, 166, 6, 208 },
+{  3, 2475000, 2475000, 0x5, 0x3, 0x1, 0x00, 0x07, 275, 0x42, 167, 6, 209 },
+{  4, 2700000, 2700000, 0x5, 0x3, 0x1, 0x00, 0x07, 300, 0x42, 188, 6, 230 },
+{  4, 2700000, 2700000, 0x5, 0x3, 0x1, 0x00, 0x07, 400, 0x4c, 188, 6, 230 },
+{  5, 2970000, 2970000, 0x6, 0x3, 0x1, 0x00, 0x07, 330, 0x42, 183, 6, 225 },
+{  6, 3240000, 3240000, 0x6, 0x3, 0x1, 0x00, 0x07, 360, 0x42, 203, 7, 256 },
+{  6, 3240000, 3240000, 0x6, 0x3, 0x1, 0x00, 0x07, 480, 0x4c, 203, 7, 256 },
+{  7, 3712500, 3712500, 0x4, 0x3, 0x0, 0x07, 0x0F, 550, 0x4c, 212, 7, 257 },
+{  8, 3960000, 3960000, 0x5, 0x3, 0x0, 0x07, 0x0F, 440, 0x42, 184, 6, 226 },
+{  9, 4320000, 4320000, 0x5, 0x3, 0x1, 0x07, 0x0F, 480, 0x42, 205, 7, 258 },
+{ 10, 4455000, 4455000, 0x5, 0x3, 0x0, 0x07, 0x0F, 495, 0x42, 219, 7, 272 },
+{ 10, 4455000, 4455000, 0x5, 0x3, 0x0, 0x07, 0x0F, 660, 0x4c, 219, 7, 272 },
+{ 11, 4950000, 4950000, 0x6, 0x3, 0x1, 0x00, 0x07, 550, 0x42, 213, 7, 258 },
+{ 12, 5940000, 5940000, 0x7, 0x3, 0x1, 0x00, 0x07, 660, 0x42, 244, 8, 292 },
+};
+
+static int cdns_phy_reg_write(struct cdns_hdptx_hdmi_phy *cdns_phy, u32 addr, u32 val)
+{
+	return cdns_mhdp_reg_write(&cdns_phy->base, ADDR_PHY_AFE + (addr << 2), val);
+}
+
+static u32 cdns_phy_reg_read(struct cdns_hdptx_hdmi_phy *cdns_phy, u32 addr)
+{
+	u32 reg32;
+
+	cdns_mhdp_reg_read(&cdns_phy->base, ADDR_PHY_AFE + (addr << 2), &reg32);
+
+	return reg32;
+}
+
+static int wait_for_ack(struct cdns_hdptx_hdmi_phy *cdns_phy, u32 reg, u32 mask,
+			const char *err_msg)
+{
+	u32 val, i;
+
+	for (i = 0; i < 10; i++) {
+		val = cdns_phy_reg_read(cdns_phy, reg);
+		if (val & mask)
+			return 0;
+		msleep(20);
+	}
+
+	dev_err(cdns_phy->dev, "%s\n", err_msg);
+	return -ETIMEDOUT;
+}
+
+static bool hdptx_phy_check_alive(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	u32  alive, newalive;
+	u8 retries_left = 50;
+
+	alive = readl(cdns_phy->regs + KEEP_ALIVE);
+
+	while (retries_left--) {
+		udelay(2);
+
+		newalive = readl(cdns_phy->regs + KEEP_ALIVE);
+		if (alive == newalive)
+			continue;
+		return true;
+	}
+	return false;
+}
+
+static int hdptx_hdmi_clk_enable(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	struct device *dev = cdns_phy->dev;
+	u32 ref_clk_rate;
+	int ret;
+
+	cdns_phy->ref_clk = devm_clk_get(dev, "ref");
+	if (IS_ERR(cdns_phy->ref_clk)) {
+		dev_err(dev, "phy ref clock not found\n");
+		return PTR_ERR(cdns_phy->ref_clk);
+	}
+
+	cdns_phy->apb_clk = devm_clk_get(dev, "apb");
+	if (IS_ERR(cdns_phy->apb_clk)) {
+		dev_err(dev, "phy apb clock not found\n");
+		return PTR_ERR(cdns_phy->apb_clk);
+	}
+
+	ret = clk_prepare_enable(cdns_phy->ref_clk);
+	if (ret) {
+		dev_err(cdns_phy->dev, "Failed to prepare ref clock\n");
+		return ret;
+	}
+
+	ref_clk_rate = clk_get_rate(cdns_phy->ref_clk);
+	if (!ref_clk_rate) {
+		dev_err(cdns_phy->dev, "Failed to get ref clock rate\n");
+		goto err_ref_clk;
+	}
+
+	if (ref_clk_rate == REF_CLK_27MHZ) {
+		cdns_phy->ref_clk_rate = ref_clk_rate;
+	} else {
+		dev_err(cdns_phy->dev, "Not support Ref Clock Rate(%dHz)\n", ref_clk_rate);
+		goto err_ref_clk;
+	}
+
+	ret = clk_prepare_enable(cdns_phy->apb_clk);
+	if (ret) {
+		dev_err(cdns_phy->dev, "Failed to prepare apb clock\n");
+		goto err_ref_clk;
+	}
+
+	return 0;
+
+err_ref_clk:
+	clk_disable_unprepare(cdns_phy->ref_clk);
+	return -EINVAL;
+}
+
+static void hdptx_hdmi_clk_disable(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	clk_disable_unprepare(cdns_phy->apb_clk);
+	clk_disable_unprepare(cdns_phy->ref_clk);
+}
+
+static void hdptx_hdmi_arc_config(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	u16 txpu_calib_code;
+	u16 txpd_calib_code;
+	u16 txpu_adj_calib_code;
+	u16 txpd_adj_calib_code;
+	u16 prev_calib_code;
+	u16 new_calib_code;
+	u16 rdata;
+
+	/* Power ARC */
+	cdns_phy_reg_write(cdns_phy, TXDA_CYA_AUXDA_CYA, 0x0001);
+
+	prev_calib_code = cdns_phy_reg_read(cdns_phy, TX_DIG_CTRL_REG_2);
+	txpu_calib_code = cdns_phy_reg_read(cdns_phy, CMN_TXPUCAL_CTRL);
+	txpd_calib_code = cdns_phy_reg_read(cdns_phy, CMN_TXPDCAL_CTRL);
+	txpu_adj_calib_code = cdns_phy_reg_read(cdns_phy, CMN_TXPU_ADJ_CTRL);
+	txpd_adj_calib_code = cdns_phy_reg_read(cdns_phy, CMN_TXPD_ADJ_CTRL);
+
+	new_calib_code = ((txpu_calib_code + txpd_calib_code) / 2)
+		+ txpu_adj_calib_code + txpd_adj_calib_code;
+
+	if (new_calib_code != prev_calib_code) {
+		rdata = cdns_phy_reg_read(cdns_phy, TX_ANA_CTRL_REG_1);
+		rdata &= 0xdfff;
+		cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, rdata);
+		cdns_phy_reg_write(cdns_phy, TX_DIG_CTRL_REG_2, new_calib_code);
+		mdelay(10);
+		rdata |= 0x2000;
+		cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, rdata);
+		usleep_range(150, 250);
+	}
+
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x0100);
+	usleep_range(100, 200);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x0300);
+	usleep_range(100, 200);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_3, 0x0000);
+	usleep_range(100, 200);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2008);
+	usleep_range(100, 200);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2018);
+	usleep_range(100, 200);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2098);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x030c);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_5, 0x0010);
+	usleep_range(100, 200);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_4, 0x4001);
+	mdelay(5);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2198);
+	mdelay(5);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x030d);
+	usleep_range(100, 200);
+	cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x030f);
+}
+
+static void hdptx_hdmi_phy_set_vswing(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	u32 k;
+	const u32 num_lanes = 4;
+
+	for (k = 0; k < num_lanes; k++) {
+		cdns_phy_reg_write(cdns_phy, (TX_DIAG_TX_DRV | (k << 9)),
+				   TX_DRIVER_PROG_BOOST_ENABLE |
+				   FIELD_PREP(TX_DRIVER_PROG_BOOST_LEVEL_MASK, 3) |
+				   TX_DRIVER_LDO_BG_DEPENDENT_REF_ENABLE |
+				   TX_DRIVER_LDO_BANDGAP_REF_ENABLE);
+		cdns_phy_reg_write(cdns_phy, (TX_TXCC_CPOST_MULT_00_0 | (k << 9)), 0x0);
+		cdns_phy_reg_write(cdns_phy, (TX_TXCC_CAL_SCLR_MULT_0 | (k << 9)),
+				   SCALED_RESISTOR_CALIBRATION_CODE_ADD |
+				   RESISTOR_CAL_MULT_VAL_32_128);
+	}
+}
+
+static int hdptx_hdmi_feedback_factor(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	u32 feedback_factor;
+
+	switch (cdns_phy->color_space) {
+	case HDMI_COLORSPACE_YUV422:
+		feedback_factor = 1000;
+		break;
+
+	case HDMI_COLORSPACE_YUV420:
+		switch (cdns_phy->bpc) {
+		case 8:
+			feedback_factor = 500;
+			break;
+		case 10:
+			feedback_factor = 625;
+			break;
+		case 12:
+			feedback_factor = 750;
+			break;
+		case 16:
+			feedback_factor = 1000;
+			break;
+		default:
+			dev_dbg(cdns_phy->dev, "Invalid ColorDepth\n");
+			return 0;
+		}
+		break;
+
+	default:
+		/* Assume RGB/YUV444 */
+		switch (cdns_phy->bpc) {
+		case 10:
+			feedback_factor = 1250;
+			break;
+		case 12:
+			feedback_factor = 1500;
+			break;
+		case 16:
+			feedback_factor = 2000;
+			break;
+		default:
+			feedback_factor = 1000;
+		}
+	}
+
+	return feedback_factor;
+}
+
+static int hdptx_hdmi_phy_config(struct cdns_hdptx_hdmi_phy *cdns_phy,
+				 const struct hdptx_hdmi_ctrl *p_ctrl_table,
+				 const struct hdptx_hdmi_pll_tuning *p_pll_table,
+				 bool pclk_in)
+{
+	const u32 num_lanes = 4;
+	u32 val, k;
+	int ret;
+
+	/* enable PHY isolation mode only for CMN */
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_ISOLATION_CTRL, 0xd000);
+
+	/* set cmn_pll0_clk_datart1_div/cmn_pll0_clk_datart0_div dividers */
+	val = cdns_phy_reg_read(cdns_phy, PHY_PMA_ISO_PLL_CTRL1);
+	val &= ~CMN_PLL0_CLK_DATART_DIV_MASK;
+	val |= FIELD_PREP(CMN_PLL0_CLK_DATART_DIV_MASK, 0x12);
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_PLL_CTRL1, val);
+
+	/* assert PHY reset from isolation register */
+	cdns_phy_reg_write(cdns_phy, PHY_ISO_CMN_CTRL, 0x0000);
+	/* assert PMA CMN reset */
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_CMN_CTRL, 0x0000);
+
+	/* register XCVR_DIAG_BIDI_CTRL */
+	for (k = 0; k < num_lanes; k++)
+		cdns_phy_reg_write(cdns_phy, XCVR_DIAG_BIDI_CTRL | (k << 9), 0x00ff);
+
+	/* Describing Task phy_cfg_hdp */
+	val = cdns_phy_reg_read(cdns_phy, PHY_PMA_CMN_CTRL1);
+	val &= ~CMA_REF_CLK_RCV_EN_MASK;
+	val |= FIELD_PREP(CMA_REF_CLK_RCV_EN_MASK, CMA_REF_CLK_RCV_EN);
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_CMN_CTRL1, val);
+
+	/* PHY Registers */
+	val = cdns_phy_reg_read(cdns_phy, PHY_PMA_CMN_CTRL1);
+	val &= ~CMA_REF_CLK_DIG_DIV_MASK;
+	val |= FIELD_PREP(CMA_REF_CLK_DIG_DIV_MASK, p_ctrl_table->cmn_ref_clk_dig_div);
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_CMN_CTRL1, val);
+
+	val = cdns_phy_reg_read(cdns_phy, PHY_HDP_CLK_CTL);
+	val &= ~PLL_DATA_RATE_CLK_DIV_MASK;
+	val |= FIELD_PREP(PLL_DATA_RATE_CLK_DIV_MASK,
+			  PLL_DATA_RATE_CLK_DIV_HBR2);
+	cdns_phy_reg_write(cdns_phy, PHY_HDP_CLK_CTL, val);
+
+	/* Common control module control and diagnostic registers */
+	val = cdns_phy_reg_read(cdns_phy, CMN_CDIAG_REFCLK_CTRL);
+	val &= ~DIG_REF_CLK_DIV_SCALER_MASK;
+	val |= FIELD_PREP(DIG_REF_CLK_DIV_SCALER_MASK, p_ctrl_table->ref_clk_divider_scaler);
+	val |= REFCLK_TERMINATION_EN_OVERRIDE_EN | REFCLK_TERMINATION_EN_OVERRIDE;
+	cdns_phy_reg_write(cdns_phy, CMN_CDIAG_REFCLK_CTRL, val);
+
+	/* High speed clock used */
+	val = cdns_phy_reg_read(cdns_phy, CMN_DIAG_HSCLK_SEL);
+	val &= ~(HSCLK1_SEL_MASK | HSCLK0_SEL_MASK);
+	val |= FIELD_PREP(HSCLK1_SEL_MASK, (p_ctrl_table->cmnda_hs_clk_1_sel >> 1));
+	val |= FIELD_PREP(HSCLK0_SEL_MASK, (p_ctrl_table->cmnda_hs_clk_0_sel >> 1));
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_HSCLK_SEL, val);
+
+	for (k = 0; k < num_lanes; k++) {
+		val = cdns_phy_reg_read(cdns_phy, (XCVR_DIAG_HSCLK_SEL | (k << 9)));
+		val &= ~HSCLK_SEL_MODE3_MASK;
+		val |= FIELD_PREP(HSCLK_SEL_MODE3_MASK,
+				  (p_ctrl_table->cmnda_hs_clk_0_sel >> 1));
+		cdns_phy_reg_write(cdns_phy, (XCVR_DIAG_HSCLK_SEL | (k << 9)), val);
+	}
+
+	/* PLL 0 control state machine registers */
+	val = p_ctrl_table->vco_ring_select << 12;
+	cdns_phy_reg_write(cdns_phy, CMN_PLLSM0_USER_DEF_CTRL, val);
+
+	if (pclk_in) {
+		val = 0x30a0;
+	} else {
+		val = cdns_phy_reg_read(cdns_phy, CMN_PLL0_VCOCAL_START);
+		val &= ~VCO_CALIB_CODE_START_POINT_VAL_MASK;
+		val |= FIELD_PREP(VCO_CALIB_CODE_START_POINT_VAL_MASK,
+				  p_pll_table->vco_cal_code);
+	}
+	cdns_phy_reg_write(cdns_phy, CMN_PLL0_VCOCAL_START, val);
+
+	cdns_phy_reg_write(cdns_phy, CMN_PLL0_VCOCAL_INIT_TMR, 0x0064);
+	cdns_phy_reg_write(cdns_phy, CMN_PLL0_VCOCAL_ITER_TMR, 0x000a);
+
+	/* Common functions control and diagnostics registers */
+	val = p_ctrl_table->cmnda_pll0_hs_sym_div_sel << 8;
+	val |= p_ctrl_table->cmnda_pll0_ip_div;
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_INCLK_CTRL, val);
+
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_OVRD, 0x0000);
+
+	val = p_ctrl_table->cmnda_pll0_fb_div_high;
+	val |= PLL_FEEDBACK_DIV_HI_OVERRIDE_EN;
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_FBH_OVRD, val);
+
+	val = p_ctrl_table->cmnda_pll0_fb_div_low;
+	val |= PLL_FEEDBACK_DIV_LO_OVERRIDE_EN;
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_FBL_OVRD, val);
+
+	if (!pclk_in) {
+		val = p_ctrl_table->cmnda_pll0_pxdiv_low;
+		cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_PXL_DIVL, val);
+
+		val = p_ctrl_table->cmnda_pll0_pxdiv_high;
+		val |= PLL_PCLK_DIV_EN;
+		cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_PXL_DIVH, val);
+	}
+
+	val = p_pll_table->volt_to_current_coarse;
+	val |= (p_pll_table->volt_to_current) << 4;
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_V2I_TUNE, val);
+
+	val = p_pll_table->charge_pump_gain;
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_CP_TUNE, val);
+
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_LF_PROG, 0x0008);
+
+	val = p_pll_table->pmos_ctrl;
+	val |= (p_pll_table->ndac_ctrl) << 8;
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_PTATIS_TUNE1, val);
+
+	val = p_pll_table->ptat_ndac_ctrl;
+	cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_PTATIS_TUNE2, val);
+
+	if (pclk_in)
+		cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_TEST_MODE, 0x0022);
+	else
+		cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_TEST_MODE, 0x0020);
+
+	cdns_phy_reg_write(cdns_phy, CMN_PSM_CLK_CTRL, 0x0016);
+
+	/* Transceiver control and diagnostic registers */
+	for (k = 0; k < num_lanes; k++) {
+		val = cdns_phy_reg_read(cdns_phy, (XCVR_DIAG_PLLDRC_CTRL | (k << 9)));
+		val &= ~DPLL_CLK_SEL_MODE3;
+		cdns_phy_reg_write(cdns_phy, (XCVR_DIAG_PLLDRC_CTRL | (k << 9)), val);
+	}
+
+	for (k = 0; k < num_lanes; k++) {
+		val = cdns_phy_reg_read(cdns_phy, (TX_DIAG_TX_CTRL | (k << 9)));
+		val &= ~TX_IF_SUBRATE_MODE3_MASK;
+		val |= FIELD_PREP(TX_IF_SUBRATE_MODE3_MASK,
+				  (p_ctrl_table->hsclk_div_tx_sub_rate >> 1));
+		cdns_phy_reg_write(cdns_phy, (TX_DIAG_TX_CTRL | (k << 9)), val);
+	}
+
+	val = cdns_phy_reg_read(cdns_phy, PHY_PMA_CMN_CTRL1);
+	val &= ~CMA_REF_CLK_SEL_MASK;
+	/*
+	 * single ended reference clock (val |= 0x0030);
+	 * differential clock  (val |= 0x0000);
+	 * for differential clock on the refclk_p and
+	 * refclk_m off chip pins: CMN_DIAG_ACYA[8]=1'b1
+	 * cdns_phy_reg_write(cdns_phy, CMN_DIAG_ACYA, 0x0100);
+	 */
+	val |= FIELD_PREP(CMA_REF_CLK_SEL_MASK, 3);
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_CMN_CTRL1, val);
+
+	/* Deassert PHY reset */
+	cdns_phy_reg_write(cdns_phy, PHY_ISO_CMN_CTRL, 0x0001);
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_CMN_CTRL, 0x0003);
+
+	/* Power state machine registers */
+	for (k = 0; k < num_lanes; k++)
+		cdns_phy_reg_write(cdns_phy, XCVR_PSM_RCTRL | (k << 9), 0xfefc);
+
+	/* Assert cmn_macro_pwr_en */
+	cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_CMN_CTRL, 0x0013);
+
+	/* wait for cmn_macro_pwr_en_ack */
+	ret = wait_for_ack(cdns_phy, PHY_PMA_ISO_CMN_CTRL, CMN_MACRO_PWR_EN_ACK,
+			   "MA output macro power up failed");
+	if (ret < 0)
+		return ret;
+
+	/* wait for cmn_ready */
+	ret = wait_for_ack(cdns_phy, PHY_PMA_CMN_CTRL1, CMN_READY,
+			   "PMA output ready failed");
+	if (ret < 0)
+		return ret;
+
+	for (k = 0; k < num_lanes; k++) {
+		cdns_phy_reg_write(cdns_phy, TX_PSC_A0 | (k << 9), 0x6791);
+		cdns_phy_reg_write(cdns_phy, TX_PSC_A1 | (k << 9), 0x6790);
+		cdns_phy_reg_write(cdns_phy, TX_PSC_A2 | (k << 9), 0x0090);
+		cdns_phy_reg_write(cdns_phy, TX_PSC_A3 | (k << 9), 0x0090);
+
+		val = cdns_phy_reg_read(cdns_phy, RX_PSC_CAL | (k << 9));
+		val &= 0xffbb;
+		cdns_phy_reg_write(cdns_phy, RX_PSC_CAL | (k << 9), val);
+
+		val = cdns_phy_reg_read(cdns_phy, RX_PSC_A0 | (k << 9));
+		val &= 0xffbb;
+		cdns_phy_reg_write(cdns_phy, RX_PSC_A0 | (k << 9), val);
+	}
+
+	return 0;
+}
+
+static int hdptx_hdmi_phy_cfg(struct cdns_hdptx_hdmi_phy *cdns_phy, u32 rate)
+{
+	const struct hdptx_hdmi_ctrl *p_ctrl_table;
+	const struct hdptx_hdmi_pll_tuning *p_pll_table;
+	const u32 refclk_freq_khz = cdns_phy->ref_clk_rate / 1000;
+	const bool pclk_in = false;
+	u32 pixel_freq = rate;
+	u32 vco_freq, char_freq;
+	u32 div_total, feedback_factor;
+	u32 i;
+
+	feedback_factor = hdptx_hdmi_feedback_factor(cdns_phy);
+
+	char_freq = pixel_freq * feedback_factor / 1000;
+
+	dev_dbg(cdns_phy->dev,
+		"Pixel clock: (%d KHz), character clock: %d, bpc is (%0d-bit)\n",
+		pixel_freq, char_freq, cdns_phy->bpc);
+
+	/* Get right row from the ctrl_table table.
+	 * Check if 'pixel_freq_khz' value matches the PIXEL_CLK_FREQ column.
+	 * Consider only the rows with FEEDBACK_FACTOR column matching feedback_factor.
+	 */
+	for (i = 0; i < ARRAY_SIZE(pixel_clk_output_ctrl_table); i++) {
+		if (feedback_factor == pixel_clk_output_ctrl_table[i].feedback_factor &&
+		    pixel_freq == pixel_clk_output_ctrl_table[i].pixel_clk_freq_min) {
+			p_ctrl_table = &pixel_clk_output_ctrl_table[i];
+			break;
+		}
+	}
+	if (i == ARRAY_SIZE(pixel_clk_output_ctrl_table)) {
+		dev_warn(cdns_phy->dev,
+			 "Pixel clk (%d KHz) not supported, bpc is (%0d-bit)\n",
+			 pixel_freq, cdns_phy->bpc);
+		return -EINVAL;
+	}
+
+	div_total = p_ctrl_table->pll_fb_div_total;
+	vco_freq = refclk_freq_khz * div_total / p_ctrl_table->cmnda_pll0_ip_div;
+
+	/* Get right row from the pixel_clk_output_pll_table table.
+	 * Check if vco_freq_khz and feedback_div_total
+	 * column matching with pixel_clk_output_pll_table.
+	 */
+	for (i = 0; i < ARRAY_SIZE(pixel_clk_output_pll_table); i++) {
+		if (vco_freq == pixel_clk_output_pll_table[i].vco_freq_min &&
+		    div_total == pixel_clk_output_pll_table[i].feedback_div_total) {
+			p_pll_table = &pixel_clk_output_pll_table[i];
+			break;
+		}
+	}
+	if (i == ARRAY_SIZE(pixel_clk_output_pll_table)) {
+		dev_warn(cdns_phy->dev, "VCO (%d KHz) not supported\n", vco_freq);
+		return -EINVAL;
+	}
+	dev_dbg(cdns_phy->dev, "VCO frequency is (%d KHz)\n", vco_freq);
+
+	return hdptx_hdmi_phy_config(cdns_phy, p_ctrl_table, p_pll_table, pclk_in);
+}
+
+static int hdptx_hdmi_phy_power_up(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	int ret = 0;
+
+	/* set Power State to A2 */
+	cdns_phy_reg_write(cdns_phy, PHY_HDP_MODE_CTRL, POWER_STATE_A2);
+
+	cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_0, 1);
+	cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_1, 1);
+	cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_2, 1);
+	cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_3, 1);
+
+	ret = wait_for_ack(cdns_phy, PHY_HDP_MODE_CTRL, POWER_STATE_A2_ACK,
+			   "Wait A2 Ack failed");
+	if (ret < 0)
+		return ret;
+
+	/* Power up ARC */
+	hdptx_hdmi_arc_config(cdns_phy);
+
+	/* Configure PHY in A0 mode (PHY must be in the A0 power
+	 * state in order to transmit data)
+	 */
+	cdns_phy_reg_write(cdns_phy, PHY_HDP_MODE_CTRL, POWER_STATE_A0);
+
+	return wait_for_ack(cdns_phy, PHY_HDP_MODE_CTRL, POWER_STATE_A0_ACK,
+			    "Wait A0 Ack failed");
+}
+
+static int hdptx_hdmi_phy_power_down(struct cdns_hdptx_hdmi_phy *cdns_phy)
+{
+	u32 val;
+
+	val = cdns_phy_reg_read(cdns_phy, PHY_HDP_MODE_CTRL);
+	val &= ~(POWER_STATE_A0 | POWER_STATE_A1 | POWER_STATE_A2 | POWER_STATE_A3);
+	/* PHY_DP_MODE_CTL set to A3 power state */
+	cdns_phy_reg_write(cdns_phy, PHY_HDP_MODE_CTRL, val | POWER_STATE_A3);
+
+	return wait_for_ack(cdns_phy, PHY_HDP_MODE_CTRL, POWER_STATE_A3_ACK,
+			    "Wait A3 Ack failed");
+}
+
+static int cdns_hdptx_hdmi_phy_on(struct phy *phy)
+{
+	struct cdns_hdptx_hdmi_phy *cdns_phy = phy_get_drvdata(phy);
+
+	return hdptx_hdmi_phy_power_up(cdns_phy);
+}
+
+static int cdns_hdptx_hdmi_phy_off(struct phy *phy)
+{
+	struct cdns_hdptx_hdmi_phy *cdns_phy = phy_get_drvdata(phy);
+
+	hdptx_hdmi_phy_power_down(cdns_phy);
+	return 0;
+}
+
+static int
+cdns_hdptx_hdmi_phy_valid(struct phy *phy, enum phy_mode mode,
+			  int submode, union phy_configure_opts *opts)
+{
+	u32 rate = opts->hdmi.pixel_clk_rate;
+	int i;
+
+	for (i = 0; i < ARRAY_SIZE(pixel_clk_output_ctrl_table); i++)
+		if (rate == pixel_clk_output_ctrl_table[i].pixel_clk_freq_min)
+			return 0;
+
+	return -EINVAL;
+}
+
+static int cdns_hdptx_hdmi_phy_init(struct phy *phy)
+{
+	return 0;
+}
+
+static int cdns_hdptx_hdmi_configure(struct phy *phy,
+				     union phy_configure_opts *opts)
+{
+	struct cdns_hdptx_hdmi_phy *cdns_phy = phy_get_drvdata(phy);
+	int ret;
+
+	cdns_phy->pixel_clk_rate = opts->hdmi.pixel_clk_rate;
+	cdns_phy->color_space = opts->hdmi.color_space;
+	cdns_phy->bpc = opts->hdmi.bpc;
+
+	/* Check HDMI FW alive before HDMI PHY init */
+	ret = hdptx_phy_check_alive(cdns_phy);
+	if (!ret) {
+		dev_err(cdns_phy->dev, "NO HDMI FW running\n");
+		return -ENXIO;
+	}
+
+	/* Configure PHY */
+	if (hdptx_hdmi_phy_cfg(cdns_phy, cdns_phy->pixel_clk_rate) < 0) {
+		dev_err(cdns_phy->dev, "failed to set phy pclock\n");
+		return -EINVAL;
+	}
+
+	ret = hdptx_hdmi_phy_power_up(cdns_phy);
+	if (ret < 0)
+		return ret;
+
+	hdptx_hdmi_phy_set_vswing(cdns_phy);
+
+	return 0;
+}
+
+static const struct phy_ops cdns_hdptx_hdmi_phy_ops = {
+	.init = cdns_hdptx_hdmi_phy_init,
+	.configure = cdns_hdptx_hdmi_configure,
+	.power_on = cdns_hdptx_hdmi_phy_on,
+	.power_off = cdns_hdptx_hdmi_phy_off,
+	.validate = cdns_hdptx_hdmi_phy_valid,
+	.owner = THIS_MODULE,
+};
+
+static int cdns_hdptx_hdmi_phy_probe(struct platform_device *pdev)
+{
+	struct cdns_hdptx_hdmi_phy *cdns_phy;
+	struct device *dev = &pdev->dev;
+	struct device_node *node = dev->of_node;
+	struct phy_provider *phy_provider;
+	struct resource *res;
+	struct phy *phy;
+	int ret;
+
+	cdns_phy = devm_kzalloc(dev, sizeof(*cdns_phy), GFP_KERNEL);
+	if (!cdns_phy)
+		return -ENOMEM;
+
+	dev_set_drvdata(dev, cdns_phy);
+	cdns_phy->dev = dev;
+	mutex_init(&cdns_phy->mbox_mutex);
+
+	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+	if (!res)
+		return -ENODEV;
+	cdns_phy->regs = devm_ioremap(dev, res->start, resource_size(res));
+	if (IS_ERR(cdns_phy->regs))
+		return PTR_ERR(cdns_phy->regs);
+
+	phy = devm_phy_create(dev, node, &cdns_hdptx_hdmi_phy_ops);
+	if (IS_ERR(phy))
+		return PTR_ERR(phy);
+
+	phy->attrs.mode = PHY_MODE_HDMI;
+
+	cdns_phy->phy = phy;
+	phy_set_drvdata(phy, cdns_phy);
+
+	/* init base struct for access mhdp mailbox */
+	cdns_phy->base.dev = cdns_phy->dev;
+	cdns_phy->base.regs = cdns_phy->regs;
+	cdns_phy->base.mbox_mutex = &cdns_phy->mbox_mutex;
+
+	ret = hdptx_hdmi_clk_enable(cdns_phy);
+	if (ret) {
+		dev_err(dev, "Init clk fail\n");
+		return -EINVAL;
+	}
+
+	phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
+	if (IS_ERR(phy_provider)) {
+		ret = PTR_ERR(phy_provider);
+		goto clk_disable;
+	}
+
+	dev_dbg(dev, "probe success!\n");
+
+	return 0;
+
+clk_disable:
+	hdptx_hdmi_clk_disable(cdns_phy);
+
+	return -EINVAL;
+}
+
+static int cdns_hdptx_hdmi_phy_remove(struct platform_device *pdev)
+{
+	struct cdns_hdptx_hdmi_phy *cdns_phy = platform_get_drvdata(pdev);
+
+	hdptx_hdmi_clk_disable(cdns_phy);
+
+	return 0;
+}
+
+static const struct of_device_id cdns_hdptx_hdmi_phy_of_match[] = {
+	{.compatible = "fsl,imx8mq-hdmi-phy" },
+	{ /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, cdns_hdptx_hdmi_phy_of_match);
+
+static struct platform_driver cdns_hdptx_hdmi_phy_driver = {
+	.probe = cdns_hdptx_hdmi_phy_probe,
+	.remove = cdns_hdptx_hdmi_phy_remove,
+	.driver = {
+		.name	= "cdns-hdptx-hdmi-phy",
+		.of_match_table	= cdns_hdptx_hdmi_phy_of_match,
+	}
+};
+module_platform_driver(cdns_hdptx_hdmi_phy_driver);
+
+MODULE_AUTHOR("Sandor Yu <[email protected]>");
+MODULE_DESCRIPTION("Cadence HDP-TX HDMI PHY driver");
+MODULE_LICENSE("GPL");
Signature · 4 lines
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/
author
#12 lore raw

Re: [PATCH v14 7/7] phy: freescale: Add HDMI PHY driver for i.MX8MQ

Hi Alexander,

Thanks for your comments,

Quoted 13 lines

Hi,

thanks for the update.

Am Dienstag, 20. Februar 2024, 04:23:55 CET schrieb Sandor Yu:

Quoted 3 lines

Add Cadence HDP-TX HDMI PHY driver for i.MX8MQ.

Cadence HDP-TX PHY could be put in either DP mode or HDMI mode base

on

Quoted 3 lines

the configuration chosen.
HDMI PHY mode is configurated in the driver.

Quoted 8 lines
Quoted 2 lines

This still works as before. I noticed there is a lot of code duplication with patch 6. IMHO these PHY drivers should be merged into a single one where the mode is configured using phy_set_mode() from cdns-mhdp8501-core.c. This nicely matches my concerns regarding patch 5.

Yes, there are some registers offset are same and the clock management function could be reused for DP and HDMI PHY driver. But because of HDMI and DP PHY totally different work mode, the functions in struct phy_ops,
Such as ->init, ->power_on/off and ->configure could not combine into a single one,
So separate DP and HDMI PHY driver should be a better resolution.

B.R
Sandor

Quoted 1,160 lines

Best regards,
Alexander

Quoted 48 lines

---
v13->v14:
*No change.

v12->v13:
- Fix build warning

v11->v12:
- Adjust clk disable order.
- Return error code to replace -1 for function wait_for_ack().
- Use bool for variable pclk_in.
- Add year 2024 to copyright.

 drivers/phy/freescale/Kconfig               |  10 +
 drivers/phy/freescale/Makefile              |   1 +
 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c | 960
++++++++++++++++++++
 3 files changed, 971 insertions(+)
 create mode 100644 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c
(quoted diff) +0 -0

b/drivers/phy/freescale/Kconfig index c39709fd700ac..14f47b7cc77ab 100644

drivers/phy/freescale/Kconfig +10 -0
@@ -45,6 +45,16 @@ config PHY_FSL_IMX8MQ_DP
        Enable this to support the Cadence HDPTX DP PHY driver
        on i.MX8MQ SOC.
 
+config PHY_FSL_IMX8MQ_HDMI
+     tristate "Freescale i.MX8MQ HDMI PHY support"
+     depends on OF && HAS_IOMEM
+     depends on COMMON_CLK
+     select GENERIC_PHY
+     select CDNS_MHDP_HELPER
+     help
+       Enable this to support the Cadence HDPTX HDMI PHY driver
+       on i.MX8MQ SOC.
+
 endif
 
 config PHY_FSL_LYNX_28G
(quoted diff) +0 -0

b/drivers/phy/freescale/Makefile index 47e5285209fa8..1380ac31c2ead 100644

drivers/phy/freescale/Makefile line 2

Quoted 2 lines
drivers/phy/freescale/Makefile +0 -0
@@ -1,5 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-only
 obj-$(CONFIG_PHY_FSL_IMX8MQ_DP)              +=

phy-fsl-imx8mq-dp.o

Quoted 96 lines
drivers/phy/freescale/Makefile +1 -0
+obj-$(CONFIG_PHY_FSL_IMX8MQ_HDMI)    += phy-fsl-imx8mq-hdmi.o
 obj-$(CONFIG_PHY_FSL_IMX8MQ_USB)     += phy-fsl-imx8mq-usb.o
 obj-$(CONFIG_PHY_MIXEL_LVDS_PHY)     += phy-fsl-imx8qm-lvds-phy.o
 obj-$(CONFIG_PHY_MIXEL_MIPI_DPHY)    += phy-fsl-imx8-mipi-dphy.o
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +85 -0
new file mode 100644
index 0000000000000..537b1f45c91cc
@@ -0,0 +1,960 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Cadence High-Definition Multimedia Interface (HDMI) PHY driver
+ *
+ * Copyright (C) 2022-2024 NXP Semiconductor, Inc.
+ */
+#include <asm/unaligned.h>
+#include <drm/bridge/cdns-mhdp-helper.h> #include <linux/clk.h>
+#include <linux/kernel.h> #include <linux/phy/phy.h> #include
+<linux/platform_device.h> #include <linux/io.h>
+
+#define ADDR_PHY_AFE 0x80000
+
+/* PHY registers */
+#define CMN_SSM_BIAS_TMR                     0x0022
+#define CMN_PLLSM0_USER_DEF_CTRL             0x002f
+#define CMN_PSM_CLK_CTRL                     0x0061
+#define CMN_CDIAG_REFCLK_CTRL                        0x0062
+#define CMN_PLL0_VCOCAL_START                        0x0081
+#define CMN_PLL0_VCOCAL_INIT_TMR             0x0084
+#define CMN_PLL0_VCOCAL_ITER_TMR             0x0085
+#define CMN_TXPUCAL_CTRL                     0x00e0
+#define CMN_TXPDCAL_CTRL                     0x00f0
+#define CMN_TXPU_ADJ_CTRL                    0x0108
+#define CMN_TXPD_ADJ_CTRL                    0x010c
+#define CMN_DIAG_PLL0_FBH_OVRD                       0x01c0
+#define CMN_DIAG_PLL0_FBL_OVRD                       0x01c1
+#define CMN_DIAG_PLL0_OVRD                   0x01c2
+#define CMN_DIAG_PLL0_TEST_MODE                      0x01c4
+#define CMN_DIAG_PLL0_V2I_TUNE                       0x01c5
+#define CMN_DIAG_PLL0_CP_TUNE                        0x01c6
+#define CMN_DIAG_PLL0_LF_PROG                        0x01c7
+#define CMN_DIAG_PLL0_PTATIS_TUNE1           0x01c8
+#define CMN_DIAG_PLL0_PTATIS_TUNE2           0x01c9
+#define CMN_DIAG_PLL0_INCLK_CTRL             0x01ca
+#define CMN_DIAG_PLL0_PXL_DIVH                       0x01cb
+#define CMN_DIAG_PLL0_PXL_DIVL                       0x01cc
+#define CMN_DIAG_HSCLK_SEL                   0x01e0
+#define XCVR_PSM_RCTRL                               0x4001
+#define TX_TXCC_CAL_SCLR_MULT_0                      0x4047
+#define TX_TXCC_CPOST_MULT_00_0                      0x404c
+#define XCVR_DIAG_PLLDRC_CTRL                        0x40e0
+#define XCVR_DIAG_PLLDRC_CTRL                        0x40e0
+#define XCVR_DIAG_HSCLK_SEL                  0x40e1
+#define XCVR_DIAG_BIDI_CTRL                  0x40e8
+#define TX_PSC_A0                            0x4100
+#define TX_PSC_A1                            0x4101
+#define TX_PSC_A2                            0x4102
+#define TX_PSC_A3                            0x4103
+#define TX_DIAG_TX_CTRL                              0x41e0
+#define TX_DIAG_TX_DRV                               0x41e1
+#define TX_DIAG_BGREF_PREDRV_DELAY           0x41e7
+#define TX_DIAG_ACYA_0                               0x41ff
+#define TX_DIAG_ACYA_1                               0x43ff
+#define TX_DIAG_ACYA_2                               0x45ff
+#define TX_DIAG_ACYA_3                               0x47ff
+#define TX_ANA_CTRL_REG_1                    0x5020
+#define TX_ANA_CTRL_REG_2                    0x5021
+#define TX_DIG_CTRL_REG_2                    0x5024
+#define TXDA_CYA_AUXDA_CYA                   0x5025
+#define TX_ANA_CTRL_REG_3                    0x5026
+#define TX_ANA_CTRL_REG_4                    0x5027
+#define TX_ANA_CTRL_REG_5                    0x5029
+#define RX_PSC_A0                            0x8000
+#define RX_PSC_CAL                           0x8006
+#define PHY_HDP_MODE_CTRL                    0xc008
+#define PHY_HDP_CLK_CTL                              0xc009
+#define PHY_ISO_CMN_CTRL                     0xc010
+#define PHY_PMA_CMN_CTRL1                    0xc800
+#define PHY_PMA_ISO_CMN_CTRL                 0xc810
+#define PHY_PMA_ISO_PLL_CTRL1                        0xc812
+#define PHY_PMA_ISOLATION_CTRL                       0xc81f
+
+/* PHY_HDP_CLK_CTL */
+#define PLL_DATA_RATE_CLK_DIV_MASK           GENMASK(15, 8)
+#define PLL_DATA_RATE_CLK_DIV_HBR            0x24
+#define PLL_DATA_RATE_CLK_DIV_HBR2           0x12
+#define PLL_CLK_EN_ACK_EN                    BIT(3)
+#define PLL_CLK_EN                           BIT(2)
+#define PLL_READY                            BIT(1)
+#define PLL_EN                                       BIT(0)
+
+/* PHY_PMA_CMN_CTRL1 */
+#define CMA_REF_CLK_DIG_DIV_MASK             GENMASK(13, 12)
+#define CMA_REF_CLK_SEL_MASK                 GENMASK(6, 4)

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 95

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+#define CMA_REF_CLK_RCV_EN_MASK                      BIT(3)
+#define CMA_REF_CLK_RCV_EN                   1
+#define CMN_READY                            BIT(0)
+
+/* PHY_PMA_ISO_PLL_CTRL1 */
+#define CMN_PLL0_CLK_DATART_DIV_MASK         GENMASK(7, 0)
+
+/* TX_DIAG_TX_DRV */
+#define TX_DRIVER_PROG_BOOST_ENABLE          BIT(10)
+#define TX_DRIVER_PROG_BOOST_LEVEL_MASK

GENMASK(9, 8)

Quoted 4 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +4 -0
+#define TX_DRIVER_LDO_BG_DEPENDENT_REF_ENABLE        BIT(7)
+#define TX_DRIVER_LDO_BANDGAP_REF_ENABLE     BIT(6)
+
+/* TX_TXCC_CAL_SCLR_MULT_0 */

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+#define SCALED_RESISTOR_CALIBRATION_CODE_ADD BIT(8)
+#define RESISTOR_CAL_MULT_VAL_32_128         BIT(5)
+
+/* CMN_CDIAG_REFCLK_CTRL */
+#define DIG_REF_CLK_DIV_SCALER_MASK          GENMASK(14, 12)
+#define REFCLK_TERMINATION_EN_OVERRIDE_EN    BIT(7)
+#define REFCLK_TERMINATION_EN_OVERRIDE               BIT(6)
+
+/* CMN_DIAG_HSCLK_SEL */
+#define HSCLK1_SEL_MASK

GENMASK(5, 4)

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 110

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+#define HSCLK0_SEL_MASK

GENMASK(1, 0)

Quoted 75 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +75 -0
+
+/* XCVR_DIAG_HSCLK_SEL */
+#define HSCLK_SEL_MODE3_MASK                 GENMASK(13, 12)
+#define HSCLK_SEL_MODE3_HSCLK1                       1
+
+/* CMN_PLL0_VCOCAL_START */
+#define VCO_CALIB_CODE_START_POINT_VAL_MASK  GENMASK(8, 0)
+
+/* CMN_DIAG_PLL0_FBH_OVRD */
+#define PLL_FEEDBACK_DIV_HI_OVERRIDE_EN              BIT(15)
+
+/* CMN_DIAG_PLL0_FBL_OVRD */
+#define PLL_FEEDBACK_DIV_LO_OVERRIDE_EN              BIT(15)
+
+/* CMN_DIAG_PLL0_PXL_DIVH */
+#define PLL_PCLK_DIV_EN                              BIT(15)
+
+/* XCVR_DIAG_PLLDRC_CTRL */
+#define DPLL_CLK_SEL_MODE3                   BIT(14)
+
+/* TX_DIAG_TX_CTRL */
+#define TX_IF_SUBRATE_MODE3_MASK             GENMASK(7, 6)
+
+/* PHY_HDP_MODE_CTRL */
+#define POWER_STATE_A3_ACK                   BIT(7)
+#define POWER_STATE_A2_ACK                   BIT(6)
+#define POWER_STATE_A1_ACK                   BIT(5)
+#define POWER_STATE_A0_ACK                   BIT(4)
+#define POWER_STATE_A3                               BIT(3)
+#define POWER_STATE_A2                               BIT(2)
+#define POWER_STATE_A1                               BIT(1)
+#define POWER_STATE_A0                               BIT(0)
+
+/* PHY_PMA_ISO_CMN_CTRL */
+#define CMN_MACRO_PWR_EN_ACK                 BIT(5)
+
+#define KEEP_ALIVE           0x18
+
+#define REF_CLK_27MHZ                27000000
+
+/* HDMI TX clock control settings */
+struct hdptx_hdmi_ctrl {
+     u32 pixel_clk_freq_min;
+     u32 pixel_clk_freq_max;
+     u32 feedback_factor;
+     u32 data_range_kbps_min;
+     u32 data_range_kbps_max;
+     u32 cmnda_pll0_ip_div;
+     u32 cmn_ref_clk_dig_div;
+     u32 ref_clk_divider_scaler;
+     u32 pll_fb_div_total;
+     u32 cmnda_pll0_fb_div_low;
+     u32 cmnda_pll0_fb_div_high;
+     u32 pixel_div_total;
+     u32 cmnda_pll0_pxdiv_low;
+     u32 cmnda_pll0_pxdiv_high;
+     u32 vco_freq_min;
+     u32 vco_freq_max;
+     u32 vco_ring_select;
+     u32 cmnda_hs_clk_0_sel;
+     u32 cmnda_hs_clk_1_sel;
+     u32 hsclk_div_at_xcvr;
+     u32 hsclk_div_tx_sub_rate;
+     u32 cmnda_pll0_hs_sym_div_sel;
+     u32 cmnda_pll0_clk_freq_min;
+     u32 cmnda_pll0_clk_freq_max;
+};
+
+struct cdns_hdptx_hdmi_phy {
+     struct cdns_mhdp_base base;
+
+     void __iomem *regs;     /* DPTX registers base */
+     struct mutex mbox_mutex; /* mutex to protect mailbox */
+     struct device *dev;
+     struct phy *phy;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     struct clk *ref_clk, *apb_clk;
+     u32 ref_clk_rate;
+     u32 pixel_clk_rate;
+     enum hdmi_colorspace color_space;
+     u32 bpc;
+};
+
+/* HDMI TX clock control settings, pixel clock is output */ static
+const struct hdptx_hdmi_ctrl pixel_clk_output_ctrl_table[] = {
+/*Minclk  Maxclk Fdbak  DR_min   DR_max  ip_d  dig  DS    Totl

*/

Quoted 82 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +82 -0
+{ 27000,  27000, 1000,  270000,  270000, 0x03, 0x1, 0x1,  240, 0x0bc,
+0x030,  80, 0x026, 0x026, 2160000, 2160000, 0, 2, 2, 2, 4, 0x3,
+27000,  27000}, { 27000,  27000, 1250,  337500,  337500, 0x03, 0x1,
+0x1,  300, 0x0ec, 0x03c, 100, 0x030, 0x030, 2700000, 2700000, 0, 2,
+2, 2, 4, 0x3,  33750,  33750}, { 27000,  27000, 1500,  405000,
+405000, 0x03, 0x1, 0x1,  360, 0x11c, 0x048, 120, 0x03a, 0x03a,
+3240000, 3240000, 0, 2, 2, 2, 4, 0x3,  40500,  40500}, { 27000,
+27000, 2000,  540000,  540000, 0x03, 0x1, 0x1,  240, 0x0bc, 0x030,
+80, 0x026, 0x026, 2160000, 2160000, 0, 2, 2, 2, 4, 0x2,  54000,
+54000}, { 54000,  54000, 1000,  540000,  540000, 0x03, 0x1, 0x1,
+480, 0x17c, 0x060,  80, 0x026, 0x026, 4320000, 4320000, 1, 2, 2, 2,
+4, 0x3,  54000,  54000}, { 54000,  54000, 1250,  675000,  675000,
+0x04, 0x1, 0x1,  400, 0x13c, 0x050,  50, 0x017, 0x017, 2700000,
+2700000, 0, 1, 1, 2, 4, 0x2,  67500,  67500}, { 54000,  54000, 1500,
+810000,  810000, 0x04, 0x1, 0x1,  480, 0x17c, 0x060,  60, 0x01c,
+0x01c, 3240000, 3240000, 0, 2, 2, 2, 2, 0x2,  81000,  81000}, {
+54000,  54000, 2000, 1080000, 1080000, 0x03, 0x1, 0x1,  240, 0x0bc,
+0x030,  40, 0x012, 0x012, 2160000, 2160000, 0, 2, 2, 2, 1, 0x1,
+108000, 108000}, { 74250,  74250, 1000,  742500,  742500, 0x03, 0x1,
+0x1,  660, 0x20c, 0x084,  80, 0x026, 0x026, 5940000, 5940000, 1, 2,
+2, 2, 4, 0x3,  74250,  74250}, { 74250,  74250, 1250,  928125,
+928125, 0x04, 0x1, 0x1,  550, 0x1b4, 0x06e,  50, 0x017, 0x017,
+3712500, 3712500, 1, 1, 1, 2, 4, 0x2,  92812,  92812}, { 74250,
+74250, 1500, 1113750, 1113750, 0x04, 0x1, 0x1,  660, 0x20c, 0x084,
+60, 0x01c, 0x01c, 4455000, 4455000, 1, 2, 2, 2, 2, 0x2, 111375,
+111375}, { 74250,  74250, 2000, 1485000, 1485000, 0x03, 0x1, 0x1,
+330, 0x104, 0x042,  40, 0x012, 0x012, 2970000, 2970000, 0, 2, 2, 2,
+1, 0x1, 148500, 148500}, { 99000,  99000, 1000,  990000,  990000,
+0x03, 0x1, 0x1,  440, 0x15c, 0x058,  40, 0x012, 0x012, 3960000,
+3960000, 1, 2, 2, 2, 2, 0x2,  99000,  99000}, { 99000,  99000, 1250,
+1237500, 1237500, 0x03, 0x1, 0x1,  275, 0x0d8, 0x037,  25, 0x00b,
+0x00a, 2475000, 2475000, 0, 1, 1, 2, 2, 0x1, 123750, 123750}, {
+99000,  99000, 1500, 1485000, 1485000, 0x03, 0x1, 0x1,  330, 0x104,
+0x042,  30, 0x00d, 0x00d, 2970000, 2970000, 0, 2, 2, 2, 1, 0x1,
+148500, 148500}, { 99000,  99000, 2000, 1980000, 1980000, 0x03, 0x1,
+0x1,  440, 0x15c, 0x058,  40, 0x012, 0x012, 3960000, 3960000, 1, 2,
+2, 2, 1, 0x1, 198000, 198000}, {148500, 148500, 1000, 1485000,
+1485000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  40, 0x012, 0x012,
+5940000, 5940000, 1, 2, 2, 2, 2, 0x2, 148500, 148500}, {148500,
+148500, 1250, 1856250, 1856250, 0x04, 0x1, 0x1,  550, 0x1b4, 0x06e,
+25, 0x00b, 0x00a, 3712500, 3712500, 1, 1, 1, 2, 2, 0x1, 185625,
+185625}, {148500, 148500, 1500, 2227500, 2227500, 0x03, 0x1, 0x1,
+495, 0x188, 0x063,  30, 0x00d, 0x00d, 4455000, 4455000, 1, 1, 1, 2,
+2, 0x1, 222750, 222750}, {148500, 148500, 2000, 2970000, 2970000,
+0x03, 0x1, 0x1,  660, 0x20c, 0x084,  40, 0x012, 0x012, 5940000,
+5940000, 1, 2, 2, 2, 1, 0x1, 297000, 297000}, {198000, 198000, 1000,
+1980000, 1980000, 0x03, 0x1, 0x1,  220, 0x0ac, 0x02c,  10, 0x003,
+0x003, 1980000, 1980000, 0, 1, 1, 2, 1, 0x0, 198000, 198000},
+{198000, 198000, 1250, 2475000, 2475000, 0x03, 0x1, 0x1,  550, 0x1b4,
+0x06e,  25, 0x00b, 0x00a, 4950000, 4950000, 1, 1, 1, 2, 2, 0x1,
+247500, 247500}, {198000, 198000, 1500, 2970000, 2970000, 0x03, 0x1,
+0x1,  330, 0x104, 0x042,  15, 0x006, 0x005, 2970000, 2970000, 0, 1,
+1, 2, 1, 0x0, 297000, 297000}, {198000, 198000, 2000, 3960000,
+3960000, 0x03, 0x1, 0x1,  440, 0x15c, 0x058,  20, 0x008, 0x008,
+3960000, 3960000, 1, 1, 1, 2, 1, 0x0, 396000, 396000}, {297000,
+297000, 1000, 2970000, 2970000, 0x03, 0x1, 0x1,  330, 0x104, 0x042,
+10, 0x003, 0x003, 2970000, 2970000, 0, 1, 1, 2, 1, 0x0, 297000,
+297000}, {297000, 297000, 1500, 4455000, 4455000, 0x03, 0x1, 0x1,
+495, 0x188, 0x063,  15, 0x006, 0x005, 4455000, 4455000, 1, 1, 1, 2,
+1, 0x0, 445500, 445500}, {297000, 297000, 2000, 5940000, 5940000,
+0x03, 0x1, 0x1,  660, 0x20c, 0x084,  20, 0x008, 0x008, 5940000,
+5940000, 1, 1, 1, 2, 1, 0x0, 594000, 594000}, {594000, 594000, 1000,
+5940000, 5940000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  10, 0x003,
+0x003, 5940000, 5940000, 1, 1, 1, 2, 1, 0x0, 594000, 594000},
+{594000, 594000,  750, 4455000, 4455000, 0x03, 0x1, 0x1,  495, 0x188,
+0x063,  10, 0x003, 0x003, 4455000, 4455000, 1, 1, 1, 2, 1, 0x0,
+445500, 445500}, {594000, 594000,  625, 3712500, 3712500, 0x04, 0x1,
+0x1,  550, 0x1b4, 0x06e,  10, 0x003, 0x003, 3712500, 3712500, 1, 1,
+1, 2, 1, 0x0, 371250, 371250}, {594000, 594000,  500, 2970000,
+2970000, 0x03, 0x1, 0x1,  660, 0x20c, 0x084,  10, 0x003, 0x003,
+5940000, 5940000, 1, 1, 1, 2, 2, 0x1, 297000, 297000}, };
+
+/* HDMI TX PLL tuning settings */
+struct hdptx_hdmi_pll_tuning {
+     u32 vco_freq_bin;
+     u32 vco_freq_min;
+     u32 vco_freq_max;
+     u32 volt_to_current_coarse;
+     u32 volt_to_current;
+     u32 ndac_ctrl;
+     u32 pmos_ctrl;
+     u32 ptat_ndac_ctrl;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 287

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     u32 feedback_div_total;
+     u32 charge_pump_gain;
+     u32 coarse_code;
+     u32 v2i_code;
+     u32 vco_cal_code;
+};
+
+/* HDMI TX PLL tuning settings, pixel clock is output */ static const
+struct hdptx_hdmi_pll_tuning pixel_clk_output_pll_table[] = { /*bin
+VCO_freq min/max  coar  cod NDAC  PMOS PTAT div-T P-Gain Coa V2I

CAL

Quoted 12 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +12 -0
+*/ {  1, 1980000, 1980000, 0x4, 0x3, 0x0, 0x09, 0x09, 220, 0x42, 160,
+5, 183 }, {  2, 2160000, 2160000, 0x4, 0x3, 0x0, 0x09, 0x09, 240,
+0x42, 166, 6, 208 }, {  3, 2475000, 2475000, 0x5, 0x3, 0x1, 0x00,
+0x07, 275, 0x42, 167, 6, 209 }, {  4, 2700000, 2700000, 0x5, 0x3,
+0x1, 0x00, 0x07, 300, 0x42, 188, 6, 230 }, {  4, 2700000, 2700000,
+0x5, 0x3, 0x1, 0x00, 0x07, 400, 0x4c, 188, 6, 230 }, {  5, 2970000,
+2970000, 0x6, 0x3, 0x1, 0x00, 0x07, 330, 0x42, 183, 6, 225 }, {  6,
+3240000, 3240000, 0x6, 0x3, 0x1, 0x00, 0x07, 360, 0x42, 203, 7, 256
+}, {  6, 3240000, 3240000, 0x6, 0x3, 0x1, 0x00, 0x07, 480, 0x4c, 203,
+7, 256 }, {  7, 3712500, 3712500, 0x4, 0x3, 0x0, 0x07, 0x0F, 550,
+0x4c, 212, 7, 257 }, {  8, 3960000, 3960000, 0x5, 0x3, 0x0, 0x07,
+0x0F, 440, 0x42, 184, 6, 226 }, {  9, 4320000, 4320000, 0x5, 0x3,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 309

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+0x1, 0x07, 0x0F, 480, 0x42, 205, 7, 258 }, { 10, 4455000, 4455000,
+0x5, 0x3, 0x0, 0x07, 0x0F, 495, 0x42, 219, 7, 272 }, { 10, 4455000,
+4455000, 0x5, 0x3, 0x0, 0x07, 0x0F, 660, 0x4c, 219, 7, 272 }, { 11,
+4950000, 4950000, 0x6, 0x3, 0x1, 0x00, 0x07, 550, 0x42, 213, 7, 258
+}, { 12, 5940000, 5940000, 0x7, 0x3, 0x1, 0x00, 0x07, 660, 0x42, 244,
+8, 292 }, };
+
+static int cdns_phy_reg_write(struct cdns_hdptx_hdmi_phy *cdns_phy,
+u32 addr, u32 val) {
+     return cdns_mhdp_reg_write(&cdns_phy->base, ADDR_PHY_AFE +

(addr

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 316

Quoted 7 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +7 -0
+<< 2), val); }
+
+static u32 cdns_phy_reg_read(struct cdns_hdptx_hdmi_phy *cdns_phy,
+u32 addr) {
+     u32 reg32;
+
+     cdns_mhdp_reg_read(&cdns_phy->base, ADDR_PHY_AFE + (addr <<

2),

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 6 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +6 -0
+ &reg32);
+
+     return reg32;
+}
+
+static int wait_for_ack(struct cdns_hdptx_hdmi_phy *cdns_phy, u32 reg,

u32 mask,

Quoted 56 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +56 -0
+                     const char *err_msg) {
+     u32 val, i;
+
+     for (i = 0; i < 10; i++) {
+             val = cdns_phy_reg_read(cdns_phy, reg);
+             if (val & mask)
+                     return 0;
+             msleep(20);
+     }
+
+     dev_err(cdns_phy->dev, "%s\n", err_msg);
+     return -ETIMEDOUT;
+}
+
+static bool hdptx_phy_check_alive(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     u32  alive, newalive;
+     u8 retries_left = 50;
+
+     alive = readl(cdns_phy->regs + KEEP_ALIVE);
+
+     while (retries_left--) {
+             udelay(2);
+
+             newalive = readl(cdns_phy->regs + KEEP_ALIVE);
+             if (alive == newalive)
+                     continue;
+             return true;
+     }
+     return false;
+}
+
+static int hdptx_hdmi_clk_enable(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     struct device *dev = cdns_phy->dev;
+     u32 ref_clk_rate;
+     int ret;
+
+     cdns_phy->ref_clk = devm_clk_get(dev, "ref");
+     if (IS_ERR(cdns_phy->ref_clk)) {
+             dev_err(dev, "phy ref clock not found\n");
+             return PTR_ERR(cdns_phy->ref_clk);
+     }
+
+     cdns_phy->apb_clk = devm_clk_get(dev, "apb");
+     if (IS_ERR(cdns_phy->apb_clk)) {
+             dev_err(dev, "phy apb clock not found\n");
+             return PTR_ERR(cdns_phy->apb_clk);
+     }
+
+     ret = clk_prepare_enable(cdns_phy->ref_clk);
+     if (ret) {
+             dev_err(cdns_phy->dev, "Failed to prepare ref clock\n");
+             return ret;
+     }
+

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     ref_clk_rate = clk_get_rate(cdns_phy->ref_clk);
+     if (!ref_clk_rate) {
+             dev_err(cdns_phy->dev, "Failed to get ref clock rate\n");
+             goto err_ref_clk;
+     }
+
+     if (ref_clk_rate == REF_CLK_27MHZ) {
+             cdns_phy->ref_clk_rate = ref_clk_rate;
+     } else {
+             dev_err(cdns_phy->dev, "Not support Ref Clock

Rate(%dHz)\n", ref_clk_rate);

Quoted 26 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +26 -0
+             goto err_ref_clk;
+     }
+
+     ret = clk_prepare_enable(cdns_phy->apb_clk);
+     if (ret) {
+             dev_err(cdns_phy->dev, "Failed to prepare apb clock\n");
+             goto err_ref_clk;
+     }
+
+     return 0;
+
+err_ref_clk:
+     clk_disable_unprepare(cdns_phy->ref_clk);
+     return -EINVAL;
+}
+
+static void hdptx_hdmi_clk_disable(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     clk_disable_unprepare(cdns_phy->apb_clk);
+     clk_disable_unprepare(cdns_phy->ref_clk);
+}
+
+static void hdptx_hdmi_arc_config(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     u16 txpu_calib_code;
+     u16 txpd_calib_code;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     u16 txpu_adj_calib_code;
+     u16 txpd_adj_calib_code;
+     u16 prev_calib_code;
+     u16 new_calib_code;
+     u16 rdata;
+
+     /* Power ARC */
+     cdns_phy_reg_write(cdns_phy, TXDA_CYA_AUXDA_CYA, 0x0001);
+
+     prev_calib_code = cdns_phy_reg_read(cdns_phy,

TX_DIG_CTRL_REG_2);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 425

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+     txpu_calib_code = cdns_phy_reg_read(cdns_phy,

CMN_TXPUCAL_CTRL);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 426

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+     txpd_calib_code = cdns_phy_reg_read(cdns_phy,

CMN_TXPDCAL_CTRL);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 427

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+     txpu_adj_calib_code = cdns_phy_reg_read(cdns_phy,

CMN_TXPU_ADJ_CTRL);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 8 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +8 -0
+     txpd_adj_calib_code = cdns_phy_reg_read(cdns_phy,
+ CMN_TXPD_ADJ_CTRL);
+
+     new_calib_code = ((txpu_calib_code + txpd_calib_code) / 2)
+             + txpu_adj_calib_code + txpd_adj_calib_code;
+
+     if (new_calib_code != prev_calib_code) {
+             rdata = cdns_phy_reg_read(cdns_phy,

TX_ANA_CTRL_REG_1);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 437

Quoted 2 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +2 -0
+             rdata &= 0xdfff;
+             cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1,

rdata);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 438

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+             cdns_phy_reg_write(cdns_phy, TX_DIG_CTRL_REG_2,

new_calib_code);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 3 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +3 -0
+             mdelay(10);
+             rdata |= 0x2000;
+             cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1,

rdata);

Quoted 22 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +22 -0
+             usleep_range(150, 250);
+     }
+
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x0100);
+     usleep_range(100, 200);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x0300);
+     usleep_range(100, 200);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_3, 0x0000);
+     usleep_range(100, 200);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2008);
+     usleep_range(100, 200);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2018);
+     usleep_range(100, 200);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2098);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x030c);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_5, 0x0010);
+     usleep_range(100, 200);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_4, 0x4001);
+     mdelay(5);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_1, 0x2198);
+     mdelay(5);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x030d);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     usleep_range(100, 200);
+     cdns_phy_reg_write(cdns_phy, TX_ANA_CTRL_REG_2, 0x030f); }
+
+static void hdptx_hdmi_phy_set_vswing(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     u32 k;
+     const u32 num_lanes = 4;
+
+     for (k = 0; k < num_lanes; k++) {
+             cdns_phy_reg_write(cdns_phy, (TX_DIAG_TX_DRV | (k <<

9)),

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 474

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+

TX_DRIVER_PROG_BOOST_ENABLE |

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 475

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+

FIELD_PREP(TX_DRIVER_PROG_BOOST_LEVEL_MASK, 3) |

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 476

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+

TX_DRIVER_LDO_BG_DEPENDENT_REF_ENABLE |

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 477

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+

TX_DRIVER_LDO_BANDGAP_REF_ENABLE);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 478

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+             cdns_phy_reg_write(cdns_phy,

(TX_TXCC_CPOST_MULT_00_0 | (k << 9)), 0x0);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 479

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+             cdns_phy_reg_write(cdns_phy,

(TX_TXCC_CAL_SCLR_MULT_0 | (k << 9)),

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 480

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+

SCALED_RESISTOR_CALIBRATION_CODE_ADD |

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 481

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+

RESISTOR_CAL_MULT_VAL_32_128);

Quoted 18 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +18 -0
+     }
+}
+
+static int hdptx_hdmi_feedback_factor(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     u32 feedback_factor;
+
+     switch (cdns_phy->color_space) {
+     case HDMI_COLORSPACE_YUV422:
+             feedback_factor = 1000;
+             break;
+
+     case HDMI_COLORSPACE_YUV420:
+             switch (cdns_phy->bpc) {
+             case 8:
+                     feedback_factor = 500;
+                     break;
+             case 10:

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+                     feedback_factor = 625;
+                     break;
+             case 12:
+                     feedback_factor = 750;
+                     break;
+             case 16:
+                     feedback_factor = 1000;
+                     break;
+             default:
+                     dev_dbg(cdns_phy->dev, "Invalid

ColorDepth\n");

Quoted 16 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +16 -0
+                     return 0;
+             }
+             break;
+
+     default:
+             /* Assume RGB/YUV444 */
+             switch (cdns_phy->bpc) {
+             case 10:
+                     feedback_factor = 1250;
+                     break;
+             case 12:
+                     feedback_factor = 1500;
+                     break;
+             case 16:
+                     feedback_factor = 2000;
+                     break;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+             default:
+                     feedback_factor = 1000;
+             }
+     }
+
+     return feedback_factor;
+}
+
+static int hdptx_hdmi_phy_config(struct cdns_hdptx_hdmi_phy *cdns_phy,
+                              const struct hdptx_hdmi_ctrl

*p_ctrl_table,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 536

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+                              const struct hdptx_hdmi_pll_tuning

*p_pll_table,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 7 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +7 -0
+                              bool pclk_in) {
+     const u32 num_lanes = 4;
+     u32 val, k;
+     int ret;
+
+     /* enable PHY isolation mode only for CMN */
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_ISOLATION_CTRL,

0xd000);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 545

Quoted 2 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +2 -0
+
+     /* set cmn_pll0_clk_datart1_div/cmn_pll0_clk_datart0_div dividers

*/

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 13 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +13 -0
+     val = cdns_phy_reg_read(cdns_phy, PHY_PMA_ISO_PLL_CTRL1);
+     val &= ~CMN_PLL0_CLK_DATART_DIV_MASK;
+     val |= FIELD_PREP(CMN_PLL0_CLK_DATART_DIV_MASK, 0x12);
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_PLL_CTRL1, val);
+
+     /* assert PHY reset from isolation register */
+     cdns_phy_reg_write(cdns_phy, PHY_ISO_CMN_CTRL, 0x0000);
+     /* assert PMA CMN reset */
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_CMN_CTRL, 0x0000);
+
+     /* register XCVR_DIAG_BIDI_CTRL */
+     for (k = 0; k < num_lanes; k++)
+             cdns_phy_reg_write(cdns_phy, XCVR_DIAG_BIDI_CTRL |

(k <<

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 6 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +6 -0
+ 9), 0x00ff);
+
+     /* Describing Task phy_cfg_hdp */
+     val = cdns_phy_reg_read(cdns_phy, PHY_PMA_CMN_CTRL1);
+     val &= ~CMA_REF_CLK_RCV_EN_MASK;
+     val |= FIELD_PREP(CMA_REF_CLK_RCV_EN_MASK,

CMA_REF_CLK_RCV_EN);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 6 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +6 -0
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_CMN_CTRL1, val);
+
+     /* PHY Registers */
+     val = cdns_phy_reg_read(cdns_phy, PHY_PMA_CMN_CTRL1);
+     val &= ~CMA_REF_CLK_DIG_DIV_MASK;
+     val |= FIELD_PREP(CMA_REF_CLK_DIG_DIV_MASK,

p_ctrl_table->cmn_ref_clk_dig_div);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 12 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +12 -0
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_CMN_CTRL1, val);
+
+     val = cdns_phy_reg_read(cdns_phy, PHY_HDP_CLK_CTL);
+     val &= ~PLL_DATA_RATE_CLK_DIV_MASK;
+     val |= FIELD_PREP(PLL_DATA_RATE_CLK_DIV_MASK,
+                       PLL_DATA_RATE_CLK_DIV_HBR2);
+     cdns_phy_reg_write(cdns_phy, PHY_HDP_CLK_CTL, val);
+
+     /* Common control module control and diagnostic registers */
+     val = cdns_phy_reg_read(cdns_phy, CMN_CDIAG_REFCLK_CTRL);
+     val &= ~DIG_REF_CLK_DIV_SCALER_MASK;
+     val |= FIELD_PREP(DIG_REF_CLK_DIV_SCALER_MASK,

p_ctrl_table->ref_clk_divider_scaler);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 583

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+     val |= REFCLK_TERMINATION_EN_OVERRIDE_EN |

REFCLK_TERMINATION_EN_OVERRIDE;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 6 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +6 -0
+     cdns_phy_reg_write(cdns_phy, CMN_CDIAG_REFCLK_CTRL, val);
+
+     /* High speed clock used */
+     val = cdns_phy_reg_read(cdns_phy, CMN_DIAG_HSCLK_SEL);
+     val &= ~(HSCLK1_SEL_MASK | HSCLK0_SEL_MASK);
+     val |= FIELD_PREP(HSCLK1_SEL_MASK,

(p_ctrl_table->cmnda_hs_clk_1_sel >> 1));

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 590

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+     val |= FIELD_PREP(HSCLK0_SEL_MASK,

(p_ctrl_table->cmnda_hs_clk_0_sel >> 1));

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 4 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +4 -0
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_HSCLK_SEL, val);
+
+     for (k = 0; k < num_lanes; k++) {
+             val = cdns_phy_reg_read(cdns_phy,

(XCVR_DIAG_HSCLK_SEL | (k << 9)));

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 562

Quoted 3 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +3 -0
+             val &= ~HSCLK_SEL_MODE3_MASK;
+             val |= FIELD_PREP(HSCLK_SEL_MODE3_MASK,
+

(p_ctrl_table->cmnda_hs_clk_0_sel >> 1));

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 598

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+             cdns_phy_reg_write(cdns_phy, (XCVR_DIAG_HSCLK_SEL |

(k << 9)), val);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 5 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +5 -0
+     }
+
+     /* PLL 0 control state machine registers */
+     val = p_ctrl_table->vco_ring_select << 12;
+     cdns_phy_reg_write(cdns_phy, CMN_PLLSM0_USER_DEF_CTRL,

val);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 5 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +5 -0
+
+     if (pclk_in) {
+             val = 0x30a0;
+     } else {
+             val = cdns_phy_reg_read(cdns_phy,

CMN_PLL0_VCOCAL_START);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 610

Quoted 2 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +2 -0
+             val &= ~VCO_CALIB_CODE_START_POINT_VAL_MASK;
+             val |=

FIELD_PREP(VCO_CALIB_CODE_START_POINT_VAL_MASK,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 5 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +5 -0
+                               p_pll_table->vco_cal_code);
+     }
+     cdns_phy_reg_write(cdns_phy, CMN_PLL0_VCOCAL_START, val);
+
+     cdns_phy_reg_write(cdns_phy, CMN_PLL0_VCOCAL_INIT_TMR,

0x0064);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 616

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+     cdns_phy_reg_write(cdns_phy, CMN_PLL0_VCOCAL_ITER_TMR,

0x000a);

Quoted 9 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +9 -0
+
+     /* Common functions control and diagnostics registers */
+     val = p_ctrl_table->cmnda_pll0_hs_sym_div_sel << 8;
+     val |= p_ctrl_table->cmnda_pll0_ip_div;
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_INCLK_CTRL, val);
+
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_OVRD, 0x0000);
+
+     val = p_ctrl_table->cmnda_pll0_fb_div_high;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     val |= PLL_FEEDBACK_DIV_HI_OVERRIDE_EN;
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_FBH_OVRD, val);
+
+     val = p_ctrl_table->cmnda_pll0_fb_div_low;
+     val |= PLL_FEEDBACK_DIV_LO_OVERRIDE_EN;
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_FBL_OVRD, val);
+
+     if (!pclk_in) {
+             val = p_ctrl_table->cmnda_pll0_pxdiv_low;
+             cdns_phy_reg_write(cdns_phy,

CMN_DIAG_PLL0_PXL_DIVL,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 5 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +5 -0
+ val);
+
+             val = p_ctrl_table->cmnda_pll0_pxdiv_high;
+             val |= PLL_PCLK_DIV_EN;
+             cdns_phy_reg_write(cdns_phy,

CMN_DIAG_PLL0_PXL_DIVH, val);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     }
+
+     val = p_pll_table->volt_to_current_coarse;
+     val |= (p_pll_table->volt_to_current) << 4;
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_V2I_TUNE, val);
+
+     val = p_pll_table->charge_pump_gain;
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_CP_TUNE, val);
+
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_LF_PROG,

0x0008);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 4 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +4 -0
+
+     val = p_pll_table->pmos_ctrl;
+     val |= (p_pll_table->ndac_ctrl) << 8;
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_PTATIS_TUNE1,

val);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 3 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +3 -0
+
+     val = p_pll_table->ptat_ndac_ctrl;
+     cdns_phy_reg_write(cdns_phy, CMN_DIAG_PLL0_PTATIS_TUNE2,

val);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 3 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +3 -0
+
+     if (pclk_in)
+             cdns_phy_reg_write(cdns_phy,

CMN_DIAG_PLL0_TEST_MODE, 0x0022);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 662

Quoted 2 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +2 -0
+     else
+             cdns_phy_reg_write(cdns_phy,

CMN_DIAG_PLL0_TEST_MODE,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 7 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +7 -0
+ 0x0020);
+
+     cdns_phy_reg_write(cdns_phy, CMN_PSM_CLK_CTRL, 0x0016);
+
+     /* Transceiver control and diagnostic registers */
+     for (k = 0; k < num_lanes; k++) {
+             val = cdns_phy_reg_read(cdns_phy,

(XCVR_DIAG_PLLDRC_CTRL | (k << 9)));

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 671

Quoted 2 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +2 -0
+             val &= ~DPLL_CLK_SEL_MODE3;
+             cdns_phy_reg_write(cdns_phy,

(XCVR_DIAG_PLLDRC_CTRL | (k << 9)), val);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 4 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +4 -0
+     }
+
+     for (k = 0; k < num_lanes; k++) {
+             val = cdns_phy_reg_read(cdns_phy, (TX_DIAG_TX_CTRL |

(k << 9)));

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 641

Quoted 3 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +3 -0
+             val &= ~TX_IF_SUBRATE_MODE3_MASK;
+             val |= FIELD_PREP(TX_IF_SUBRATE_MODE3_MASK,
+

(p_ctrl_table->hsclk_div_tx_sub_rate >> 1));

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 679

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+             cdns_phy_reg_write(cdns_phy, (TX_DIAG_TX_CTRL | (k <<

9)), val);

Quoted 11 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +11 -0
+     }
+
+     val = cdns_phy_reg_read(cdns_phy, PHY_PMA_CMN_CTRL1);
+     val &= ~CMA_REF_CLK_SEL_MASK;
+     /*
+      * single ended reference clock (val |= 0x0030);
+      * differential clock  (val |= 0x0000);
+      * for differential clock on the refclk_p and
+      * refclk_m off chip pins: CMN_DIAG_ACYA[8]=1'b1
+      * cdns_phy_reg_write(cdns_phy, CMN_DIAG_ACYA, 0x0100);
+      */

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     val |= FIELD_PREP(CMA_REF_CLK_SEL_MASK, 3);
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_CMN_CTRL1, val);
+
+     /* Deassert PHY reset */
+     cdns_phy_reg_write(cdns_phy, PHY_ISO_CMN_CTRL, 0x0001);
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_CMN_CTRL, 0x0003);
+
+     /* Power state machine registers */
+     for (k = 0; k < num_lanes; k++)
+             cdns_phy_reg_write(cdns_phy, XCVR_PSM_RCTRL | (k <<

9),

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 7 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +7 -0
+ 0xfefc);
+
+     /* Assert cmn_macro_pwr_en */
+     cdns_phy_reg_write(cdns_phy, PHY_PMA_ISO_CMN_CTRL, 0x0013);
+
+     /* wait for cmn_macro_pwr_en_ack */
+     ret = wait_for_ack(cdns_phy, PHY_PMA_ISO_CMN_CTRL,

CMN_MACRO_PWR_EN_ACK,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 12 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +12 -0
+                        "MA output macro power up failed");
+     if (ret < 0)
+             return ret;
+
+     /* wait for cmn_ready */
+     ret = wait_for_ack(cdns_phy, PHY_PMA_CMN_CTRL1, CMN_READY,
+                        "PMA output ready failed");
+     if (ret < 0)
+             return ret;
+
+     for (k = 0; k < num_lanes; k++) {
+             cdns_phy_reg_write(cdns_phy, TX_PSC_A0 | (k << 9),

0x6791);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 720

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+             cdns_phy_reg_write(cdns_phy, TX_PSC_A1 | (k << 9),

0x6790);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 721

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+             cdns_phy_reg_write(cdns_phy, TX_PSC_A2 | (k << 9),

0x0090);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 725

Quoted 4 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +4 -0
+             cdns_phy_reg_write(cdns_phy, TX_PSC_A3 | (k << 9),
+ 0x0090);
+
+             val = cdns_phy_reg_read(cdns_phy, RX_PSC_CAL | (k <<

9));

Quoted 19 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +19 -0
+             val &= 0xffbb;
+             cdns_phy_reg_write(cdns_phy, RX_PSC_CAL | (k << 9),
+ val);
+
+             val = cdns_phy_reg_read(cdns_phy, RX_PSC_A0 | (k << 9));
+             val &= 0xffbb;
+             cdns_phy_reg_write(cdns_phy, RX_PSC_A0 | (k << 9), val);
+     }
+
+     return 0;
+}
+
+static int hdptx_hdmi_phy_cfg(struct cdns_hdptx_hdmi_phy *cdns_phy,
+u32 rate) {
+     const struct hdptx_hdmi_ctrl *p_ctrl_table;
+     const struct hdptx_hdmi_pll_tuning *p_pll_table;
+     const u32 refclk_freq_khz = cdns_phy->ref_clk_rate / 1000;
+     const bool pclk_in = false;
+     u32 pixel_freq = rate;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     u32 vco_freq, char_freq;
+     u32 div_total, feedback_factor;
+     u32 i;
+
+     feedback_factor = hdptx_hdmi_feedback_factor(cdns_phy);
+
+     char_freq = pixel_freq * feedback_factor / 1000;
+
+     dev_dbg(cdns_phy->dev,
+             "Pixel clock: (%d KHz), character clock: %d, bpc is

(%0d-bit)\n",

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 4 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +4 -0
+             pixel_freq, char_freq, cdns_phy->bpc);
+
+     /* Get right row from the ctrl_table table.
+      * Check if 'pixel_freq_khz' value matches the PIXEL_CLK_FREQ

column.

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 759

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+      * Consider only the rows with FEEDBACK_FACTOR column

matching feedback_factor.

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 3 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +3 -0
+      */
+     for (i = 0; i < ARRAY_SIZE(pixel_clk_output_ctrl_table); i++) {
+             if (feedback_factor ==

pixel_clk_output_ctrl_table[i].feedback_factor &&

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 763

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+                 pixel_freq ==

pixel_clk_output_ctrl_table[i].pixel_clk_freq_min) {

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 7 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +7 -0
+                     p_ctrl_table = &pixel_clk_output_ctrl_table[i];
+                     break;
+             }
+     }
+     if (i == ARRAY_SIZE(pixel_clk_output_ctrl_table)) {
+             dev_warn(cdns_phy->dev,
+                      "Pixel clk (%d KHz) not supported, bpc is

(%0d-bit)\n",

Quoted 4 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +4 -0
+                      pixel_freq, cdns_phy->bpc);
+             return -EINVAL;
+     }
+

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     div_total = p_ctrl_table->pll_fb_div_total;
+     vco_freq = refclk_freq_khz * div_total /
+ p_ctrl_table->cmnda_pll0_ip_div;
+
+     /* Get right row from the pixel_clk_output_pll_table table.
+      * Check if vco_freq_khz and feedback_div_total
+      * column matching with pixel_clk_output_pll_table.
+      */
+     for (i = 0; i < ARRAY_SIZE(pixel_clk_output_pll_table); i++) {
+             if (vco_freq == pixel_clk_output_pll_table[i].vco_freq_min

&&

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 785

Quoted 1 line
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +1 -0
+                 div_total ==

pixel_clk_output_pll_table[i].feedback_div_total) {

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 6 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +6 -0
+                     p_pll_table = &pixel_clk_output_pll_table[i];
+                     break;
+             }
+     }
+     if (i == ARRAY_SIZE(pixel_clk_output_pll_table)) {
+             dev_warn(cdns_phy->dev, "VCO (%d KHz) not supported\n",

vco_freq);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 13 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +13 -0
+             return -EINVAL;
+     }
+     dev_dbg(cdns_phy->dev, "VCO frequency is (%d KHz)\n", vco_freq);
+
+     return hdptx_hdmi_phy_config(cdns_phy, p_ctrl_table,
+p_pll_table, pclk_in); }
+
+static int hdptx_hdmi_phy_power_up(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     int ret = 0;
+
+     /* set Power State to A2 */
+     cdns_phy_reg_write(cdns_phy, PHY_HDP_MODE_CTRL,

POWER_STATE_A2);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 7 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +7 -0
+
+     cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_0, 1);
+     cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_1, 1);
+     cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_2, 1);
+     cdns_phy_reg_write(cdns_phy, TX_DIAG_ACYA_3, 1);
+
+     ret = wait_for_ack(cdns_phy, PHY_HDP_MODE_CTRL,

POWER_STATE_A2_ACK,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 11 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +11 -0
+                        "Wait A2 Ack failed");
+     if (ret < 0)
+             return ret;
+
+     /* Power up ARC */
+     hdptx_hdmi_arc_config(cdns_phy);
+
+     /* Configure PHY in A0 mode (PHY must be in the A0 power
+      * state in order to transmit data)
+      */
+     cdns_phy_reg_write(cdns_phy, PHY_HDP_MODE_CTRL,

POWER_STATE_A0);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 824

Quoted 2 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +2 -0
+
+     return wait_for_ack(cdns_phy, PHY_HDP_MODE_CTRL,

POWER_STATE_A0_ACK,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 8 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +8 -0
+                         "Wait A0 Ack failed"); }
+
+static int hdptx_hdmi_phy_power_down(struct cdns_hdptx_hdmi_phy
+*cdns_phy) {
+     u32 val;
+
+     val = cdns_phy_reg_read(cdns_phy, PHY_HDP_MODE_CTRL);
+     val &= ~(POWER_STATE_A0 | POWER_STATE_A1 |

POWER_STATE_A2 | POWER_STATE_A3);

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 837

Quoted 5 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +5 -0
+     /* PHY_DP_MODE_CTL set to A3 power state */
+     cdns_phy_reg_write(cdns_phy, PHY_HDP_MODE_CTRL, val |
+ POWER_STATE_A3);
+
+     return wait_for_ack(cdns_phy, PHY_HDP_MODE_CTRL,

POWER_STATE_A3_ACK,

Quoted 8 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +8 -0
+                         "Wait A3 Ack failed"); }
+
+static int cdns_hdptx_hdmi_phy_on(struct phy *phy) {
+     struct cdns_hdptx_hdmi_phy *cdns_phy = phy_get_drvdata(phy);
+
+     return hdptx_hdmi_phy_power_up(cdns_phy);
+}
+

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 814

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+static int cdns_hdptx_hdmi_phy_off(struct phy *phy) {
+     struct cdns_hdptx_hdmi_phy *cdns_phy = phy_get_drvdata(phy);
+
+     hdptx_hdmi_phy_power_down(cdns_phy);
+     return 0;
+}
+
+static int
+cdns_hdptx_hdmi_phy_valid(struct phy *phy, enum phy_mode mode,
+                       int submode, union phy_configure_opts *opts)

{

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 5 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +5 -0
+     u32 rate = opts->hdmi.pixel_clk_rate;
+     int i;
+
+     for (i = 0; i < ARRAY_SIZE(pixel_clk_output_ctrl_table); i++)
+             if (rate ==

pixel_clk_output_ctrl_table[i].pixel_clk_freq_min)

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 832

Quoted 11 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +11 -0
+                     return 0;
+
+     return -EINVAL;
+}
+
+static int cdns_hdptx_hdmi_phy_init(struct phy *phy) {
+     return 0;
+}
+
+static int cdns_hdptx_hdmi_configure(struct phy *phy,
+                                  union phy_configure_opts *opts)

{

Quoted 73 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +73 -0
+     struct cdns_hdptx_hdmi_phy *cdns_phy = phy_get_drvdata(phy);
+     int ret;
+
+     cdns_phy->pixel_clk_rate = opts->hdmi.pixel_clk_rate;
+     cdns_phy->color_space = opts->hdmi.color_space;
+     cdns_phy->bpc = opts->hdmi.bpc;
+
+     /* Check HDMI FW alive before HDMI PHY init */
+     ret = hdptx_phy_check_alive(cdns_phy);
+     if (!ret) {
+             dev_err(cdns_phy->dev, "NO HDMI FW running\n");
+             return -ENXIO;
+     }
+
+     /* Configure PHY */
+     if (hdptx_hdmi_phy_cfg(cdns_phy, cdns_phy->pixel_clk_rate) < 0) {
+             dev_err(cdns_phy->dev, "failed to set phy pclock\n");
+             return -EINVAL;
+     }
+
+     ret = hdptx_hdmi_phy_power_up(cdns_phy);
+     if (ret < 0)
+             return ret;
+
+     hdptx_hdmi_phy_set_vswing(cdns_phy);
+
+     return 0;
+}
+
+static const struct phy_ops cdns_hdptx_hdmi_phy_ops = {
+     .init = cdns_hdptx_hdmi_phy_init,
+     .configure = cdns_hdptx_hdmi_configure,
+     .power_on = cdns_hdptx_hdmi_phy_on,
+     .power_off = cdns_hdptx_hdmi_phy_off,
+     .validate = cdns_hdptx_hdmi_phy_valid,
+     .owner = THIS_MODULE,
+};
+
+static int cdns_hdptx_hdmi_phy_probe(struct platform_device *pdev) {
+     struct cdns_hdptx_hdmi_phy *cdns_phy;
+     struct device *dev = &pdev->dev;
+     struct device_node *node = dev->of_node;
+     struct phy_provider *phy_provider;
+     struct resource *res;
+     struct phy *phy;
+     int ret;
+
+     cdns_phy = devm_kzalloc(dev, sizeof(*cdns_phy), GFP_KERNEL);
+     if (!cdns_phy)
+             return -ENOMEM;
+
+     dev_set_drvdata(dev, cdns_phy);
+     cdns_phy->dev = dev;
+     mutex_init(&cdns_phy->mbox_mutex);
+
+     res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+     if (!res)
+             return -ENODEV;
+     cdns_phy->regs = devm_ioremap(dev, res->start, resource_size(res));
+     if (IS_ERR(cdns_phy->regs))
+             return PTR_ERR(cdns_phy->regs);
+
+     phy = devm_phy_create(dev, node, &cdns_hdptx_hdmi_phy_ops);
+     if (IS_ERR(phy))
+             return PTR_ERR(phy);
+
+     phy->attrs.mode = PHY_MODE_HDMI;
+
+     cdns_phy->phy = phy;
+     phy_set_drvdata(phy, cdns_phy);
+
+     /* init base struct for access mhdp mailbox */
+     cdns_phy->base.dev = cdns_phy->dev;

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     cdns_phy->base.regs = cdns_phy->regs;
+     cdns_phy->base.mbox_mutex = &cdns_phy->mbox_mutex;
+
+     ret = hdptx_hdmi_clk_enable(cdns_phy);
+     if (ret) {
+             dev_err(dev, "Init clk fail\n");
+             return -EINVAL;
+     }
+
+     phy_provider = devm_of_phy_provider_register(dev,

of_phy_simple_xlate);

Quoted 34 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +33 -0
+     if (IS_ERR(phy_provider)) {
+             ret = PTR_ERR(phy_provider);
+             goto clk_disable;
+     }
+
+     dev_dbg(dev, "probe success!\n");
+
+     return 0;
+
+clk_disable:
+     hdptx_hdmi_clk_disable(cdns_phy);
+
+     return -EINVAL;
+}
+
+static int cdns_hdptx_hdmi_phy_remove(struct platform_device *pdev) {
+     struct cdns_hdptx_hdmi_phy *cdns_phy =
+platform_get_drvdata(pdev);
+
+     hdptx_hdmi_clk_disable(cdns_phy);
+
+     return 0;
+}
+
+static const struct of_device_id cdns_hdptx_hdmi_phy_of_match[] = {
+     {.compatible = "fsl,imx8mq-hdmi-phy" },
+     { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, cdns_hdptx_hdmi_phy_of_match);
+
+static struct platform_driver cdns_hdptx_hdmi_phy_driver = {
+     .probe = cdns_hdptx_hdmi_phy_probe,
+     .remove = cdns_hdptx_hdmi_phy_remove,

drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c line 960

Quoted 10 lines
drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c new file +10 -0
+     .driver = {
+             .name   = "cdns-hdptx-hdmi-phy",
+             .of_match_table = cdns_hdptx_hdmi_phy_of_match,
+     }
+};
+module_platform_driver(cdns_hdptx_hdmi_phy_driver);
+
+MODULE_AUTHOR("Sandor Yu <[email protected]>");
+MODULE_DESCRIPTION("Cadence HDP-TX HDMI PHY driver");
+MODULE_LICENSE("GPL");

--
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-/
group.com%2F&data=05%7C02%7CSandor.yu%40nxp.com%7C68abe9642bd9 4d237a4e08dc3200861a%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C 0%7C638440225041723926%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLj AwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C0%7C%7C% 7C&sdata=QVMdobK1UfMi1Pnjg5YLEvfLbPDjLm7xzIEV5Y97xEw%3D&reserve d=0

#13 lore raw

Re: [PATCH v14 7/7] phy: freescale: Add HDMI PHY driver for i.MX8MQ

Hi Sandor,

Am Mittwoch, 21. Februar 2024, 08:47:52 CET schrieb Sandor Yu:

Quoted 25 lines

Hi Alexander,

Thanks for your comments,

Quoted 13 lines

Hi,

thanks for the update.

Am Dienstag, 20. Februar 2024, 04:23:55 CET schrieb Sandor Yu:

Quoted 3 lines

Add Cadence HDP-TX HDMI PHY driver for i.MX8MQ.

Cadence HDP-TX PHY could be put in either DP mode or HDMI mode base

on

Quoted 3 lines

the configuration chosen.
HDMI PHY mode is configurated in the driver.

Quoted 8 lines
Quoted 2 lines

This still works as before. I noticed there is a lot of code duplication with patch 6. IMHO these PHY drivers should be merged into a single one where the mode is configured using phy_set_mode() from cdns-mhdp8501-core.c. This nicely matches my concerns regarding patch 5.

Quoted 4 lines

Yes, there are some registers offset are same and the clock management function could be reused for DP and HDMI PHY driver. But because of HDMI and DP PHY totally different work mode, the functions in struct phy_ops,
Such as ->init, ->power_on/off and ->configure could not combine into a single one,
So separate DP and HDMI PHY driver should be a better resolution.

Despite some output type (DP/HDMI) specific settings, the similarities are quite huge actually. Even though apparently DP has it's own ->init setup, this can be delayed until ->set_mode or even ->configure.
The distinction between each mode can be done by checking phy->attrs.mode. For a prove of concept I've hacked both drivers into a single one. I can't test DP, but HDMI still works. Feel free to contact me in private.

Best regards,
Alexander

Quoted 34 lines

B.R
Sandor

Quoted 28 lines

Best regards,
Alexander

Quoted 25 lines

---
v13->v14:
*No change.

v12->v13:
- Fix build warning

v11->v12:
- Adjust clk disable order.
- Return error code to replace -1 for function wait_for_ack().
- Use bool for variable pclk_in.
- Add year 2024 to copyright.

 drivers/phy/freescale/Kconfig               |  10 +
 drivers/phy/freescale/Makefile              |   1 +
 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c | 960
++++++++++++++++++++
 3 files changed, 971 insertions(+)
 create mode 100644 drivers/phy/freescale/phy-fsl-imx8mq-hdmi.c
(quoted diff) +0 -0

b/drivers/phy/freescale/Kconfig index c39709fd700ac..14f47b7cc77ab 100644

drivers/phy/freescale/Kconfig +0 -0

[snip]

Signature · 4 lines
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/