lkml.org 
[lkml]   [2020]   [Feb]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 5.5 139/542] drm/amdgpu: remove 4 set but not used variable in amdgpu_atombios_get_connector_info_from_object_table
    Date
    From: yu kuai <yukuai3@huawei.com>

    [ Upstream commit bae028e3e521e8cb8caf2cc16a455ce4c55f2332 ]

    Fixes gcc '-Wunused-but-set-variable' warning:

    drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c: In function
    'amdgpu_atombios_get_connector_info_from_object_table':
    drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c:376:26: warning: variable
    'grph_obj_num' set but not used [-Wunused-but-set-variable]
    drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c:376:13: warning: variable
    'grph_obj_id' set but not used [-Wunused-but-set-variable]
    drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c:341:37: warning: variable
    'con_obj_type' set but not used [-Wunused-but-set-variable]
    drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c:341:24: warning: variable
    'con_obj_num' set but not used [-Wunused-but-set-variable]

    They are never used, so can be removed.

    Fixes: d38ceaf99ed0 ("drm/amdgpu: add core driver (v4)")
    Signed-off-by: yu kuai <yukuai3@huawei.com>
    Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c | 19 ++-----------------
    1 file changed, 2 insertions(+), 17 deletions(-)

    diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c
    index 72232fccf61a7..be6d0cfe41aec 100644
    --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c
    +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_atombios.c
    @@ -338,17 +338,9 @@ bool amdgpu_atombios_get_connector_info_from_object_table(struct amdgpu_device *
    path_size += le16_to_cpu(path->usSize);

    if (device_support & le16_to_cpu(path->usDeviceTag)) {
    - uint8_t con_obj_id, con_obj_num, con_obj_type;
    -
    - con_obj_id =
    + uint8_t con_obj_id =
    (le16_to_cpu(path->usConnObjectId) & OBJECT_ID_MASK)
    >> OBJECT_ID_SHIFT;
    - con_obj_num =
    - (le16_to_cpu(path->usConnObjectId) & ENUM_ID_MASK)
    - >> ENUM_ID_SHIFT;
    - con_obj_type =
    - (le16_to_cpu(path->usConnObjectId) &
    - OBJECT_TYPE_MASK) >> OBJECT_TYPE_SHIFT;

    /* Skip TV/CV support */
    if ((le16_to_cpu(path->usDeviceTag) ==
    @@ -373,14 +365,7 @@ bool amdgpu_atombios_get_connector_info_from_object_table(struct amdgpu_device *
    router.ddc_valid = false;
    router.cd_valid = false;
    for (j = 0; j < ((le16_to_cpu(path->usSize) - 8) / 2); j++) {
    - uint8_t grph_obj_id, grph_obj_num, grph_obj_type;
    -
    - grph_obj_id =
    - (le16_to_cpu(path->usGraphicObjIds[j]) &
    - OBJECT_ID_MASK) >> OBJECT_ID_SHIFT;
    - grph_obj_num =
    - (le16_to_cpu(path->usGraphicObjIds[j]) &
    - ENUM_ID_MASK) >> ENUM_ID_SHIFT;
    + uint8_t grph_obj_type=
    grph_obj_type =
    (le16_to_cpu(path->usGraphicObjIds[j]) &
    OBJECT_TYPE_MASK) >> OBJECT_TYPE_SHIFT;
    --
    2.20.1
    \
     
     \ /
      Last update: 2020-02-14 16:52    [W:3.766 / U:0.100 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site