* Re: Request to update venus-5.10 FW files
@ 2021-04-21 11:18 ` Josh Boyer
0 siblings, 0 replies; 5+ messages in thread
From: Josh Boyer @ 2021-04-21 11:18 UTC (permalink / raw)
To: smagar
Cc: Linux Firmware, linux-arm-kernel, Linux Media Mailing List,
Mauro Carvalho Chehab, Stanimir Varbanov, linux-media-owner,
vgarodia, dikshita, adhudase, sampnimm, mansur
On Wed, Apr 21, 2021 at 2:59 AM <smagar@codeaurora.org> wrote:
>
> Hello Team,
>
> Please include updated firmware bins for venus-5.10.
>
> Snapshot of pull request is as below, let me know if anything is
> missing.
>
>
>
> The following changes since commit
> 5c893c65b862906a627cea238ae8c2d099027dee:
>
> Updated Venus firmware files for v5.10 (2021-04-20 11:48:28 +0530)
>
> are available in the git repository at:
>
> https://github.com/suraj714/linux-firmware-venus.git master
>
> for you to fetch changes up to 2565c9d2576ce1275206483e106d6fb627ace042:
>
> qcom :venus_v5.10: updated venus firmware files for v5.10 (2021-04-21
> 12:04:59 +0530)
>
> ----------------------------------------------------------------
> smagar (1):
> qcom :venus_v5.10: updated venus firmware files for v5.10
>
> qcom/sc7280/vpu20_1v.b00 | Bin 0 -> 692 bytes
> qcom/sc7280/vpu20_1v.b01 | Bin 0 -> 7376 bytes
> qcom/sc7280/vpu20_1v.b02 | Bin 0 -> 300 bytes
> qcom/sc7280/vpu20_1v.b03 | Bin 0 -> 20 bytes
> qcom/sc7280/vpu20_1v.b04 | Bin 0 -> 20 bytes
> qcom/sc7280/vpu20_1v.b05 | Bin 0 -> 20 bytes
> qcom/sc7280/vpu20_1v.b06 | Bin 0 -> 20 bytes
> qcom/sc7280/vpu20_1v.b07 | Bin 0 -> 24 bytes
> qcom/sc7280/vpu20_1v.b08 | Bin 0 -> 16 bytes
> qcom/sc7280/vpu20_1v.b09 | Bin 0 -> 939184 bytes
> qcom/sc7280/vpu20_1v.b10 | Bin 0 -> 42976 bytes
> qcom/sc7280/vpu20_1v.b11 | 0
> qcom/sc7280/vpu20_1v.b12 | 0
> qcom/sc7280/vpu20_1v.b13 | 0
> qcom/sc7280/vpu20_1v.b14 | 0
> qcom/sc7280/vpu20_1v.b15 | 0
> qcom/sc7280/vpu20_1v.b16 | 0
> qcom/sc7280/vpu20_1v.b17 | 0
> qcom/sc7280/vpu20_1v.b18 | 0
> qcom/sc7280/vpu20_1v.b19 | 1 +
> qcom/sc7280/vpu20_1v.mbn | Bin 0 -> 2031188 bytes
> qcom/sc7280/vpu20_1v.mdt | Bin 0 -> 8068 bytes
> qcom/sc7280/vpu20_1v_unsigned.mbn | Bin 0 -> 2031188 bytes
This looks weird. You're adding 23 new files, none of which are
mentioned in WHENCE. Also, why do you have a bunch of 0 byte files?
Is this correct?
josh
> 23 files changed, 1 insertion(+)
> create mode 100644 qcom/sc7280/vpu20_1v.b00
> create mode 100644 qcom/sc7280/vpu20_1v.b01
> create mode 100644 qcom/sc7280/vpu20_1v.b02
> create mode 100644 qcom/sc7280/vpu20_1v.b03
> create mode 100644 qcom/sc7280/vpu20_1v.b04
> create mode 100644 qcom/sc7280/vpu20_1v.b05
> create mode 100644 qcom/sc7280/vpu20_1v.b06
> create mode 100644 qcom/sc7280/vpu20_1v.b07
> create mode 100644 qcom/sc7280/vpu20_1v.b08
> create mode 100644 qcom/sc7280/vpu20_1v.b09
> create mode 100644 qcom/sc7280/vpu20_1v.b10
> create mode 100644 qcom/sc7280/vpu20_1v.b11
> create mode 100644 qcom/sc7280/vpu20_1v.b12
> create mode 100644 qcom/sc7280/vpu20_1v.b13
> create mode 100644 qcom/sc7280/vpu20_1v.b14
> create mode 100644 qcom/sc7280/vpu20_1v.b15
> create mode 100644 qcom/sc7280/vpu20_1v.b16
> create mode 100644 qcom/sc7280/vpu20_1v.b17
> create mode 100644 qcom/sc7280/vpu20_1v.b18
> create mode 100644 qcom/sc7280/vpu20_1v.b19
> create mode 100644 qcom/sc7280/vpu20_1v.mbn
> create mode 100644 qcom/sc7280/vpu20_1v.mdt
> create mode 100644 qcom/sc7280/vpu20_1v_unsigned.mbn
>
> Regards,
> Suraj Magar
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Request to update venus-5.10 FW files
2021-04-21 11:18 ` Josh Boyer
(?)
@ 2021-04-23 13:27 ` vgarodia
-1 siblings, 0 replies; 5+ messages in thread
From: vgarodia @ 2021-04-23 13:27 UTC (permalink / raw)
To: Josh Boyer
Cc: smagar, Linux Firmware, linux-arm-kernel,
Linux Media Mailing List, Mauro Carvalho Chehab,
Stanimir Varbanov, linux-media-owner, dikshita, adhudase,
sampnimm, mansur
On 2021-04-21 16:48, Josh Boyer wrote:
> On Wed, Apr 21, 2021 at 2:59 AM <smagar@codeaurora.org> wrote:
>>
>> Hello Team,
>>
>> Please include updated firmware bins for venus-5.10.
>>
>> Snapshot of pull request is as below, let me know if anything is
>> missing.
>>
>>
>>
>> The following changes since commit
>> 5c893c65b862906a627cea238ae8c2d099027dee:
>>
>> Updated Venus firmware files for v5.10 (2021-04-20 11:48:28 +0530)
>>
>> are available in the git repository at:
>>
>> https://github.com/suraj714/linux-firmware-venus.git master
>>
>> for you to fetch changes up to
>> 2565c9d2576ce1275206483e106d6fb627ace042:
>>
>> qcom :venus_v5.10: updated venus firmware files for v5.10
>> (2021-04-21
>> 12:04:59 +0530)
>>
>> ----------------------------------------------------------------
>> smagar (1):
>> qcom :venus_v5.10: updated venus firmware files for v5.10
This should be changed to "qcom: Add venus firmware files for VPU-2.0"
>>
>> qcom/sc7280/vpu20_1v.b00 | Bin 0 -> 692 bytes
>> qcom/sc7280/vpu20_1v.b01 | Bin 0 -> 7376 bytes
>> qcom/sc7280/vpu20_1v.b02 | Bin 0 -> 300 bytes
>> qcom/sc7280/vpu20_1v.b03 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b04 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b05 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b06 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b07 | Bin 0 -> 24 bytes
>> qcom/sc7280/vpu20_1v.b08 | Bin 0 -> 16 bytes
>> qcom/sc7280/vpu20_1v.b09 | Bin 0 -> 939184 bytes
>> qcom/sc7280/vpu20_1v.b10 | Bin 0 -> 42976 bytes
>> qcom/sc7280/vpu20_1v.b11 | 0
>> qcom/sc7280/vpu20_1v.b12 | 0
>> qcom/sc7280/vpu20_1v.b13 | 0
>> qcom/sc7280/vpu20_1v.b14 | 0
>> qcom/sc7280/vpu20_1v.b15 | 0
>> qcom/sc7280/vpu20_1v.b16 | 0
>> qcom/sc7280/vpu20_1v.b17 | 0
>> qcom/sc7280/vpu20_1v.b18 | 0
>> qcom/sc7280/vpu20_1v.b19 | 1 +
>> qcom/sc7280/vpu20_1v.mbn | Bin 0 -> 2031188 bytes
>> qcom/sc7280/vpu20_1v.mdt | Bin 0 -> 8068 bytes
>> qcom/sc7280/vpu20_1v_unsigned.mbn | Bin 0 -> 2031188 bytes
>
> This looks weird. You're adding 23 new files, none of which are
> mentioned in WHENCE. Also, why do you have a bunch of 0 byte files?
>
> Is this correct?
0 byte files are not needed. Please remove.
> josh
>
>> 23 files changed, 1 insertion(+)
>> create mode 100644 qcom/sc7280/vpu20_1v.b00
>> create mode 100644 qcom/sc7280/vpu20_1v.b01
>> create mode 100644 qcom/sc7280/vpu20_1v.b02
>> create mode 100644 qcom/sc7280/vpu20_1v.b03
>> create mode 100644 qcom/sc7280/vpu20_1v.b04
>> create mode 100644 qcom/sc7280/vpu20_1v.b05
>> create mode 100644 qcom/sc7280/vpu20_1v.b06
>> create mode 100644 qcom/sc7280/vpu20_1v.b07
>> create mode 100644 qcom/sc7280/vpu20_1v.b08
>> create mode 100644 qcom/sc7280/vpu20_1v.b09
>> create mode 100644 qcom/sc7280/vpu20_1v.b10
>> create mode 100644 qcom/sc7280/vpu20_1v.b11
>> create mode 100644 qcom/sc7280/vpu20_1v.b12
>> create mode 100644 qcom/sc7280/vpu20_1v.b13
>> create mode 100644 qcom/sc7280/vpu20_1v.b14
>> create mode 100644 qcom/sc7280/vpu20_1v.b15
>> create mode 100644 qcom/sc7280/vpu20_1v.b16
>> create mode 100644 qcom/sc7280/vpu20_1v.b17
>> create mode 100644 qcom/sc7280/vpu20_1v.b18
>> create mode 100644 qcom/sc7280/vpu20_1v.b19
>> create mode 100644 qcom/sc7280/vpu20_1v.mbn
>> create mode 100644 qcom/sc7280/vpu20_1v.mdt
>> create mode 100644 qcom/sc7280/vpu20_1v_unsigned.mbn
>>
>> Regards,
>> Suraj Magar
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Request to update venus-5.10 FW files
2021-04-21 11:18 ` Josh Boyer
(?)
(?)
@ 2021-04-28 12:33 ` smagar
-1 siblings, 0 replies; 5+ messages in thread
From: smagar @ 2021-04-28 12:33 UTC (permalink / raw)
To: Josh Boyer
Cc: Linux Firmware, linux-arm-kernel, Linux Media Mailing List,
Mauro Carvalho Chehab, Stanimir Varbanov, linux-media-owner,
vgarodia, dikshita, adhudase, sampnimm, mansur
On 2021-04-21 16:48, Josh Boyer wrote:
> On Wed, Apr 21, 2021 at 2:59 AM <smagar@codeaurora.org> wrote:
>>
>> Hello Team,
>>
>> Please include updated firmware bins for venus-5.10.
>>
>> Snapshot of pull request is as below, let me know if anything is
>> missing.
>>
>>
>>
>> The following changes since commit
>> 5c893c65b862906a627cea238ae8c2d099027dee:
>>
>> Updated Venus firmware files for v5.10 (2021-04-20 11:48:28 +0530)
>>
>> are available in the git repository at:
>>
>> https://github.com/suraj714/linux-firmware-venus.git master
>>
>> for you to fetch changes up to
>> 2565c9d2576ce1275206483e106d6fb627ace042:
>>
>> qcom :venus_v5.10: updated venus firmware files for v5.10
>> (2021-04-21
>> 12:04:59 +0530)
>>
>> ----------------------------------------------------------------
>> smagar (1):
>> qcom :venus_v5.10: updated venus firmware files for v5.10
>>
>> qcom/sc7280/vpu20_1v.b00 | Bin 0 -> 692 bytes
>> qcom/sc7280/vpu20_1v.b01 | Bin 0 -> 7376 bytes
>> qcom/sc7280/vpu20_1v.b02 | Bin 0 -> 300 bytes
>> qcom/sc7280/vpu20_1v.b03 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b04 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b05 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b06 | Bin 0 -> 20 bytes
>> qcom/sc7280/vpu20_1v.b07 | Bin 0 -> 24 bytes
>> qcom/sc7280/vpu20_1v.b08 | Bin 0 -> 16 bytes
>> qcom/sc7280/vpu20_1v.b09 | Bin 0 -> 939184 bytes
>> qcom/sc7280/vpu20_1v.b10 | Bin 0 -> 42976 bytes
>> qcom/sc7280/vpu20_1v.b11 | 0
>> qcom/sc7280/vpu20_1v.b12 | 0
>> qcom/sc7280/vpu20_1v.b13 | 0
>> qcom/sc7280/vpu20_1v.b14 | 0
>> qcom/sc7280/vpu20_1v.b15 | 0
>> qcom/sc7280/vpu20_1v.b16 | 0
>> qcom/sc7280/vpu20_1v.b17 | 0
>> qcom/sc7280/vpu20_1v.b18 | 0
>> qcom/sc7280/vpu20_1v.b19 | 1 +
>> qcom/sc7280/vpu20_1v.mbn | Bin 0 -> 2031188 bytes
>> qcom/sc7280/vpu20_1v.mdt | Bin 0 -> 8068 bytes
>> qcom/sc7280/vpu20_1v_unsigned.mbn | Bin 0 -> 2031188 bytes
>
> This looks weird. You're adding 23 new files, none of which are
> mentioned in WHENCE. Also, why do you have a bunch of 0 byte files?
>
> Is this correct?
>
> josh
>
>> 23 files changed, 1 insertion(+)
>> create mode 100644 qcom/sc7280/vpu20_1v.b00
>> create mode 100644 qcom/sc7280/vpu20_1v.b01
>> create mode 100644 qcom/sc7280/vpu20_1v.b02
>> create mode 100644 qcom/sc7280/vpu20_1v.b03
>> create mode 100644 qcom/sc7280/vpu20_1v.b04
>> create mode 100644 qcom/sc7280/vpu20_1v.b05
>> create mode 100644 qcom/sc7280/vpu20_1v.b06
>> create mode 100644 qcom/sc7280/vpu20_1v.b07
>> create mode 100644 qcom/sc7280/vpu20_1v.b08
>> create mode 100644 qcom/sc7280/vpu20_1v.b09
>> create mode 100644 qcom/sc7280/vpu20_1v.b10
>> create mode 100644 qcom/sc7280/vpu20_1v.b11
>> create mode 100644 qcom/sc7280/vpu20_1v.b12
>> create mode 100644 qcom/sc7280/vpu20_1v.b13
>> create mode 100644 qcom/sc7280/vpu20_1v.b14
>> create mode 100644 qcom/sc7280/vpu20_1v.b15
>> create mode 100644 qcom/sc7280/vpu20_1v.b16
>> create mode 100644 qcom/sc7280/vpu20_1v.b17
>> create mode 100644 qcom/sc7280/vpu20_1v.b18
>> create mode 100644 qcom/sc7280/vpu20_1v.b19
>> create mode 100644 qcom/sc7280/vpu20_1v.mbn
>> create mode 100644 qcom/sc7280/vpu20_1v.mdt
>> create mode 100644 qcom/sc7280/vpu20_1v_unsigned.mbn
>>
>> Regards,
>> Suraj Magar
Hi Josh,
I have fixed commit and title updated.
Removed unwanted bins and 0 byte files, Also updated WHENCE.
Both commit are combined in single pull request.
Created fresh mail thread for firmware venus-5.4 and vpu-2.0,Please
review once and comment it.
Regards,
Suraj
^ permalink raw reply [flat|nested] 5+ messages in thread