WHen adding HDMI fields I didn't notice the private: declaration for HPD
fields. Move private fields to the end of the struct drm_bride to have
clear distinction between private and public fields.

Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
---
 include/drm/drm_bridge.h | 31 ++++++++++++++++---------------
 1 file changed, 16 insertions(+), 15 deletions(-)

diff --git a/include/drm/drm_bridge.h b/include/drm/drm_bridge.h
index 
1f1670e3c6aac39b8b891b0d5e7e91254eb0d3a1..348778f233b06265a6ae577762c6558e69cdb396
 100644
--- a/include/drm/drm_bridge.h
+++ b/include/drm/drm_bridge.h
@@ -872,21 +872,6 @@ struct drm_bridge {
         * @ddc: Associated I2C adapter for DDC access, if any.
         */
        struct i2c_adapter *ddc;
-       /** private: */
-       /**
-        * @hpd_mutex: Protects the @hpd_cb and @hpd_data fields.
-        */
-       struct mutex hpd_mutex;
-       /**
-        * @hpd_cb: Hot plug detection callback, registered with
-        * drm_bridge_hpd_enable().
-        */
-       void (*hpd_cb)(void *data, enum drm_connector_status status);
-       /**
-        * @hpd_data: Private data passed to the Hot plug detection callback
-        * @hpd_cb.
-        */
-       void *hpd_data;
 
        /**
         * @vendor: Vendor of the product to be used for the SPD InfoFrame
@@ -938,6 +923,22 @@ struct drm_bridge {
         * @hdmi_cec_notifier: use this bridge to register a CEC notifier
         */
        bool hdmi_cec_notifier;
+
+       /** private: */
+       /**
+        * @hpd_mutex: Protects the @hpd_cb and @hpd_data fields.
+        */
+       struct mutex hpd_mutex;
+       /**
+        * @hpd_cb: Hot plug detection callback, registered with
+        * drm_bridge_hpd_enable().
+        */
+       void (*hpd_cb)(void *data, enum drm_connector_status status);
+       /**
+        * @hpd_data: Private data passed to the Hot plug detection callback
+        * @hpd_cb.
+        */
+       void *hpd_data;
 };
 
 static inline struct drm_bridge *

-- 
2.39.5

Reply via email to