drm: meson: global clean-up
authorJulien Masson <jmasson@baylibre.com>
Mon, 24 Jun 2019 14:48:57 +0000 (16:48 +0200)
committerNeil Armstrong <narmstrong@baylibre.com>
Fri, 9 Aug 2019 10:06:14 +0000 (12:06 +0200)
This patch aims to:
- Add general and TODO comments
- Respect coding style for multi-line comments
- Align macro definitions
- Remove useless macro

Signed-off-by: Julien Masson <jmasson@baylibre.com>
Reviewed-by: Neil Armstrong <narmstrong@baylibre.com>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
Link: https://patchwork.freedesktop.org/patch/msgid/86pnn382e8.fsf@baylibre.com
drivers/gpu/drm/meson/meson_dw_hdmi.c
drivers/gpu/drm/meson/meson_dw_hdmi.h
drivers/gpu/drm/meson/meson_registers.h
drivers/gpu/drm/meson/meson_vclk.c
drivers/gpu/drm/meson/meson_venc.c

index 9f0b08eaf00323b224dd0b70c0748e173c009ae8..f893ebd0b799255c06f5308b36be834966d0b0d2 100644 (file)
@@ -429,6 +429,8 @@ static int dw_hdmi_phy_init(struct dw_hdmi *hdmi, void *data,
        /* Enable internal pixclk, tmds_clk, spdif_clk, i2s_clk, cecclk */
        dw_hdmi_top_write_bits(dw_hdmi, HDMITX_TOP_CLK_CNTL,
                               0x3, 0x3);
+
+       /* Enable cec_clk and hdcp22_tmdsclk_en */
        dw_hdmi_top_write_bits(dw_hdmi, HDMITX_TOP_CLK_CNTL,
                               0x3 << 4, 0x3 << 4);
 
index 1b2ef043eb5ca2eef621ca6b9b80ae90fc7dcae8..08e1c14e4ea07c694f2c5fccbbf593661a8a3feb 100644 (file)
 #define HDMITX_TOP_INTR_RXSENSE_RISE   BIT(6)
 #define HDMITX_TOP_INTR_RXSENSE_FALL   BIT(7)
 
-/* Bit 14:12 RW tmds_sel: 3'b000=Output zero; 3'b001=Output normal TMDS data;
+/*
+ * Bit 14:12 RW tmds_sel: 3'b000=Output zero; 3'b001=Output normal TMDS data;
  *     3'b010=Output PRBS data; 3'b100=Output shift pattern. Default 0.
  * Bit 11: 9 RW shift_pttn_repeat: 0=New pattern every clk cycle; 1=New pattern
  *     every 2 clk cycles; ...; 7=New pattern every 8 clk cycles. Default 0.
 /* Bit  9: 0 RW tmds_clk_pttn[29:20]. Default 0. */
 #define HDMITX_TOP_TMDS_CLK_PTTN_23             (0x00B)
 
-/* Bit 1 RW shift_tmds_clk_pttn:1=Enable shifting clk pattern,
+/*
+ * Bit 1 RW shift_tmds_clk_pttn:1=Enable shifting clk pattern,
  * used when TMDS CLK rate = TMDS character rate /4. Default 0.
  * Bit 0 R  Reserved. Default 0.
  * [   1] shift_tmds_clk_pttn
  */
 #define HDMITX_TOP_TMDS_CLK_PTTN_CNTL           (0x00C)
 
-/* Bit 0 RW revocmem_wr_fail: Read back 1 to indicate Host write REVOC MEM
+/*
+ * Bit 0 RW revocmem_wr_fail: Read back 1 to indicate Host write REVOC MEM
  * failure, write 1 to clear the failure flag.  Default 0.
  */
 #define HDMITX_TOP_REVOCMEM_STAT                (0x00D)
 
-/* Bit    1 R  filtered RxSense status
+/*
+ * Bit    1 R  filtered RxSense status
  * Bit     0 R  filtered HPD status.
  */
 #define HDMITX_TOP_STAT0                        (0x00E)
index a96293880e821a23e183d43e1b9922c11e6c2fc5..05fce48ceee0df57eba349df48aa92031b3c6427 100644 (file)
 #define VPP_PREBLEND_CURRENT_XY 0x1d24
 #define VPP_POSTBLEND_CURRENT_XY 0x1d25
 #define VPP_MISC 0x1d26
-#define                VPP_PREBLEND_ENABLE     BIT(6)
-#define                VPP_POSTBLEND_ENABLE    BIT(7)
-#define                VPP_OSD2_ALPHA_PREMULT  BIT(8)
-#define                VPP_OSD1_ALPHA_PREMULT  BIT(9)
-#define                VPP_VD1_POSTBLEND       BIT(10)
-#define                VPP_VD2_POSTBLEND       BIT(11)
-#define                VPP_OSD1_POSTBLEND      BIT(12)
-#define                VPP_OSD2_POSTBLEND      BIT(13)
-#define                VPP_VD1_PREBLEND        BIT(14)
-#define                VPP_VD2_PREBLEND        BIT(15)
-#define                VPP_OSD1_PREBLEND       BIT(16)
-#define                VPP_OSD2_PREBLEND       BIT(17)
-#define                VPP_COLOR_MNG_ENABLE    BIT(28)
+#define                VPP_PREBLEND_ENABLE             BIT(6)
+#define                VPP_POSTBLEND_ENABLE            BIT(7)
+#define                VPP_OSD2_ALPHA_PREMULT          BIT(8)
+#define                VPP_OSD1_ALPHA_PREMULT          BIT(9)
+#define                VPP_VD1_POSTBLEND               BIT(10)
+#define                VPP_VD2_POSTBLEND               BIT(11)
+#define                VPP_OSD1_POSTBLEND              BIT(12)
+#define                VPP_OSD2_POSTBLEND              BIT(13)
+#define                VPP_VD1_PREBLEND                BIT(14)
+#define                VPP_VD2_PREBLEND                BIT(15)
+#define                VPP_OSD1_PREBLEND               BIT(16)
+#define                VPP_OSD2_PREBLEND               BIT(17)
+#define                VPP_COLOR_MNG_ENABLE            BIT(28)
 #define VPP_OFIFO_SIZE 0x1d27
 #define                VPP_OFIFO_SIZE_MASK             GENMASK(13, 0)
 #define                VPP_OFIFO_SIZE_DEFAULT          (0xfff << 20 | 0x1000)
 #define OSD34_SCI_WH_M1 0x3d29
 #define OSD34_SCO_H_START_END 0x3d2a
 #define OSD34_SCO_V_START_END 0x3d2b
+
 /* viu2 */
 #define VIU2_ADDR_START 0x1e00
 #define VIU2_ADDR_END 0x1eff
 #define OSD1_AFBCD_STATUS 0x31a8
 #define OSD1_AFBCD_PIXEL_HSCOPE 0x31a9
 #define OSD1_AFBCD_PIXEL_VSCOPE 0x31aa
-#define VIU_MISC_CTRL1 0x1a07
 
 /* add for gxm and 962e dv core2 */
 #define DOLBY_CORE2A_SWAP_CTRL1        0x3434
 #define VPU_MAFBC_COMMAND 0x3a05
 #define VPU_MAFBC_STATUS 0x3a06
 #define VPU_MAFBC_SURFACE_CFG 0x3a07
-
-/* osd afbc on g12a */
 #define VPU_MAFBC_HEADER_BUF_ADDR_LOW_S0 0x3a10
 #define VPU_MAFBC_HEADER_BUF_ADDR_HIGH_S0 0x3a11
 #define VPU_MAFBC_FORMAT_SPECIFIER_S0 0x3a12
 #define VPP_POST_BLEND_DUMMY_ALPHA 0x3969
 #define VPP_RDARB_MODE 0x3978
 #define VPP_RDARB_REQEN_SLV 0x3979
-#define VPU_RDARB_MODE_L2C1 0x279d
 
 #endif /* __MESON_REGISTERS_H */
index 8abff51f937d8a40195396baf4915ee56b53ae3b..c11b55237917e095146e49c04e6e5a5e00855bfe 100644 (file)
@@ -496,6 +496,7 @@ void meson_hdmi_pll_set_params(struct meson_drm *priv, unsigned int m,
                regmap_write(priv->hhi, HHI_HDMI_PLL_CNTL, 0x0b3a0400 | m);
 
                /* Enable and reset */
+               /* TODO: add specific macro for g12a here */
                regmap_update_bits(priv->hhi, HHI_HDMI_PLL_CNTL,
                                   0x3 << 28, 0x3 << 28);
 
@@ -970,7 +971,8 @@ void meson_vclk_setup(struct meson_drm *priv, unsigned int target,
                meson_venci_cvbs_clock_config(priv);
                return;
        } else if (target == MESON_VCLK_TARGET_DMT) {
-               /* The DMT clock path is fixed after the PLL:
+               /*
+                * The DMT clock path is fixed after the PLL:
                 * - automatic PLL freq + OD management
                 * - vid_pll_div = VID_PLL_DIV_5
                 * - vclk_div = 2
index 9defedaac60c61c7b59ac1d84b33d435ec0291e6..cb569d3104a88a3f03e9ce7cd128e612b89decb7 100644 (file)
@@ -61,9 +61,9 @@
 /* HHI Registers */
 #define HHI_GCLK_MPEG2         0x148 /* 0x52 offset in data sheet */
 #define HHI_VDAC_CNTL0         0x2F4 /* 0xbd offset in data sheet */
-#define HHI_VDAC_CNTL0_G12A    0x2EC /* 0xbd offset in data sheet */
+#define HHI_VDAC_CNTL0_G12A    0x2EC /* 0xbb offset in data sheet */
 #define HHI_VDAC_CNTL1         0x2F8 /* 0xbe offset in data sheet */
-#define HHI_VDAC_CNTL1_G12A    0x2F0 /* 0xbe offset in data sheet */
+#define HHI_VDAC_CNTL1_G12A    0x2F0 /* 0xbc offset in data sheet */
 #define HHI_HDMI_PHY_CNTL0     0x3a0 /* 0xe8 offset in data sheet */
 
 struct meson_cvbs_enci_mode meson_cvbs_enci_pal = {
@@ -1085,7 +1085,8 @@ void meson_venc_hdmi_mode_set(struct meson_drm *priv, int vic,
                writel_relaxed(vmode->enci.video_mode,
                                priv->io_base + _REG(ENCI_VIDEO_MODE));
 
-               /* Advanced Video Mode :
+               /*
+                * Advanced Video Mode :
                 * Demux shifting 0x2
                 * Blank line end at line17/22
                 * High bandwidth Luma Filter
@@ -1599,7 +1600,8 @@ void meson_venci_cvbs_mode_set(struct meson_drm *priv,
        writel_relaxed(mode->video_mode,
                        priv->io_base + _REG(ENCI_VIDEO_MODE));
 
-       /* Advanced Video Mode :
+       /*
+        * Advanced Video Mode :
         * Demux shifting 0x2
         * Blank line end at line17/22
         * High bandwidth Luma Filter