[PATCH v2] drm/amdgpu: fix power distribution issue for Polaris10 XT
Deucher, Alexander
Alexander.Deucher at amd.com
Tue Jul 12 02:54:20 UTC 2016
> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of zhoucm1
> Sent: Monday, July 11, 2016 10:43 PM
> To: Wang, Qingqing; amd-gfx at lists.freedesktop.org
> Subject: Re: [PATCH v2] drm/amdgpu: fix power distribution issue for
> Polaris10 XT
>
>
>
> On 2016年07月12日 10:39, Ken Wang wrote:
> > Change-Id: I8676174acd3273dd8dccc72ecc1d54e9626bdb71
> > Signed-off-by: Ken Wang <Qingqing.Wang at amd.com>
> > ---
> > drivers/gpu/drm/amd/amdgpu/atombios_i2c.c | 15 +++++++++++++++
> > drivers/gpu/drm/amd/amdgpu/atombios_i2c.h | 1 +
> > drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 5 +++++
> > 3 files changed, 21 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/atombios_i2c.c
> b/drivers/gpu/drm/amd/amdgpu/atombios_i2c.c
> > index 13cdb01..145d752 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/atombios_i2c.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/atombios_i2c.c
> > @@ -156,3 +156,18 @@ u32 amdgpu_atombios_i2c_func(struct
> i2c_adapter *adap)
> > return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL;
> > }
> >
> > +void amdgpu_atombios_i2c_channel_trans(struct amdgpu_device* adev,
> u8 offset, u8 data)
> > +{
> > + PROCESS_I2C_CHANNEL_TRANSACTION_PS_ALLOCATION args;
> > + int index = GetIndexIntoMasterTable(COMMAND,
> ProcessI2cChannelTransaction);
> > +
> > + args.ucRegIndex = offset;
> > + args.lpI2CDataOut = data;
> > + args.ucFlag = 1;
> > + args.ucI2CSpeed = TARGET_HW_I2C_CLOCK;
> > + args.ucTransBytes = 1;
> > + args.ucSlaveAddr = 0x10;
>
> Is this Polaris10 XT specific? if yes, seems the func name need to be
> specific as well.
If that's the case, then just make it a generic i2c write function like I suggested in my other email and pass in the slave_addr and i2c_line. Then it can be used anywhere as a simple helper.
Alex
>
> Regards,
> David Zhou
>
> > + args.ucLineNumber = 0x96;
> > +
> > + amdgpu_atom_execute_table(adev->mode_info.atom_context,
> index, (uint32_t *)&args);
> > +}
> > diff --git a/drivers/gpu/drm/amd/amdgpu/atombios_i2c.h
> b/drivers/gpu/drm/amd/amdgpu/atombios_i2c.h
> > index d6128d9d..d31344f 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/atombios_i2c.h
> > +++ b/drivers/gpu/drm/amd/amdgpu/atombios_i2c.h
> > @@ -27,5 +27,6 @@
> > int amdgpu_atombios_i2c_xfer(struct i2c_adapter *i2c_adap,
> > struct i2c_msg *msgs, int num);
> > u32 amdgpu_atombios_i2c_func(struct i2c_adapter *adap);
> > +void amdgpu_atombios_i2c_channel_trans(struct amdgpu_device* adev,
> u8 offset, u8 data);
> >
> > #endif
> > diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> > index dbef1ed..362eab7 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> > @@ -28,6 +28,7 @@
> > #include "vid.h"
> > #include "amdgpu_ucode.h"
> > #include "amdgpu_atombios.h"
> > +#include "atombios_i2c.h"
> > #include "clearstate_vi.h"
> >
> > #include "gmc/gmc_8_2_d.h"
> > @@ -693,6 +694,10 @@ static void gfx_v8_0_init_golden_registers(struct
> amdgpu_device *adev)
> > amdgpu_program_register_sequence(adev,
> >
> polaris10_golden_common_all,
> > (const
> u32)ARRAY_SIZE(polaris10_golden_common_all));
> > + if (adev->pdev->revision == 0xc7) {
> > + amdgpu_atombios_i2c_channel_trans(adev, 0x1E,
> 0xDD);
> > + amdgpu_atombios_i2c_channel_trans(adev, 0x1F,
> 0xD0);
> > + }
> > break;
> > case CHIP_CARRIZO:
> > amdgpu_program_register_sequence(adev,
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
More information about the amd-gfx
mailing list