On Sat, May 26, 2018 at 4:18 AM Vikash Garodia [off-list ref] wrote:

Hi Josh,

On 2018-05-25 17:34, Josh Boyer wrote:

quoted

On Fri, May 25, 2018 at 7:03 AM Vikash Garodia
[off-list ref]
wrote:

quoted

This pull request adds firmware files for Venus h/w codec found on the

Qualcomm SDM845 chipset.

quoted

The following changes since commit

2a9b2cf50fb32e36e4fc1586c2f6f1421913b553:

quoted

   Merge branch 'for-upstreaming-v1.7.2' of

https://github.com/felix-cavium/linux-firmware (2018-05-18 08:35:22
-0400)

quoted

are available in the git repository at:

quoted

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

quoted

for you to fetch changes up to
d6088b9c9d7f49d3c6c43681190889eca0abdcce:

quoted

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

quoted

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

quoted

  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 at vader linux-firmware]$ ./check_whence.py
E: qcom/venus-5.2/venus.mbn not listed in WHENCE
[jwboyer at 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