On 11/6/2014 2:13 PM, Rob Clark wrote:
> On Fri, Oct 31, 2014 at 11:08 AM, Ganesan, Aravind
> <aravindg@xxxxxxxxxxxxxx> wrote:
>> Added a4xx GPU support.
>>
>> Signed-off-by: Aravind Ganesan <aravindg@xxxxxxxxxxxxxx>
>> ---
>> Resend the patch-set with the same thread-id
>> Resend in patch-set format and with dri-devel@xxxxxxxxxxxxxxxxxxxxx on
>> the CC.
>> drivers/gpu/drm/msm/Makefile | 1 +
>> drivers/gpu/drm/msm/adreno/a4xx_gpu.c | 620
>> +++++++++++++++++++++++++++++
>> drivers/gpu/drm/msm/adreno/a4xx_gpu.h | 34 ++
>> drivers/gpu/drm/msm/adreno/adreno_device.c | 13 +
>> drivers/gpu/drm/msm/adreno/adreno_gpu.c | 2 +-
>> drivers/gpu/drm/msm/adreno/adreno_gpu.h | 14 +-
>> 6 files changed, 681 insertions(+), 3 deletions(-)
>> create mode 100644 drivers/gpu/drm/msm/adreno/a4xx_gpu.c
>> create mode 100644 drivers/gpu/drm/msm/adreno/a4xx_gpu.h
>>
>> diff --git a/drivers/gpu/drm/msm/Makefile b/drivers/gpu/drm/msm/Makefile
>> index 20b7eda..21020f9 100644
>> --- a/drivers/gpu/drm/msm/Makefile
>> +++ b/drivers/gpu/drm/msm/Makefile
>> @@ -7,6 +7,7 @@ msm-y := \
>> adreno/adreno_device.o \
>> adreno/adreno_gpu.o \
>> adreno/a3xx_gpu.o \
>> + adreno/a4xx_gpu.o \
>> hdmi/hdmi.o \
>> hdmi/hdmi_audio.o \
>> hdmi/hdmi_bridge.o \
>> diff --git a/drivers/gpu/drm/msm/adreno/a4xx_gpu.c
>> b/drivers/gpu/drm/msm/adreno/a4xx_gpu.c
>> new file mode 100644
>> index 0000000..50cbaf4
>> --- /dev/null
>> +++ b/drivers/gpu/drm/msm/adreno/a4xx_gpu.c
> [snip]
>> +#ifdef CONFIG_DEBUG_FS
>> +static void a4xx_show(struct msm_gpu *gpu, struct seq_file *m)
>> +{
>> + struct drm_device *dev = gpu->dev;
>> +
>> + mutex_lock(&dev->struct_mutex);
>
> so, probably something that was missed in the rebase, but struct_mutex
> is acquired/dropped in show_locked() (in msm_drv.c) now.. so you need
> to drop the mutex_lock/_unlock() here.
>
> BR,
> -R
>
Agreed.
>> + gpu->funcs->pm_resume(gpu);
>> +
>> + seq_printf(m, "status: %08x\n",
>> + gpu_read(gpu, REG_A4XX_RBBM_STATUS));
>> + gpu->funcs->pm_suspend(gpu);
>> +
>> + adreno_show(gpu, m);
>> +
>> + mutex_unlock(&dev->struct_mutex);
>> +}
>> +#endif
>> +
> --
> To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
