Coo, pushed to staging.
Tom ________________________________ From: Deucher, Alexander Sent: Wednesday, January 18, 2017 13:47 To: StDenis, Tom; Alex Deucher Cc: amd-gfx list Subject: RE: [PATCH] drm/amd/amdgpu: Add PCI info to gca_config debugfs Whoops, sorry, I glanced quickly and thought you were grabbing the vendor and device ids, not the revision. Carry on :) From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf Of StDenis, Tom Sent: Wednesday, January 18, 2017 1:45 PM To: Alex Deucher Cc: amd-gfx list Subject: Re: [PATCH] drm/amd/amdgpu: Add PCI info to gca_config debugfs I am grabbing revision. Unless there's some other PCI revision value I'm missing :-) Tom ________________________________ From: Alex Deucher <alexdeuc...@gmail.com<mailto:alexdeuc...@gmail.com>> Sent: Wednesday, January 18, 2017 13:40 To: Tom St Denis Cc: amd-gfx list; StDenis, Tom Subject: Re: [PATCH] drm/amd/amdgpu: Add PCI info to gca_config debugfs On Wed, Jan 18, 2017 at 1:25 PM, Tom St Denis <tstdeni...@gmail.com<mailto:tstdeni...@gmail.com>> wrote: > So we can determine which device the entry is before connecting > a display. > > Signed-off-by: Tom St Denis <tom.stde...@amd.com<mailto:tom.stde...@amd.com>> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > index 28681286d57c..8640f9216d93 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > @@ -2967,7 +2967,7 @@ static ssize_t amdgpu_debugfs_gca_config_read(struct > file *f, char __user *buf, > return -ENOMEM; > > /* version, increment each time something is added */ > - config[no_regs++] = 2; > + config[no_regs++] = 3; > config[no_regs++] = adev->gfx.config.max_shader_engines; > config[no_regs++] = adev->gfx.config.max_tile_pipes; > config[no_regs++] = adev->gfx.config.max_cu_per_sh; > @@ -3001,6 +3001,12 @@ static ssize_t amdgpu_debugfs_gca_config_read(struct > file *f, char __user *buf, > config[no_regs++] = adev->family; > config[no_regs++] = adev->external_rev_id; > > + /* rev==3 */ > + config[no_regs++] = adev->pdev->device; > + config[no_regs++] = adev->pdev->revision; > + config[no_regs++] = adev->pdev->subsystem_device; > + config[no_regs++] = adev->pdev->subsystem_vendor; For completeness, please add pdev->revision as well. With that: Reviewed-by: Alex Deucher <alexander.deuc...@amd.com<mailto:alexander.deuc...@amd.com>> > + > while (size && (*pos < no_regs * 4)) { > uint32_t value; > > -- > 2.11.0 > > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> > https://lists.freedesktop.org/mailman/listinfo/amd-gfx amd-gfx Info Page - lists.freedesktop.org<https://lists.freedesktop.org/mailman/listinfo/amd-gfx> lists.freedesktop.org To see the collection of prior postings to the list, visit the amd-gfx Archives. Using amd-gfx: To post a message to all the list members, send email ...
_______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
