tree:   git://people.freedesktop.org/~agd5f/linux.git amd-mainline-dkms-4.15
head:   9556f93f18f7923978fb90f860c107fed9ca7f57
commit: 265083076187e619aa9176aeb05ad630013429b4 [1231/1759] drm/amd/display: 
Hookup color management functions

smatch warnings:
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:2183 
fill_plane_attributes() warn: if statement not indented

git remote add radeon-alex git://people.freedesktop.org/~agd5f/linux.git
git remote update radeon-alex
git checkout 265083076187e619aa9176aeb05ad630013429b4
vim +2183 drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c

e7b07ceef Harry Wentland   2017-08-10  2143  
3ee6b26b7 Alex Deucher     2017-10-10  2144  static int 
fill_plane_attributes(struct amdgpu_device *adev,
3be5262e3 Harry Wentland   2017-07-27  2145                              struct 
dc_plane_state *dc_plane_state,
e7b07ceef Harry Wentland   2017-08-10  2146                              struct 
drm_plane_state *plane_state,
ab8968728 Michel Dänzer    2017-10-26  2147                              struct 
drm_crtc_state *crtc_state)
e7b07ceef Harry Wentland   2017-08-10  2148  {
e7b07ceef Harry Wentland   2017-08-10  2149     const struct amdgpu_framebuffer 
*amdgpu_fb =
e7b07ceef Harry Wentland   2017-08-10  2150             
to_amdgpu_framebuffer(plane_state->fb);
25b138493 Roger He         2018-02-26  2151  #if LINUX_VERSION_CODE >= 
KERNEL_VERSION(4, 6, 0) || \
25b138493 Roger He         2018-02-26  2152     defined(OS_NAME_RHEL_7_3) || \
25b138493 Roger He         2018-02-26  2153     defined(OS_NAME_RHEL_7_4_5)
e7b07ceef Harry Wentland   2017-08-10  2154     const struct drm_crtc *crtc = 
plane_state->crtc;
30b13a59d Junwei Zhang     2018-02-08  2155  #else
30b13a59d Junwei Zhang     2018-02-08  2156     struct drm_crtc *crtc = 
plane_state->crtc;
30b13a59d Junwei Zhang     2018-02-08  2157  #endif
e7b07ceef Harry Wentland   2017-08-10  2158     struct dc_transfer_func 
*input_tf;
e7b07ceef Harry Wentland   2017-08-10  2159     int ret = 0;
e7b07ceef Harry Wentland   2017-08-10  2160  
3be5262e3 Harry Wentland   2017-07-27  2161     if 
(!fill_rects_from_plane_state(plane_state, dc_plane_state))
e7b07ceef Harry Wentland   2017-08-10  2162             return -EINVAL;
e7b07ceef Harry Wentland   2017-08-10  2163  
e7b07ceef Harry Wentland   2017-08-10  2164     ret = 
fill_plane_attributes_from_fb(
e7b07ceef Harry Wentland   2017-08-10  2165             crtc->dev->dev_private,
3be5262e3 Harry Wentland   2017-07-27  2166             dc_plane_state,
ab8968728 Michel Dänzer    2017-10-26  2167             amdgpu_fb);
e7b07ceef Harry Wentland   2017-08-10  2168  
e7b07ceef Harry Wentland   2017-08-10  2169     if (ret)
e7b07ceef Harry Wentland   2017-08-10  2170             return ret;
e7b07ceef Harry Wentland   2017-08-10  2171  
e7b07ceef Harry Wentland   2017-08-10  2172     input_tf = 
dc_create_transfer_func();
e7b07ceef Harry Wentland   2017-08-10  2173  
e7b07ceef Harry Wentland   2017-08-10  2174     if (input_tf == NULL)
e7b07ceef Harry Wentland   2017-08-10  2175             return -ENOMEM;
e7b07ceef Harry Wentland   2017-08-10  2176  
3be5262e3 Harry Wentland   2017-07-27  2177     
dc_plane_state->in_transfer_func = input_tf;
e7b07ceef Harry Wentland   2017-08-10  2178  
e7b07ceef Harry Wentland   2017-08-10  2179     /* In case of gamma set, update 
gamma value */
25b138493 Roger He         2018-02-26  2180  #if LINUX_VERSION_CODE >= 
KERNEL_VERSION(4, 6, 0) || \
25b138493 Roger He         2018-02-26  2181     defined(OS_NAME_RHEL_7_3) || \
25b138493 Roger He         2018-02-26  2182     defined(OS_NAME_RHEL_7_4_5)
e7b07ceef Harry Wentland   2017-08-10 @2183     if (crtc_state->gamma_lut)
265083076 Leo (Sunpeng  Li 2018-02-02  2184)    /*
265083076 Leo (Sunpeng  Li 2018-02-02  2185)     * Always set input transfer 
function, since plane state is refreshed
265083076 Leo (Sunpeng  Li 2018-02-02  2186)     * every time.
265083076 Leo (Sunpeng  Li 2018-02-02  2187)     */
265083076 Leo (Sunpeng  Li 2018-02-02  2188)    ret = 
amdgpu_dm_set_degamma_lut(crtc_state, dc_plane_state);

It should be indented and I would normally use curly braces around the
whole thing as well for readability.

3f4b5749a Junwei Zhang     2018-02-07  2189  #else
3f4b5749a Junwei Zhang     2018-02-07  2190     if (crtc->mode.private_flags & 
AMDGPU_CRTC_MODE_PRIVATE_FLAGS_GAMMASET) {
3f4b5749a Junwei Zhang     2018-02-07  2191             
fill_gamma_from_crtc(crtc, dc_plane_state);
3f4b5749a Junwei Zhang     2018-02-07  2192             /* reset trigger of 
gamma */
3f4b5749a Junwei Zhang     2018-02-07  2193             
crtc->mode.private_flags &=
3f4b5749a Junwei Zhang     2018-02-07  2194                     
~AMDGPU_CRTC_MODE_PRIVATE_FLAGS_GAMMASET;
3f4b5749a Junwei Zhang     2018-02-07  2195     }
3f4b5749a Junwei Zhang     2018-02-07  2196  #endif
e7b07ceef Harry Wentland   2017-08-10  2197  
e7b07ceef Harry Wentland   2017-08-10  2198     return ret;
e7b07ceef Harry Wentland   2017-08-10  2199  }
e7b07ceef Harry Wentland   2017-08-10  2200  

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to