CC: [email protected]
BCC: [email protected]
CC: [email protected]
TO: Allen Chen <[email protected]>
CC: Robert Foss <[email protected]>
CC: Hermes Wu <[email protected]>
CC: AngeloGioacchino Del Regno <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   672c0c5173427e6b3e2a9bbb7be51ceeec78093a
commit: b5c84a9edcd418cd055becad6a22439e7c5e3bf8 drm/bridge: add it6505 driver
date:   3 months ago
:::::: branch date: 13 hours ago
:::::: commit date: 3 months ago
compiler: ia64-linux-gcc (GCC) 11.3.0
reproduce (cppcheck warning):
        # apt-get install cppcheck
        git checkout b5c84a9edcd418cd055becad6a22439e7c5e3bf8
        cppcheck --quiet --enable=style,performance,portability --template=gcc 
FILE

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>


cppcheck warnings: (new ones prefixed by >>)
>> drivers/gpu/drm/etnaviv/etnaviv_gem.c:373:9: warning: Local variable 'ret' 
>> shadows outer variable [shadowVariable]
     void *ret;
           ^
   drivers/gpu/drm/etnaviv/etnaviv_gem.c:370:6: note: Shadowed declaration
    int ret;
        ^
   drivers/gpu/drm/etnaviv/etnaviv_gem.c:373:9: note: Shadow variable
     void *ret;
           ^
>> drivers/gpu/drm/radeon/radeon_cs.c:105:31: warning: Local variable 'r' 
>> shadows outer variable [shadowVariable]
     struct drm_radeon_cs_reloc *r;
                                 ^
   drivers/gpu/drm/radeon/radeon_cs.c:87:6: note: Shadowed declaration
    int r;
        ^
   drivers/gpu/drm/radeon/radeon_cs.c:105:31: note: Shadow variable
     struct drm_radeon_cs_reloc *r;
                                 ^

cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> drivers/gpu/drm/etnaviv/etnaviv_gem.c:216:39: warning: Parameter 'context' 
>> can be declared with const [constParameter]
           struct etnaviv_iommu_context *context)
                                         ^
>> drivers/gpu/drm/etnaviv/etnaviv_gem.c:221:16: warning: Uninitialized 
>> variable: mapping->context [uninitvar]
     if (mapping->context == context)
                  ^
>> drivers/gpu/drm/etnaviv/etnaviv_gem.c:502:37: warning: Uninitialized 
>> variables: mapping.obj_node, mapping.scan_node, mapping.mmu_node, 
>> mapping.object, mapping.context, mapping.vram_node, mapping.iova [uninitvar]
      etnaviv_iommu_unmap_gem(context, mapping);
                                       ^
--
>> drivers/gpu/drm/radeon/atombios_dp.c:351:19: warning: Assigning an integer 
>> to a pointer is not portable. [AssignmentIntegerToAddress]
    args.ucLinkClock = dp_clock / 10;
                     ^
--
>> drivers/gpu/drm/radeon/radeon_cs.c:259:17: warning: Uninitialized variable: 
>> reloc->robj [uninitvar]
     resv = reloc->robj->tbo.base.resv;
                   ^
--
>> drivers/gpu/drm/radeon/radeon_connectors.c:1865:25: warning: Parameter 'hpd' 
>> can be declared with const [constParameter]
        struct radeon_hpd *hpd,
                           ^
   drivers/gpu/drm/radeon/radeon_connectors.c:2390:27: warning: Parameter 'hpd' 
can be declared with const [constParameter]
          struct radeon_hpd *hpd)
                             ^
>> drivers/gpu/drm/radeon/radeon_connectors.c:418:33: warning: Parameter 
>> 'encoder' can be declared with const [constParameter]
               struct drm_encoder *encoder,
                                   ^
>> drivers/gpu/drm/radeon/radeon_connectors.c:547:96: warning: Parameter 
>> 'property' can be declared with const [constParameter]
   static int radeon_connector_set_property(struct drm_connector *connector, 
struct drm_property *property,
                                                                                
                  ^
   drivers/gpu/drm/radeon/radeon_connectors.c:940:30: warning: Parameter 
'property' can be declared with const [constParameter]
           struct drm_property *property,
                                ^
>> drivers/gpu/drm/bridge/ite-it6505.c:1330:5: warning: Variable 
>> 'audio_word_length_map' can be declared with const [constVariable]
    u8 audio_word_length_map[] = { 0x02, 0x04, 0x03, 0x0B };
       ^
>> drivers/gpu/drm/bridge/ite-it6505.c:1620:56: warning: Parameter 
>> 'lane_voltage_swing' can be declared with const [constParameter]
   static bool it6505_check_max_voltage_swing_reached(u8 *lane_voltage_swing,
                                                          ^

vim +/audio_word_length_map +1330 drivers/gpu/drm/bridge/ite-it6505.c

b5c84a9edcd418 Allen Chen 2022-01-14  1326  
b5c84a9edcd418 Allen Chen 2022-01-14  1327  static void 
it6505_setup_audio_channel_status(struct it6505 *it6505)
b5c84a9edcd418 Allen Chen 2022-01-14  1328  {
b5c84a9edcd418 Allen Chen 2022-01-14  1329      enum it6505_audio_sample_rate 
sample_rate = it6505->audio.sample_rate;
b5c84a9edcd418 Allen Chen 2022-01-14 @1330      u8 audio_word_length_map[] = { 
0x02, 0x04, 0x03, 0x0B };
b5c84a9edcd418 Allen Chen 2022-01-14  1331  
b5c84a9edcd418 Allen Chen 2022-01-14  1332      /* Channel Status */
b5c84a9edcd418 Allen Chen 2022-01-14  1333      it6505_write(it6505, 
REG_IEC958_STS0, it6505->audio.type << 1);
b5c84a9edcd418 Allen Chen 2022-01-14  1334      it6505_write(it6505, 
REG_IEC958_STS1, 0x00);
b5c84a9edcd418 Allen Chen 2022-01-14  1335      it6505_write(it6505, 
REG_IEC958_STS2, 0x00);
b5c84a9edcd418 Allen Chen 2022-01-14  1336      it6505_write(it6505, 
REG_IEC958_STS3, sample_rate);
b5c84a9edcd418 Allen Chen 2022-01-14  1337      it6505_write(it6505, 
REG_IEC958_STS4, (~sample_rate << 4) |
b5c84a9edcd418 Allen Chen 2022-01-14  1338                   
audio_word_length_map[it6505->audio.word_length]);
b5c84a9edcd418 Allen Chen 2022-01-14  1339  }
b5c84a9edcd418 Allen Chen 2022-01-14  1340  

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to