xf86-video-siliconmotion: Changes to 'master'

[GitLab Mirror]

 src/smi501_output.c |    1 
 src/smi_501.h       |  102 ++++++++++++++++++++++++++--------------------------
 src/smi_driver.c    |    3 -
 src/smi_exa.c       |    4 +-
 src/smi_video.c     |   19 ++++-----
 5 files changed, 63 insertions(+), 66 deletions(-)

New commits:
commit 2af1096f004a77f7970473786e13ef54abb0e023
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Apr 12 18:02:57 2025 -0700

    Handle 3 -Wmissing-variable-declarations warnings
    
    Reported by clang 19.1.7:
    
    smi_exa.c:201:7: warning: no previous extern declaration for non-static
     variable 'SMI_BltRop' [-Wmissing-variable-declarations]
      201 | CARD8 SMI_BltRop[16] =  /* table stolen from KAA */
          |       ^
    
    smi_exa.c:347:7: warning: no previous extern declaration for non-static
     variable 'SMI_SolidRop' [-Wmissing-variable-declarations]
      347 | CARD8 SMI_SolidRop[16] =        /* table stolen from KAA */
          |       ^
    
    smi_driver.c:101:5: warning: no previous extern declaration for non-static
     variable 'gSMIEntityIndex' [-Wmissing-variable-declarations]
      101 | int gSMIEntityIndex = -1;
          |     ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-siliconmotion/-/merge_requests/9>

commit 523f670b38daa85be473c57975b1fe95d9b22940
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Apr 12 17:58:21 2025 -0700

    SMI501_OutputDPMS_lcd: Fix -Wimplicit-fallthrough warning
    
    Reported by clang 19.1.7:
    
    smi501_output.c:52:5: warning: unannotated fall-through between switch labels
     [-Wimplicit-fallthrough]
       52 |     case DPMSModeStandby:
          |     ^
    smi501_output.c:52:5: note: insert 'break;' to avoid fall-through
       52 |     case DPMSModeStandby:
          |     ^
          |     break;
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-siliconmotion/-/merge_requests/9>

commit 70ac2f1c41bc4d56bfe71fedb5e5f233d2ec8325
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Apr 12 17:53:19 2025 -0700

    SMI_DisplayVideo0501_CSC: Handle -Wshift-sign-overflow warnings
    
    Reported by clang 19.1.7:
    
    smi_video.c:1836:14: warning: signed shift result (0x80000000) sets the
     sign bit of the shift expression's type ('int') and becomes negative
     [-Wshift-sign-overflow]
     1836 |     csc = (1 << 31) | (1 << 25);
          |            ~ ^  ~~
    
    smi_video.c:1897:35: warning: signed shift result (0x80000000) sets the
     sign bit of the shift expression's type ('int') and becomes negative
     [-Wshift-sign-overflow]
     1897 |         while (READ_DPR(pSmi, 0xfc) & (1 << 31))
          |                                        ~ ^  ~~
    
    smi_video.c:1901:35: warning: signed shift result (0x80000000) sets the
     sign bit of the shift expression's type ('int') and becomes negative
     [-Wshift-sign-overflow]
     1901 |         while (READ_DPR(pSmi, 0xfc) & (1 << 31))
          |                                        ~ ^  ~~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-siliconmotion/-/merge_requests/9>

commit b0846c0d8f37186ccb46f45c1edde7e9a3dcf29a
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Apr 12 17:46:24 2025 -0700

    Handle -Wconditional-uninitialized warning
    
    Reported by clang 19.1.7:
    
    smi_video.c:1629:5: warning: variable 'crtc' may be uninitialized when
     used here [-Wconditional-uninitialized]
     1629 |         if(crtc == crtcConf->crtc[0])
          |            ^~~~
    smi_video.c:1493:21: note: initialize the variable 'crtc' to silence this
     warning
     1493 |     xf86CrtcPtr crtc;
          |                     ^
          |                      = NULL
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-siliconmotion/-/merge_requests/9>

commit 14913de4d9d8f95313a287079cba400a3dabc7ab
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Apr 12 17:42:57 2025 -0700

    smi_video.c: Removed unused macro ABS()
    
    Reported by clang 19.1.7:
    
    smi_video.c:67:9: warning: macro is not used [-Wunused-macros]
       67 | #define ABS(n) (((n) < 0) ? -(n) : (n))
          |         ^
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-siliconmotion/-/merge_requests/9>

commit 63c44903e565f0a7fe3a8d8166ffaa012aeae673
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sat Apr 12 17:38:40 2025 -0700

    Clear up -Wsingle-bit-bitfield-constant-conversion warnings
    
    Gets rid of 21 warnings from clang 19.1.7 of the form:
    
    smi_501.c:191:25: warning: implicit truncation from 'int' to a
     one-bit wide bit-field changes value from 1 to -1
     [-Wsingle-bit-bitfield-constant-conversion]
      191 |     mode->gate.f.engine = 1;
          |                         ^ ~
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
    Part-of: <https://gitlab.freedesktop.org/xorg/driver/xf86-video-siliconmotion/-/merge_requests/9>