Re: [PATCH 1/2] drm/amdgpu: Add NBIO SMN headers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 4, 2019 at 7:35 AM Russell, Kent <Kent.Russell@xxxxxxx> wrote:
>
> We need these offsets for PCIE perf counters, so include them as well as
> the the previously-used defines from the nbio_*.c files
>
> Change-Id: I0e090e5b5b97359a67b28952f069ff6e603edf10
> Signed-off-by: Kent Russell <kent.russell@xxxxxxx>
> ---
>  drivers/gpu/drm/amd/amdgpu/nbio_v6_1.c             |  6 +--
>  drivers/gpu/drm/amd/amdgpu/nbio_v7_0.c             |  6 +--
>  drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c             |  6 +--
>  .../drm/amd/include/asic_reg/nbio/nbio_6_1_smn.h   | 58 ++++++++++++++++++++++
>  .../drm/amd/include/asic_reg/nbio/nbio_7_0_smn.h   | 55 ++++++++++++++++++++
>  .../drm/amd/include/asic_reg/nbio/nbio_7_4_0_smn.h | 53 ++++++++++++++++++++
>  6 files changed, 169 insertions(+), 15 deletions(-)
>  create mode 100644 drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_6_1_smn.h
>  create mode 100644 drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_0_smn.h
>  create mode 100644 drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_4_0_smn.h
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v6_1.c b/drivers/gpu/drm/amd/amdgpu/nbio_v6_1.c
> index accdedd..1965756 100644
> --- a/drivers/gpu/drm/amd/amdgpu/nbio_v6_1.c
> +++ b/drivers/gpu/drm/amd/amdgpu/nbio_v6_1.c
> @@ -27,13 +27,9 @@
>  #include "nbio/nbio_6_1_default.h"
>  #include "nbio/nbio_6_1_offset.h"
>  #include "nbio/nbio_6_1_sh_mask.h"
> +#include "nbio/nbio_6_1_smn.h"
>  #include "vega10_enum.h"
>
> -#define smnCPM_CONTROL                                                                                  0x11180460
> -#define smnPCIE_CNTL2                                                                                   0x11180070
> -#define smnPCIE_CONFIG_CNTL                                                                             0x11180044
> -#define smnPCIE_CI_CNTL                                                                                 0x11180080
> -
>  static u32 nbio_v6_1_get_rev_id(struct amdgpu_device *adev)
>  {
>          u32 tmp = RREG32_SOC15(NBIO, 0, mmRCC_DEV0_EPF0_STRAP0);
> diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v7_0.c b/drivers/gpu/drm/amd/amdgpu/nbio_v7_0.c
> index df34dc7..f718fe2 100644
> --- a/drivers/gpu/drm/amd/amdgpu/nbio_v7_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/nbio_v7_0.c
> @@ -27,13 +27,9 @@
>  #include "nbio/nbio_7_0_default.h"
>  #include "nbio/nbio_7_0_offset.h"
>  #include "nbio/nbio_7_0_sh_mask.h"
> +#include "nbio/nbio_7_0_smn.h"
>  #include "vega10_enum.h"
>
> -#define smnNBIF_MGCG_CTRL_LCLK 0x1013a05c
> -
> -#define smnCPM_CONTROL                                                                                  0x11180460
> -#define smnPCIE_CNTL2                                                                                   0x11180070
> -
>  static u32 nbio_v7_0_get_rev_id(struct amdgpu_device *adev)
>  {
>          u32 tmp = RREG32_SOC15(NBIO, 0, mmRCC_DEV0_EPF0_STRAP0);
> diff --git a/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c b/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c
> index 4cd31a2..c402727 100644
> --- a/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c
> +++ b/drivers/gpu/drm/amd/amdgpu/nbio_v7_4.c
> @@ -26,12 +26,8 @@
>
>  #include "nbio/nbio_7_4_offset.h"
>  #include "nbio/nbio_7_4_sh_mask.h"
> +#include "nbio/nbio_7_4_0_smn.h"
>
> -#define smnNBIF_MGCG_CTRL_LCLK 0x1013a21c

I think technically the smnNBIF registers should be in the
nbif_*_smn.h headers so maybe leave that here.  Same comment on other
IP revisions.  Either way:
Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

> -
> -#define smnCPM_CONTROL                                                                                  0x11180460
> -#define smnPCIE_CNTL2                                                                                   0x11180070
> -#define smnPCIE_CI_CNTL                                                                                 0x11180080
>
>  static u32 nbio_v7_4_get_rev_id(struct amdgpu_device *adev)
>  {
> diff --git a/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_6_1_smn.h b/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_6_1_smn.h
> new file mode 100644
> index 0000000..8c75669
> --- /dev/null
> +++ b/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_6_1_smn.h
> @@ -0,0 +1,58 @@
> +/*
> + * Copyright (C) 2019  Advanced Micro Devices, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included
> + * in all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
> + * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE COPYRIGHT HOLDER(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN
> + * AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> + */
> +
> +#ifndef _nbio_6_1_SMN_HEADER
> +#define _nbio_6_1_SMN_HEADER
> +
> +
> +#define smnCPM_CONTROL                                 0x11180460
> +#define smnPCIE_CNTL2                                  0x11180070
> +#define smnPCIE_CONFIG_CNTL                            0x11180044
> +#define smnPCIE_CI_CNTL                                        0x11180080
> +
> +
> +#define smnPCIE_PERF_COUNT_CNTL                                0x11180200
> +#define smnPCIE_PERF_CNTL_TXCLK                                0x11180204
> +#define smnPCIE_PERF_COUNT0_TXCLK                      0x11180208
> +#define smnPCIE_PERF_COUNT1_TXCLK                      0x1118020c
> +#define smnPCIE_PERF_CNTL_MST_R_CLK                    0x11180210
> +#define smnPCIE_PERF_COUNT0_MST_R_CLK                  0x11180214
> +#define smnPCIE_PERF_COUNT1_MST_R_CLK                  0x11180218
> +#define smnPCIE_PERF_CNTL_MST_C_CLK                    0x1118021c
> +#define smnPCIE_PERF_COUNT0_MST_C_CLK                  0x11180220
> +#define smnPCIE_PERF_COUNT1_MST_C_CLK                  0x11180224
> +#define smnPCIE_PERF_CNTL_SLV_R_CLK                    0x11180228
> +#define smnPCIE_PERF_COUNT0_SLV_R_CLK                  0x1118022c
> +#define smnPCIE_PERF_COUNT1_SLV_R_CLK                  0x11180230
> +#define smnPCIE_PERF_CNTL_SLV_S_C_CLK                  0x11180234
> +#define smnPCIE_PERF_COUNT0_SLV_S_C_CLK                        0x11180238
> +#define smnPCIE_PERF_COUNT1_SLV_S_C_CLK                        0x1118023c
> +#define smnPCIE_PERF_CNTL_SLV_NS_C_CLK                 0x11180240
> +#define smnPCIE_PERF_COUNT0_SLV_NS_C_CLK               0x11180244
> +#define smnPCIE_PERF_COUNT1_SLV_NS_C_CLK               0x11180248
> +#define smnPCIE_PERF_CNTL_EVENT0_PORT_SEL              0x1118024c
> +#define smnPCIE_PERF_CNTL_EVENT1_PORT_SEL              0x11180250
> +#define smnPCIE_PERF_CNTL_TXCLK2                       0x11180254
> +#define smnPCIE_PERF_COUNT0_TXCLK2                     0x11180258
> +#define smnPCIE_PERF_COUNT1_TXCLK2                     0x1118025c
> +
> +#endif // _nbio_6_1_SMN_HEADER
> +
> diff --git a/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_0_smn.h b/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_0_smn.h
> new file mode 100644
> index 0000000..3ffd221
> --- /dev/null
> +++ b/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_0_smn.h
> @@ -0,0 +1,55 @@
> +/*
> + * Copyright (C) 2019  Advanced Micro Devices, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included
> + * in all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
> + * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE COPYRIGHT HOLDER(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN
> + * AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> + */
> +
> +#ifndef _nbio_7_0_SMN_HEADER
> +#define _nbio_7_0_SMN_HEADER
> +
> +
> +#define smnNBIF_MGCG_CTRL_LCLK                         0x1013a05c
> +#define smnCPM_CONTROL                                 0x11180460
> +#define smnPCIE_CNTL2                                  0x11180070
> +
> +#define smnPCIE_PERF_COUNT_CNTL                                0x11180200
> +#define smnPCIE_PERF_CNTL_TXCLK                                0x11180204
> +#define smnPCIE_PERF_COUNT0_TXCLK                      0x11180208
> +#define smnPCIE_PERF_COUNT1_TXCLK                      0x1118020c
> +#define smnPCIE_PERF_CNTL_MST_R_CLK                    0x11180210
> +#define smnPCIE_PERF_COUNT0_MST_R_CLK                  0x11180214
> +#define smnPCIE_PERF_COUNT1_MST_R_CLK                  0x11180218
> +#define smnPCIE_PERF_CNTL_MST_C_CLK                    0x1118021c
> +#define smnPCIE_PERF_COUNT0_MST_C_CLK                  0x11180220
> +#define smnPCIE_PERF_COUNT1_MST_C_CLK                  0x11180224
> +#define smnPCIE_PERF_CNTL_SLV_R_CLK                    0x11180228
> +#define smnPCIE_PERF_COUNT0_SLV_R_CLK                  0x1118022c
> +#define smnPCIE_PERF_COUNT1_SLV_R_CLK                  0x11180230
> +#define smnPCIE_PERF_CNTL_SLV_S_C_CLK                  0x11180234
> +#define smnPCIE_PERF_COUNT0_SLV_S_C_CLK                        0x11180238
> +#define smnPCIE_PERF_COUNT1_SLV_S_C_CLK                        0x1118023c
> +#define smnPCIE_PERF_CNTL_SLV_NS_C_CLK                 0x11180240
> +#define smnPCIE_PERF_COUNT0_SLV_NS_C_CLK               0x11180244
> +#define smnPCIE_PERF_COUNT1_SLV_NS_C_CLK               0x11180248
> +#define smnPCIE_PERF_CNTL_EVENT0_PORT_SEL              0x1118024c
> +#define smnPCIE_PERF_CNTL_EVENT1_PORT_SEL              0x11180250
> +#define smnPCIE_PERF_CNTL_TXCLK2                       0x11180254
> +#define smnPCIE_PERF_COUNT0_TXCLK2                     0x11180258
> +#define smnPCIE_PERF_COUNT1_TXCLK2                     0x1118025c
> +
> +#endif // _nbio_7_0_SMN_HEADER
> diff --git a/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_4_0_smn.h b/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_4_0_smn.h
> new file mode 100644
> index 0000000..c1457d8
> --- /dev/null
> +++ b/drivers/gpu/drm/amd/include/asic_reg/nbio/nbio_7_4_0_smn.h
> @@ -0,0 +1,53 @@
> +/*
> + * Copyright (C) 2019  Advanced Micro Devices, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice shall be included
> + * in all copies or substantial portions of the Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
> + * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> + * THE COPYRIGHT HOLDER(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN
> + * AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> + */
> +
> +#ifndef _nbio_7_4_0_SMN_HEADER
> +#define _nbio_7_4_0_SMN_HEADER
> +
> +
> +#define smnNBIF_MGCG_CTRL_LCLK                         0x1013a21c
> +#define smnCPM_CONTROL                                 0x11180460
> +#define smnPCIE_CNTL2                                  0x11180070
> +#define smnPCIE_CI_CNTL                                        0x11180080
> +
> +#define smnPCIE_PERF_COUNT_CNTL                                0x11180200
> +#define smnPCIE_PERF_CNTL_TXCLK1                       0x11180204
> +#define smnPCIE_PERF_COUNT0_TXCLK1                     0x11180208
> +#define smnPCIE_PERF_COUNT1_TXCLK1                     0x1118020c
> +#define smnPCIE_PERF_CNTL_TXCLK2                       0x11180210
> +#define smnPCIE_PERF_COUNT0_TXCLK2                     0x11180214
> +#define smnPCIE_PERF_COUNT1_TXCLK2                     0x11180218
> +#define smnPCIE_PERF_CNTL_TXCLK3                       0x1118021c
> +#define smnPCIE_PERF_COUNT0_TXCLK3                     0x11180220
> +#define smnPCIE_PERF_COUNT1_TXCLK3                     0x11180224
> +#define smnPCIE_PERF_CNTL_TXCLK4                       0x11180228
> +#define smnPCIE_PERF_COUNT0_TXCLK4                     0x1118022c
> +#define smnPCIE_PERF_COUNT1_TXCLK4                     0x11180230
> +#define smnPCIE_PERF_CNTL_SCLK1                                0x11180234
> +#define smnPCIE_PERF_COUNT0_SCLK1                      0x11180238
> +#define smnPCIE_PERF_COUNT1_SCLK1                      0x1118023c
> +#define smnPCIE_PERF_CNTL_SCLK2                                0x11180240
> +#define smnPCIE_PERF_COUNT0_SCLK2                      0x11180244
> +#define smnPCIE_PERF_COUNT1_SCLK2                      0x11180248
> +#define smnPCIE_PERF_CNTL_EVENT_LC_PORT_SEL            0x1118024c
> +#define smnPCIE_PERF_CNTL_EVENT_CI_PORT_SEL            0x11180250
> +
> +#endif // _nbio_7_4_0_SMN_HEADER
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux