<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=gb2312"> <style type="text/css" style="display:none;"></style> </head> <body dir="ltr"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;"> ok, I get it, will change soon. <div style="color: rgb(0, 0, 0);"> <div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="x_divRplyFwdMsg" dir="ltr">发件人: Deucher, Alexander 发送时间: 2016年7月12日 10:44 收件人: Wang, Qingqing; amd-gfx@lists.freedesktop.org 抄送: Wang, Qingqing 主题: RE: [PATCH v2] drm/amdgpu: fix power distribution issue for Polaris10 XT <div> </div> </div> </div> <div class="PlainText">> -----Original Message----- > From: amd-gfx [<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">mailto:amd-gfx-bounces@lists.freedesktop.org</a>] On Behalf > Of Ken Wang > Sent: Monday, July 11, 2016 10:39 PM > To: amd-gfx@lists.freedesktop.org > Cc: Wang, Qingqing > Subject: [PATCH v2] drm/amdgpu: fix power distribution issue for Polaris10 > XT > > Change-Id: I8676174acd3273dd8dccc72ecc1d54e9626bdb71 > Signed-off-by: Ken Wang <Qingqing.Wang@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; > + 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); channel_trans isn't much better than the previous name. How about something like amdgpu_atombios_i2c_set_power_dist_reg() or something like that? Alex > > #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, > -- > 2.7.4 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> </div> </div> </div> </body> </html>