mbox series

qcom: add firmware file for Venus on SDM845

Message ID 1527246209-26685-2-git-send-email-vgarodia@codeaurora.org
State New
Headers show
Series qcom: add firmware file for Venus on SDM845 | expand

Pull-request

https://github.com/vgarodia/linux-firmware master

Message

Vikash Garodia May 25, 2018, 11:03 a.m. UTC 
This pull request adds firmware files for Venus h/w codec found on the Qualcomm SDM845 chipset.

The following changes since commit 2a9b2cf50fb32e36e4fc1586c2f6f1421913b553:

  Merge branch 'for-upstreaming-v1.7.2' of https://github.com/felix-cavium/linux-firmware (2018-05-18 08:35:22 -0400)

are available in the git repository at:


  https://github.com/vgarodia/linux-firmware master

for you to fetch changes up to d6088b9c9d7f49d3c6c43681190889eca0abdcce:

  qcom: add venus firmware files for v5.2 (2018-05-25 15:16:43 +0530)

----------------------------------------------------------------
Vikash Garodia (1):
      qcom: add venus firmware files for v5.2

 WHENCE                   |   9 +++++++++
 qcom/venus-5.2/venus.b00 | Bin 0 -> 212 bytes
 qcom/venus-5.2/venus.b01 | Bin 0 -> 6600 bytes
 qcom/venus-5.2/venus.b02 | Bin 0 -> 819552 bytes
 qcom/venus-5.2/venus.b03 | Bin 0 -> 33536 bytes
 qcom/venus-5.2/venus.b04 |   1 +
 qcom/venus-5.2/venus.mbn | Bin 0 -> 865408 bytes
 qcom/venus-5.2/venus.mdt | Bin 0 -> 6812 bytes
 8 files changed, 10 insertions(+)
 create mode 100644 qcom/venus-5.2/venus.b00
 create mode 100644 qcom/venus-5.2/venus.b01
 create mode 100644 qcom/venus-5.2/venus.b02
 create mode 100644 qcom/venus-5.2/venus.b03
 create mode 100644 qcom/venus-5.2/venus.b04
 create mode 100644 qcom/venus-5.2/venus.mbn
 create mode 100644 qcom/venus-5.2/venus.mdt

Comments

Josh Boyer May 25, 2018, 12:04 p.m. UTC | #1 
On Fri, May 25, 2018 at 7:03 AM Vikash Garodia <vgarodia@codeaurora.org>
wrote:

> This pull request adds firmware files for Venus h/w codec found on the
Qualcomm SDM845 chipset.

> The following changes since commit
2a9b2cf50fb32e36e4fc1586c2f6f1421913b553:

>    Merge branch 'for-upstreaming-v1.7.2' of
https://github.com/felix-cavium/linux-firmware (2018-05-18 08:35:22 -0400)

> are available in the git repository at:


>    https://github.com/vgarodia/linux-firmware master

> for you to fetch changes up to d6088b9c9d7f49d3c6c43681190889eca0abdcce:

>    qcom: add venus firmware files for v5.2 (2018-05-25 15:16:43 +0530)

> ----------------------------------------------------------------
> Vikash Garodia (1):
>        qcom: add venus firmware files for v5.2

>   WHENCE                   |   9 +++++++++
>   qcom/venus-5.2/venus.b00 | Bin 0 -> 212 bytes
>   qcom/venus-5.2/venus.b01 | Bin 0 -> 6600 bytes
>   qcom/venus-5.2/venus.b02 | Bin 0 -> 819552 bytes
>   qcom/venus-5.2/venus.b03 | Bin 0 -> 33536 bytes
>   qcom/venus-5.2/venus.b04 |   1 +
>   qcom/venus-5.2/venus.mbn | Bin 0 -> 865408 bytes
>   qcom/venus-5.2/venus.mdt | Bin 0 -> 6812 bytes
>   8 files changed, 10 insertions(+)
>   create mode 100644 qcom/venus-5.2/venus.b00
>   create mode 100644 qcom/venus-5.2/venus.b01
>   create mode 100644 qcom/venus-5.2/venus.b02
>   create mode 100644 qcom/venus-5.2/venus.b03
>   create mode 100644 qcom/venus-5.2/venus.b04
>   create mode 100644 qcom/venus-5.2/venus.mbn
>   create mode 100644 qcom/venus-5.2/venus.mdt

The venus.mbn file isn't mentioned in WHENCE:

[jwboyer@vader linux-firmware]$ ./check_whence.py
E: qcom/venus-5.2/venus.mbn not listed in WHENCE
[jwboyer@vader linux-firmware]$

Can you fix that up and let me know when to re-pull?

josh
Vikash Garodia May 26, 2018, 8:18 a.m. UTC | #2 
Hi Josh,

