Drop obsolete functions to access audio packet headers. The dp_audio.c
now writes them using msm_dp_write_link() directly.

Reviewed-by: Stephen Boyd <swb...@chromium.org>
Tested-by: Stephen Boyd <swb...@chromium.org> # sc7180-trogdor
Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
---
 drivers/gpu/drm/msm/dp/dp_audio.c   |  2 -
 drivers/gpu/drm/msm/dp/dp_catalog.c | 76 -------------------------------------
 drivers/gpu/drm/msm/dp/dp_catalog.h | 24 ------------
 3 files changed, 102 deletions(-)

diff --git a/drivers/gpu/drm/msm/dp/dp_audio.c 
b/drivers/gpu/drm/msm/dp/dp_audio.c
index 
46fbf8601eea8e43a152049dfd1dc1d77943d922..9e402d71b9761f640f1a8733e10e09477ff675cb
 100644
--- a/drivers/gpu/drm/msm/dp/dp_audio.c
+++ b/drivers/gpu/drm/msm/dp/dp_audio.c
@@ -375,8 +375,6 @@ struct msm_dp_audio *msm_dp_audio_get(struct 
platform_device *pdev,
 
        msm_dp_audio = &audio->msm_dp_audio;
 
-       msm_dp_catalog_audio_init(catalog);
-
        return msm_dp_audio;
 error:
        return ERR_PTR(rc);
diff --git a/drivers/gpu/drm/msm/dp/dp_catalog.c 
b/drivers/gpu/drm/msm/dp/dp_catalog.c
index 
8fddfe5d85d6398c6582e1f74647f4cd83f5a4d9..7b7eadb2f83b169d8df27ee93589abe05b38f3ae
 100644
--- a/drivers/gpu/drm/msm/dp/dp_catalog.c
+++ b/drivers/gpu/drm/msm/dp/dp_catalog.c
@@ -79,7 +79,6 @@ struct msm_dp_catalog_private {
        struct device *dev;
        struct drm_device *drm_dev;
        struct dss_io_data io;
-       u32 (*audio_map)[DP_AUDIO_SDP_HEADER_MAX];
        struct msm_dp_catalog msm_dp_catalog;
 };
 
@@ -1193,40 +1192,6 @@ void msm_dp_catalog_write_audio_isrc(struct 
msm_dp_catalog *msm_dp_catalog,
        msm_dp_write_link(catalog, MMSS_DP_AUDIO_ISRC_1, header[1]);
 }
 
-u32 msm_dp_catalog_audio_get_header(struct msm_dp_catalog *msm_dp_catalog,
-                               enum msm_dp_catalog_audio_sdp_type sdp,
-                               enum msm_dp_catalog_audio_header_type header)
-{
-       struct msm_dp_catalog_private *catalog;
-       u32 (*sdp_map)[DP_AUDIO_SDP_HEADER_MAX];
-
-       catalog = container_of(msm_dp_catalog,
-               struct msm_dp_catalog_private, msm_dp_catalog);
-
-       sdp_map = catalog->audio_map;
-
-       return msm_dp_read_link(catalog, sdp_map[sdp][header]);
-}
-
-void msm_dp_catalog_audio_set_header(struct msm_dp_catalog *msm_dp_catalog,
-                                enum msm_dp_catalog_audio_sdp_type sdp,
-                                enum msm_dp_catalog_audio_header_type header,
-                                u32 data)
-{
-       struct msm_dp_catalog_private *catalog;
-       u32 (*sdp_map)[DP_AUDIO_SDP_HEADER_MAX];
-
-       if (!msm_dp_catalog)
-               return;
-
-       catalog = container_of(msm_dp_catalog,
-               struct msm_dp_catalog_private, msm_dp_catalog);
-
-       sdp_map = catalog->audio_map;
-
-       msm_dp_write_link(catalog, sdp_map[sdp][header], data);
-}
-
 void msm_dp_catalog_audio_config_acr(struct msm_dp_catalog *msm_dp_catalog, 
u32 select)
 {
        struct msm_dp_catalog_private *catalog;
@@ -1310,47 +1275,6 @@ void msm_dp_catalog_audio_config_sdp(struct 
msm_dp_catalog *msm_dp_catalog)
        msm_dp_write_link(catalog, MMSS_DP_SDP_CFG2, sdp_cfg2);
 }
 
-void msm_dp_catalog_audio_init(struct msm_dp_catalog *msm_dp_catalog)
-{
-       struct msm_dp_catalog_private *catalog;
-
-       static u32 sdp_map[][DP_AUDIO_SDP_HEADER_MAX] = {
-               {
-                       MMSS_DP_AUDIO_STREAM_0,
-                       MMSS_DP_AUDIO_STREAM_1,
-                       MMSS_DP_AUDIO_STREAM_1,
-               },
-               {
-                       MMSS_DP_AUDIO_TIMESTAMP_0,
-                       MMSS_DP_AUDIO_TIMESTAMP_1,
-                       MMSS_DP_AUDIO_TIMESTAMP_1,
-               },
-               {
-                       MMSS_DP_AUDIO_INFOFRAME_0,
-                       MMSS_DP_AUDIO_INFOFRAME_1,
-                       MMSS_DP_AUDIO_INFOFRAME_1,
-               },
-               {
-                       MMSS_DP_AUDIO_COPYMANAGEMENT_0,
-                       MMSS_DP_AUDIO_COPYMANAGEMENT_1,
-                       MMSS_DP_AUDIO_COPYMANAGEMENT_1,
-               },
-               {
-                       MMSS_DP_AUDIO_ISRC_0,
-                       MMSS_DP_AUDIO_ISRC_1,
-                       MMSS_DP_AUDIO_ISRC_1,
-               },
-       };
-
-       if (!msm_dp_catalog)
-               return;
-
-       catalog = container_of(msm_dp_catalog,
-               struct msm_dp_catalog_private, msm_dp_catalog);
-
-       catalog->audio_map = sdp_map;
-}
-
 void msm_dp_catalog_audio_sfe_level(struct msm_dp_catalog *msm_dp_catalog, u32 
safe_to_exit_level)
 {
        struct msm_dp_catalog_private *catalog;
diff --git a/drivers/gpu/drm/msm/dp/dp_catalog.h 
b/drivers/gpu/drm/msm/dp/dp_catalog.h
index 
4a5591d9f82a21d7a3bb64ad6b486e39bd406cd5..6678b0ac9a67881244884d59487fa288d33d1be7
 100644
--- a/drivers/gpu/drm/msm/dp/dp_catalog.h
+++ b/drivers/gpu/drm/msm/dp/dp_catalog.h
@@ -31,22 +31,6 @@
 #define DP_HW_VERSION_1_0      0x10000000
 #define DP_HW_VERSION_1_2      0x10020000
 
-enum msm_dp_catalog_audio_sdp_type {
-       DP_AUDIO_SDP_STREAM,
-       DP_AUDIO_SDP_TIMESTAMP,
-       DP_AUDIO_SDP_INFOFRAME,
-       DP_AUDIO_SDP_COPYMANAGEMENT,
-       DP_AUDIO_SDP_ISRC,
-       DP_AUDIO_SDP_MAX,
-};
-
-enum msm_dp_catalog_audio_header_type {
-       DP_AUDIO_SDP_HEADER_1,
-       DP_AUDIO_SDP_HEADER_2,
-       DP_AUDIO_SDP_HEADER_3,
-       DP_AUDIO_SDP_HEADER_MAX,
-};
-
 struct msm_dp_catalog {
        bool wide_bus_en;
 };
@@ -121,17 +105,9 @@ void msm_dp_catalog_write_audio_copy_mgmt(struct 
msm_dp_catalog *msm_dp_catalog,
                                          struct dp_sdp_header *sdp_hdr);
 void msm_dp_catalog_write_audio_isrc(struct msm_dp_catalog *msm_dp_catalog,
                                     struct dp_sdp_header *sdp_hdr);
-u32 msm_dp_catalog_audio_get_header(struct msm_dp_catalog *msm_dp_catalog,
-                               enum msm_dp_catalog_audio_sdp_type sdp,
-                               enum msm_dp_catalog_audio_header_type header);
-void msm_dp_catalog_audio_set_header(struct msm_dp_catalog *msm_dp_catalog,
-                                enum msm_dp_catalog_audio_sdp_type sdp,
-                                enum msm_dp_catalog_audio_header_type header,
-                                u32 data);
 void msm_dp_catalog_audio_config_acr(struct msm_dp_catalog *catalog, u32 
select);
 void msm_dp_catalog_audio_enable(struct msm_dp_catalog *catalog, bool enable);
 void msm_dp_catalog_audio_config_sdp(struct msm_dp_catalog *catalog);
-void msm_dp_catalog_audio_init(struct msm_dp_catalog *catalog);
 void msm_dp_catalog_audio_sfe_level(struct msm_dp_catalog *catalog, u32 
safe_to_exit_level);
 
 #endif /* _DP_CATALOG_H_ */

-- 
2.39.5

Reply via email to