This patch allows LVDS PHYs to be configured through
the generic functions and through a custom structure
added to the generic union.

The parameters added here are based on common LVDS PHY
implementation practices.  The set of parameters
should cover all potential users.

Cc: Kishon Vijay Abraham I <kis...@ti.com>
Cc: Vinod Koul <vk...@kernel.org>
Cc: NXP Linux Team <linux-...@nxp.com>
Signed-off-by: Liu Ying <victor....@nxp.com>
---
v6->v7:
* Update the year of copyright.
* Better variable explaination for bits_per_lane_and_dclk_cycle.

v5->v6:
* Rebase upon v5.17-rc1.

v4->v5:
* Align kernel-doc style to include/linux/phy/phy.h. (Vinod)
* Trivial tweaks.
* Drop Robert's R-b tag.

v3->v4:
* Add Robert's R-b tag.

v2->v3:
* No change.

v1->v2:
* No change.

 include/linux/phy/phy-lvds.h | 32 ++++++++++++++++++++++++++++++++
 include/linux/phy/phy.h      |  4 ++++
 2 files changed, 36 insertions(+)
 create mode 100644 include/linux/phy/phy-lvds.h

diff --git a/include/linux/phy/phy-lvds.h b/include/linux/phy/phy-lvds.h
new file mode 100644
index 000000000000..9f06dfedc435
--- /dev/null
+++ b/include/linux/phy/phy-lvds.h
@@ -0,0 +1,32 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+/*
+ * Copyright 2020,2022 NXP
+ */
+
+#ifndef __PHY_LVDS_H_
+#define __PHY_LVDS_H_
+
+/**
+ * struct phy_configure_opts_lvds - LVDS configuration set
+ * @bits_per_lane_and_dclk_cycle:      Number of bits per lane per differential
+                                       clock cycle.
+ * @differential_clk_rate:             Clock rate, in Hertz, of the LVDS
+ *                                     differential clock.
+ * @lanes:                             Number of active, consecutive,
+ *                                     data lanes, starting from lane 0,
+ *                                     used for the transmissions.
+ * @is_slave:                          Boolean, true if the phy is a slave
+ *                                     which works together with a master
+ *                                     phy to support dual link transmission,
+ *                                     otherwise a regular phy or a master phy.
+ *
+ * This structure is used to represent the configuration state of a LVDS phy.
+ */
+struct phy_configure_opts_lvds {
+       unsigned int    bits_per_lane_and_dclk_cycle;
+       unsigned long   differential_clk_rate;
+       unsigned int    lanes;
+       bool            is_slave;
+};
+
+#endif /* __PHY_LVDS_H_ */
diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h
index f3286f4cd306..b1413757fcc3 100644
--- a/include/linux/phy/phy.h
+++ b/include/linux/phy/phy.h
@@ -17,6 +17,7 @@
 #include <linux/regulator/consumer.h>
 
 #include <linux/phy/phy-dp.h>
+#include <linux/phy/phy-lvds.h>
 #include <linux/phy/phy-mipi-dphy.h>
 
 struct phy;
@@ -57,10 +58,13 @@ enum phy_media {
  *             the MIPI_DPHY phy mode.
  * @dp:                Configuration set applicable for phys supporting
  *             the DisplayPort protocol.
+ * @lvds:      Configuration set applicable for phys supporting
+ *             the LVDS 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;
 };
 
 /**
-- 
2.25.1

Reply via email to