On 2018-05-25 17:34, Josh Boyer wrote:
> On Fri, May 25, 2018 at 7:03 AM Vikash Garodia 
> <vgarodia@codeaurora.org>
> wrote:
> 
>> This pull request adds firmware files for Venus h/w codec found on the
> Qualcomm SDM845 chipset.
> 
>> The following changes since commit
> 2a9b2cf50fb32e36e4fc1586c2f6f1421913b553:
> 
>>    Merge branch 'for-upstreaming-v1.7.2' of
> https://github.com/felix-cavium/linux-firmware (2018-05-18 08:35:22 
> -0400)
> 
>> are available in the git repository at:
> 
> 
>>    https://github.com/vgarodia/linux-firmware master
> 
>> for you to fetch changes up to 
>> d6088b9c9d7f49d3c6c43681190889eca0abdcce:
> 
>>    qcom: add venus firmware files for v5.2 (2018-05-25 15:16:43 +0530)
> 
>> ----------------------------------------------------------------
>> Vikash Garodia (1):
>>        qcom: add venus firmware files for v5.2
> 
>>   WHENCE                   |   9 +++++++++
>>   qcom/venus-5.2/venus.b00 | Bin 0 -> 212 bytes
>>   qcom/venus-5.2/venus.b01 | Bin 0 -> 6600 bytes
>>   qcom/venus-5.2/venus.b02 | Bin 0 -> 819552 bytes
>>   qcom/venus-5.2/venus.b03 | Bin 0 -> 33536 bytes
>>   qcom/venus-5.2/venus.b04 |   1 +
>>   qcom/venus-5.2/venus.mbn | Bin 0 -> 865408 bytes
>>   qcom/venus-5.2/venus.mdt | Bin 0 -> 6812 bytes
>>   8 files changed, 10 insertions(+)
>>   create mode 100644 qcom/venus-5.2/venus.b00
>>   create mode 100644 qcom/venus-5.2/venus.b01
>>   create mode 100644 qcom/venus-5.2/venus.b02
>>   create mode 100644 qcom/venus-5.2/venus.b03
>>   create mode 100644 qcom/venus-5.2/venus.b04
>>   create mode 100644 qcom/venus-5.2/venus.mbn
>>   create mode 100644 qcom/venus-5.2/venus.mdt
> 
> The venus.mbn file isn't mentioned in WHENCE:
> 
> [jwboyer@vader linux-firmware]$ ./check_whence.py
> E: qcom/venus-5.2/venus.mbn not listed in WHENCE
> [jwboyer@vader linux-firmware]$
> 
> Can you fix that up and let me know when to re-pull?
I have fixed the error and is ready to be re-pulled from the same git 
repository.
I have noted the process to run check_whence.py before uploading the 
firmwares.

Do i need to resend the pull request as the end commit is now changed to
7602644358157e4b89960472b6d59ffcc040ca52 ?

-Vikash
Josh Boyer June 4, 2018, 12:52 p.m. UTC | #3 
On Sat, May 26, 2018 at 4:18 AM Vikash Garodia <vgarodia@codeaurora.org> wrote:
>
> Hi Josh,
>
> On 2018-05-25 17:34, Josh Boyer wrote:
> > On Fri, May 25, 2018 at 7:03 AM Vikash Garodia
> > <vgarodia@codeaurora.org>
> > wrote:
> >
> >> This pull request adds firmware files for Venus h/w codec found on the
> > Qualcomm SDM845 chipset.
> >
> >> The following changes since commit
> > 2a9b2cf50fb32e36e4fc1586c2f6f1421913b553:
> >
> >>    Merge branch 'for-upstreaming-v1.7.2' of
> > https://github.com/felix-cavium/linux-firmware (2018-05-18 08:35:22
> > -0400)
> >
> >> are available in the git repository at:
> >
> >
> >>    https://github.com/vgarodia/linux-firmware master
> >
> >> for you to fetch changes up to
> >> d6088b9c9d7f49d3c6c43681190889eca0abdcce:
> >
> >>    qcom: add venus firmware files for v5.2 (2018-05-25 15:16:43 +0530)
> >
> >> ----------------------------------------------------------------
> >> Vikash Garodia (1):
> >>        qcom: add venus firmware files for v5.2
> >
> >>   WHENCE                   |   9 +++++++++
> >>   qcom/venus-5.2/venus.b00 | Bin 0 -> 212 bytes
> >>   qcom/venus-5.2/venus.b01 | Bin 0 -> 6600 bytes
> >>   qcom/venus-5.2/venus.b02 | Bin 0 -> 819552 bytes
> >>   qcom/venus-5.2/venus.b03 | Bin 0 -> 33536 bytes
> >>   qcom/venus-5.2/venus.b04 |   1 +
> >>   qcom/venus-5.2/venus.mbn | Bin 0 -> 865408 bytes
> >>   qcom/venus-5.2/venus.mdt | Bin 0 -> 6812 bytes
> >>   8 files changed, 10 insertions(+)
> >>   create mode 100644 qcom/venus-5.2/venus.b00
> >>   create mode 100644 qcom/venus-5.2/venus.b01
> >>   create mode 100644 qcom/venus-5.2/venus.b02
> >>   create mode 100644 qcom/venus-5.2/venus.b03
> >>   create mode 100644 qcom/venus-5.2/venus.b04
> >>   create mode 100644 qcom/venus-5.2/venus.mbn
> >>   create mode 100644 qcom/venus-5.2/venus.mdt
> >
> > The venus.mbn file isn't mentioned in WHENCE:
> >
> > [jwboyer@vader linux-firmware]$ ./check_whence.py
> > E: qcom/venus-5.2/venus.mbn not listed in WHENCE
> > [jwboyer@vader linux-firmware]$
> >
> > Can you fix that up and let me know when to re-pull?
> I have fixed the error and is ready to be re-pulled from the same git
> repository.
> I have noted the process to run check_whence.py before uploading the
> firmwares.
>
> Do i need to resend the pull request as the end commit is now changed to
> 7602644358157e4b89960472b6d59ffcc040ca52 ?

Nope.  Pulled and pushed out now.  Thanks.

josh