Re: [PATCH] drm/msm: update adreno firmware path in MODULE_FIRMWARE

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

 



On Thu, Nov 23, 2017 at 3:54 PM, Rob Clark <robdclark@xxxxxxxxx> wrote:
> On Thu, Nov 23, 2017 at 5:32 AM, Nicolas Dechesne
> <nicolas.dechesne@xxxxxxxxxx> wrote:
>> The preferred location for Adreno firmware files is now in qcom/ subfolder,
>> especially now that we are adding some of them in linux-firmware.
>>
>> Reported-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx>
>> Signed-off-by: Nicolas Dechesne <nicolas.dechesne@xxxxxxxxxx>
>
> Thanks, I was wondering if we should perhaps list both old and new
> paths?  I'm not sure, maybe we don't need to care about dracut or
> initrd generation for the legacy case (since mostly there you are
> using fastboot).

I've been going back and forth on that too. and i decided to ignore
the legacy paths... alternatively we could use the legacy paths for
a3xx and the new path for a5xx since we have links for a3xx files...
but i thought it was too much noise..

>
> Also, I noticed we are missing a few a5xx fw files, but perhaps that
> should be fixed with a separate patch.

as you want. I can include them if you prefer and resend.

>
> Either way,
>
> Reviewed-by: Rob Clark <robdclark@xxxxxxxxx>
>
>> ---
>>  drivers/gpu/drm/msm/adreno/adreno_device.c | 16 ++++++++--------
>>  1 file changed, 8 insertions(+), 8 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/msm/adreno/adreno_device.c b/drivers/gpu/drm/msm/adreno/adreno_device.c
>> index 05022ea2a007..3c1d23b9ddc3 100644
>> --- a/drivers/gpu/drm/msm/adreno/adreno_device.c
>> +++ b/drivers/gpu/drm/msm/adreno/adreno_device.c
>> @@ -90,14 +90,14 @@ static const struct adreno_info gpulist[] = {
>>         },
>>  };
>>
>> -MODULE_FIRMWARE("a300_pm4.fw");
>> -MODULE_FIRMWARE("a300_pfp.fw");
>> -MODULE_FIRMWARE("a330_pm4.fw");
>> -MODULE_FIRMWARE("a330_pfp.fw");
>> -MODULE_FIRMWARE("a420_pm4.fw");
>> -MODULE_FIRMWARE("a420_pfp.fw");
>> -MODULE_FIRMWARE("a530_fm4.fw");
>> -MODULE_FIRMWARE("a530_pfp.fw");
>> +MODULE_FIRMWARE("qcom/a300_pm4.fw");
>> +MODULE_FIRMWARE("qcom/a300_pfp.fw");
>> +MODULE_FIRMWARE("qcom/a330_pm4.fw");
>> +MODULE_FIRMWARE("qcom/a330_pfp.fw");
>> +MODULE_FIRMWARE("qcom/a420_pm4.fw");
>> +MODULE_FIRMWARE("qcom/a420_pfp.fw");
>> +MODULE_FIRMWARE("qcom/a530_fm4.fw");
>> +MODULE_FIRMWARE("qcom/a530_pfp.fw");
>>
>>  static inline bool _rev_match(uint8_t entry, uint8_t id)
>>  {
>> --
>> 2.15.0
>>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux