Hi Vinod, I love your patch! Perhaps something to improve: [auto build test WARNING on v5.13] [cannot apply to linus/master v5.14-rc1 next-20210716] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Vinod-Koul/drm-msm-Add-Display-Stream-Compression-Support/20210715-145540 base: 62fb9874f5da54fdb243003b386128037319b219 config: arm-defconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (GCC) 10.3.0 reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/67410f51d0cd25256b7926c30f27071291244ef3 git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Vinod-Koul/drm-msm-Add-Display-Stream-Compression-Support/20210715-145540 git checkout 67410f51d0cd25256b7926c30f27071291244ef3 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross ARCH=arm If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@xxxxxxxxx> All warnings (new ones prefixed by >>): drivers/gpu/drm/msm/dsi/dsi_host.c: In function 'dsi_timing_setup': >> drivers/gpu/drm/msm/dsi/dsi_host.c:1022:41: warning: variable 'width' set but not used [-Wunused-but-set-variable] 1022 | u32 reg, intf_width, slice_per_intf, width; | ^~~~~ vim +/width +1022 drivers/gpu/drm/msm/dsi/dsi_host.c 964 965 static void dsi_timing_setup(struct msm_dsi_host *msm_host, bool is_dual_dsi) 966 { 967 struct drm_display_mode *mode = msm_host->mode; 968 u32 hs_start = 0, vs_start = 0; /* take sync start as 0 */ 969 u32 h_total = mode->htotal; 970 u32 v_total = mode->vtotal; 971 u32 hs_end = mode->hsync_end - mode->hsync_start; 972 u32 vs_end = mode->vsync_end - mode->vsync_start; 973 u32 ha_start = h_total - mode->hsync_start; 974 u32 ha_end = ha_start + mode->hdisplay; 975 u32 va_start = v_total - mode->vsync_start; 976 u32 va_end = va_start + mode->vdisplay; 977 u32 hdisplay = mode->hdisplay; 978 u32 wc; 979 u32 data; 980 981 DBG(""); 982 983 /* 984 * For dual DSI mode, the current DRM mode has 985 * the complete width of the panel. Since, the complete 986 * panel is driven by two DSI controllers, the horizontal 987 * timings have to be split between the two dsi controllers. 988 * Adjust the DSI host timing values accordingly. 989 */ 990 if (is_dual_dsi) { 991 h_total /= 2; 992 hs_end /= 2; 993 ha_start /= 2; 994 ha_end /= 2; 995 hdisplay /= 2; 996 } 997 998 if (msm_host->dsc) { 999 struct msm_display_dsc_config *dsc = msm_host->dsc; 1000 1001 /* update dsc params with timing params */ 1002 dsi_dsc_update_pic_dim(dsc, mode->hdisplay, mode->vdisplay); 1003 DBG("Mode Width- %d x Height %d\n", dsc->drm->pic_width, dsc->drm->pic_height); 1004 1005 /* we do the calculations for dsc parameters here so that 1006 * panel can use these parameters 1007 */ 1008 dsi_populate_dsc_params(dsc); 1009 1010 /* Divide the display by 3 but keep back/font porch and 1011 * pulse width same 1012 */ 1013 h_total -= hdisplay; 1014 hdisplay /= 3; 1015 h_total += hdisplay; 1016 ha_end = ha_start + hdisplay; 1017 } 1018 1019 if (msm_host->mode_flags & MIPI_DSI_MODE_VIDEO) { 1020 if (msm_host->dsc) { 1021 struct msm_display_dsc_config *dsc = msm_host->dsc; > 1022 u32 reg, intf_width, slice_per_intf, width; 1023 u32 total_bytes_per_intf; 1024 1025 /* first calculate dsc parameters and then program 1026 * compress mode registers 1027 */ 1028 intf_width = hdisplay; 1029 slice_per_intf = DIV_ROUND_UP(intf_width, dsc->drm->slice_width); 1030 1031 /* If slice_count > slice_per_intf, then use 1 1032 * This can happen during partial update 1033 */ 1034 dsc->drm->slice_count = 1; 1035 1036 dsc->bytes_in_slice = DIV_ROUND_UP(dsc->drm->slice_width * 8, 8); 1037 total_bytes_per_intf = dsc->bytes_in_slice * slice_per_intf; 1038 1039 dsc->eol_byte_num = total_bytes_per_intf % 3; 1040 dsc->pclk_per_line = DIV_ROUND_UP(total_bytes_per_intf, 3); 1041 dsc->bytes_per_pkt = dsc->bytes_in_slice * dsc->drm->slice_count; 1042 dsc->pkt_per_line = slice_per_intf / dsc->drm->slice_count; 1043 1044 width = dsc->pclk_per_line; 1045 reg = dsc->bytes_per_pkt << 16; 1046 reg |= (0x0b << 8); /* dtype of compressed image */ 1047 1048 /* pkt_per_line: 1049 * 0 == 1 pkt 1050 * 1 == 2 pkt 1051 * 2 == 4 pkt 1052 * 3 pkt is not supported 1053 * above translates to ffs() - 1 1054 */ 1055 reg |= (ffs(dsc->pkt_per_line) - 1) << 6; 1056 1057 dsc->eol_byte_num = total_bytes_per_intf % 3; 1058 reg |= dsc->eol_byte_num << 4; 1059 reg |= 1; 1060 1061 dsi_write(msm_host, 1062 REG_DSI_VIDEO_COMPRESSION_MODE_CTRL, reg); 1063 } 1064 1065 dsi_write(msm_host, REG_DSI_ACTIVE_H, 1066 DSI_ACTIVE_H_START(ha_start) | 1067 DSI_ACTIVE_H_END(ha_end)); 1068 dsi_write(msm_host, REG_DSI_ACTIVE_V, 1069 DSI_ACTIVE_V_START(va_start) | 1070 DSI_ACTIVE_V_END(va_end)); 1071 dsi_write(msm_host, REG_DSI_TOTAL, 1072 DSI_TOTAL_H_TOTAL(h_total - 1) | 1073 DSI_TOTAL_V_TOTAL(v_total - 1)); 1074 1075 dsi_write(msm_host, REG_DSI_ACTIVE_HSYNC, 1076 DSI_ACTIVE_HSYNC_START(hs_start) | 1077 DSI_ACTIVE_HSYNC_END(hs_end)); 1078 dsi_write(msm_host, REG_DSI_ACTIVE_VSYNC_HPOS, 0); 1079 dsi_write(msm_host, REG_DSI_ACTIVE_VSYNC_VPOS, 1080 DSI_ACTIVE_VSYNC_VPOS_START(vs_start) | 1081 DSI_ACTIVE_VSYNC_VPOS_END(vs_end)); 1082 } else { /* command mode */ 1083 if (msm_host->dsc) { 1084 struct msm_display_dsc_config *dsc = msm_host->dsc; 1085 u32 reg, reg_ctrl, reg_ctrl2; 1086 u32 slice_per_intf, bytes_in_slice, total_bytes_per_intf; 1087 1088 reg_ctrl = dsi_read(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL); 1089 reg_ctrl2 = dsi_read(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL2); 1090 1091 slice_per_intf = DIV_ROUND_UP(hdisplay, dsc->drm->slice_width); 1092 bytes_in_slice = DIV_ROUND_UP(dsc->drm->slice_width * 1093 dsc->drm->bits_per_pixel, 8); 1094 dsc->drm->slice_chunk_size = bytes_in_slice; 1095 total_bytes_per_intf = dsc->bytes_in_slice * slice_per_intf; 1096 dsc->pkt_per_line = slice_per_intf / dsc->drm->slice_count; 1097 1098 reg = 0x39 << 8; 1099 reg |= ffs(dsc->pkt_per_line) << 6; 1100 1101 dsc->eol_byte_num = total_bytes_per_intf % 3; 1102 reg |= dsc->eol_byte_num << 4; 1103 reg |= 1; 1104 1105 reg_ctrl |= reg; 1106 reg_ctrl2 |= bytes_in_slice; 1107 1108 dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL, reg); 1109 dsi_write(msm_host, REG_DSI_COMMAND_COMPRESSION_MODE_CTRL2, reg_ctrl2); 1110 } 1111 1112 /* image data and 1 byte write_memory_start cmd */ 1113 if (!msm_host->dsc) 1114 wc = hdisplay * dsi_get_bpp(msm_host->format) / 8 + 1; 1115 else 1116 wc = mode->hdisplay / 2 + 1; 1117 1118 data = DSI_CMD_MDP_STREAM0_CTRL_WORD_COUNT(wc) | 1119 DSI_CMD_MDP_STREAM0_CTRL_VIRTUAL_CHANNEL(msm_host->channel) | 1120 DSI_CMD_MDP_STREAM0_CTRL_DATA_TYPE(MIPI_DSI_DCS_LONG_WRITE); 1121 1122 dsi_write(msm_host, REG_DSI_CMD_MDP_STREAM0_CTRL, data); 1123 1124 data = DSI_CMD_MDP_STREAM0_TOTAL_H_TOTAL(hdisplay) | 1125 DSI_CMD_MDP_STREAM0_TOTAL_V_TOTAL(mode->vdisplay); 1126 dsi_write(msm_host, REG_DSI_CMD_MDP_STREAM0_TOTAL, data); 1127 } 1128 } 1129 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
Attachment:
.config.gz
Description: application/gzip