lkml.org 
[lkml]   [2020]   [Nov]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH 4/6] drm/amdgpu/sdma: use "*" adjacent to data name
    When declaring pointer data, the "*" symbol should be used adjacent to
    the data name as per the coding standards. This resolves following
    issues reported by checkpatch script:
    ERROR: "foo * bar" should be "foo *bar"
    ERROR: "foo * bar" should be "foo *bar"
    ERROR: "foo* bar" should be "foo *bar"
    ERROR: "(foo*)" should be "(foo *)"

    Signed-off-by: Deepak R Varma <mh12gx2825@gmail.com>
    ---
    drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c | 6 +++---
    drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c | 6 +++---
    2 files changed, 6 insertions(+), 6 deletions(-)

    diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
    index 86fb1eddf5a6..d1150c33d02e 100644
    --- a/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
    +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v4_0.c
    @@ -568,7 +568,7 @@ static void sdma_v4_0_destroy_inst_ctx(struct amdgpu_device *adev)
    break;
    }

    - memset((void*)adev->sdma.instance, 0,
    + memset((void *)adev->sdma.instance, 0,
    sizeof(struct amdgpu_sdma_instance) * AMDGPU_MAX_SDMA_INSTANCES);
    }

    @@ -639,8 +639,8 @@ static int sdma_v4_0_init_microcode(struct amdgpu_device *adev)
    if (adev->asic_type == CHIP_ARCTURUS) {
    /* Acturus will leverage the same FW memory
    for every SDMA instance */
    - memcpy((void*)&adev->sdma.instance[i],
    - (void*)&adev->sdma.instance[0],
    + memcpy((void *)&adev->sdma.instance[i],
    + (void *)&adev->sdma.instance[0],
    sizeof(struct amdgpu_sdma_instance));
    }
    else {
    diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
    index 9f3952723c63..90c47f1801fd 100644
    --- a/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
    +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v5_2.c
    @@ -124,7 +124,7 @@ static void sdma_v5_2_destroy_inst_ctx(struct amdgpu_device *adev)
    break;
    }

    - memset((void*)adev->sdma.instance, 0,
    + memset((void *)adev->sdma.instance, 0,
    sizeof(struct amdgpu_sdma_instance) * AMDGPU_MAX_SDMA_INSTANCES);
    }

    @@ -177,8 +177,8 @@ static int sdma_v5_2_init_microcode(struct amdgpu_device *adev)
    for (i = 1; i < adev->sdma.num_instances; i++) {
    if (adev->asic_type == CHIP_SIENNA_CICHLID ||
    adev->asic_type == CHIP_NAVY_FLOUNDER) {
    - memcpy((void*)&adev->sdma.instance[i],
    - (void*)&adev->sdma.instance[0],
    + memcpy((void *)&adev->sdma.instance[i],
    + (void *)&adev->sdma.instance[0],
    sizeof(struct amdgpu_sdma_instance));
    } else {
    snprintf(fw_name, sizeof(fw_name), "amdgpu/%s_sdma%d.bin", chip_name, i);
    --
    2.25.1
    \
     
     \ /
      Last update: 2020-11-02 20:37    [W:2.558 / U:0.220 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site