• Lucas De Marchi's avatar
    drm: Convert open-coded yes/no strings to yesno() · b8c75bd9
    Lucas De Marchi authored
    linux/string_helpers.h provides a helper to return "yes"/"no" strings.
    Replace the open coded versions with str_yes_no(). The places were
    identified with the following semantic patch:
    
    	@@
    	expression b;
    	@@
    
    	- b ? "yes" : "no"
    	+ str_yes_no(b)
    
    Then the includes were added, so we include-what-we-use, and parenthesis
    adjusted in drivers/gpu/drm/v3d/v3d_debugfs.c. After the conversion we
    still see the same binary sizes:
    
       text    data     bss     dec     hex filename
      51149    3295     212   54656    d580 virtio/virtio-gpu.ko.old
      51149    3295     212   54656    d580 virtio/virtio-gpu.ko
    1441491   60340     800 1502631  16eda7 radeon/radeon.ko.old
    1441491   60340     800 1502631  16eda7 radeon/radeon.ko
    6125369  328538   34000 6487907  62ff63 amd/amdgpu/amdgpu.ko.old
    6125369  328538   34000 6487907  62ff63 amd/amdgpu/amdgpu.ko
     411986   10490    6176  428652   68a6c drm.ko.old
     411986   10490    6176  428652   68a6c drm.ko
      98129    1636     264  100029   186bd dp/drm_dp_helper.ko.old
      98129    1636     264  100029   186bd dp/drm_dp_helper.ko
    1973432  109640    2352 2085424  1fd230 nouveau/nouveau.ko.old
    1973432  109640    2352 2085424  1fd230 nouveau/nouveau.ko
    Signed-off-by: default avatarLucas De Marchi <lucas.demarchi@intel.com>
    Reviewed-by: default avatarJani Nikula <jani.nikula@intel.com>
    Link: https://patchwork.freedesktop.org/patch/msgid/20220126093951.1470898-10-lucas.demarchi@intel.com
    b8c75bd9
drm_dp.c 104 KB