mbox series

[PULL,V2] : qcom: SC7180 and SC7280 venus firmware updates

Message ID CANHAJhG7wCv4TwVCtWa2wqHzaG5ncbtfnVJtnQqqDSbEoPmoHw@mail.gmail.com (mailing list archive)
State Superseded
Headers show
Series [PULL,V2] : qcom: SC7180 and SC7280 venus firmware updates | expand

Pull-request

https://github.com/nathan-google/linux-firmware.git update_sc7180_and_sc7280_firmwares_v2

Message

Nathan Hebert Oct. 20, 2022, 5:07 p.m. UTC 
The following changes since commit 48407ffd7adb9511701547068b1e6f0956bd1c94:

  cnm: update chips&media wave521c firmware. (2022-10-17 10:20:43 -0400)

are available in the Git repository at:

  https://github.com/nathan-google/linux-firmware.git
update_sc7180_and_sc7280_firmwares_v2

for you to fetch changes up to 054271e4164b6b4c069a3aaaeb9bfcde8c850339:

  qcom: update venus firmware files for VPU-2.0 (2022-10-20 09:37:35 -0700)

----------------------------------------------------------------
Nathan Hebert (3):
      qcom: update venus firmware files for v5.4
      qcom: remove split SC7280 venus firmware images
      qcom: update venus firmware files for VPU-2.0

 WHENCE                   |  17 ++---------------
 qcom/venus-5.4/venus.b00 | Bin 212 -> 212 bytes
 qcom/venus-5.4/venus.b01 | Bin 6808 -> 6808 bytes
 qcom/venus-5.4/venus.b02 | Bin 873596 -> 875020 bytes
 qcom/venus-5.4/venus.b03 | Bin 33792 -> 33896 bytes
 qcom/venus-5.4/venus.mbn | Bin 919708 -> 921236 bytes
 qcom/venus-5.4/venus.mdt | Bin 7020 -> 7020 bytes
 qcom/vpu-2.0/venus.b00   | Bin 692 -> 0 bytes
 qcom/vpu-2.0/venus.b01   | Bin 7376 -> 0 bytes
 qcom/vpu-2.0/venus.b02   | Bin 300 -> 0 bytes
 qcom/vpu-2.0/venus.b03   | Bin 20 -> 0 bytes
 qcom/vpu-2.0/venus.b04   | Bin 20 -> 0 bytes
 qcom/vpu-2.0/venus.b05   | Bin 20 -> 0 bytes
 qcom/vpu-2.0/venus.b06   | Bin 20 -> 0 bytes
 qcom/vpu-2.0/venus.b07   | Bin 24 -> 0 bytes
 qcom/vpu-2.0/venus.b08   | Bin 16 -> 0 bytes
 qcom/vpu-2.0/venus.b09   | Bin 939184 -> 0 bytes
 qcom/vpu-2.0/venus.b10   | Bin 42976 -> 0 bytes
 qcom/vpu-2.0/venus.b19   |   1 -
 qcom/vpu-2.0/venus.mbn   | Bin 2031188 -> 2031620 bytes
 qcom/vpu-2.0/venus.mdt   | Bin 8068 -> 0 bytes
 21 files changed, 2 insertions(+), 16 deletions(-)
 delete mode 100644 qcom/vpu-2.0/venus.b00
 delete mode 100644 qcom/vpu-2.0/venus.b01
 delete mode 100644 qcom/vpu-2.0/venus.b02
 delete mode 100644 qcom/vpu-2.0/venus.b03
 delete mode 100644 qcom/vpu-2.0/venus.b04
 delete mode 100644 qcom/vpu-2.0/venus.b05
 delete mode 100644 qcom/vpu-2.0/venus.b06
 delete mode 100644 qcom/vpu-2.0/venus.b07
 delete mode 100644 qcom/vpu-2.0/venus.b08
 delete mode 100644 qcom/vpu-2.0/venus.b09
 delete mode 100644 qcom/vpu-2.0/venus.b10
 delete mode 100644 qcom/vpu-2.0/venus.b19
 delete mode 100644 qcom/vpu-2.0/venus.mdt

Comments

Dmitry Baryshkov Oct. 20, 2022, 6:02 p.m. UTC | #1 
On Thu, 20 Oct 2022 at 20:08, Nathan Hebert <nhebert@chromium.org> wrote:
>
> The following changes since commit 48407ffd7adb9511701547068b1e6f0956bd1c94:
>
>   cnm: update chips&media wave521c firmware. (2022-10-17 10:20:43 -0400)
>
> are available in the Git repository at:
>
>   https://github.com/nathan-google/linux-firmware.git
> update_sc7180_and_sc7280_firmwares_v2
>
> for you to fetch changes up to 054271e4164b6b4c069a3aaaeb9bfcde8c850339:
>
>   qcom: update venus firmware files for VPU-2.0 (2022-10-20 09:37:35 -0700)
>
> ----------------------------------------------------------------
> Nathan Hebert (3):
>       qcom: update venus firmware files for v5.4
>       qcom: remove split SC7280 venus firmware images
>       qcom: update venus firmware files for VPU-2.0
>
>  WHENCE                   |  17 ++---------------
>  qcom/venus-5.4/venus.b00 | Bin 212 -> 212 bytes
>  qcom/venus-5.4/venus.b01 | Bin 6808 -> 6808 bytes
>  qcom/venus-5.4/venus.b02 | Bin 873596 -> 875020 bytes
>  qcom/venus-5.4/venus.b03 | Bin 33792 -> 33896 bytes
>  qcom/venus-5.4/venus.mbn | Bin 919708 -> 921236 bytes
>  qcom/venus-5.4/venus.mdt | Bin 7020 -> 7020 bytes
>  qcom/vpu-2.0/venus.b00   | Bin 692 -> 0 bytes
>  qcom/vpu-2.0/venus.b01   | Bin 7376 -> 0 bytes
>  qcom/vpu-2.0/venus.b02   | Bin 300 -> 0 bytes
>  qcom/vpu-2.0/venus.b03   | Bin 20 -> 0 bytes
>  qcom/vpu-2.0/venus.b04   | Bin 20 -> 0 bytes
>  qcom/vpu-2.0/venus.b05   | Bin 20 -> 0 bytes
>  qcom/vpu-2.0/venus.b06   | Bin 20 -> 0 bytes
>  qcom/vpu-2.0/venus.b07   | Bin 24 -> 0 bytes
>  qcom/vpu-2.0/venus.b08   | Bin 16 -> 0 bytes
>  qcom/vpu-2.0/venus.b09   | Bin 939184 -> 0 bytes
>  qcom/vpu-2.0/venus.b10   | Bin 42976 -> 0 bytes
>  qcom/vpu-2.0/venus.b19   |   1 -
>  qcom/vpu-2.0/venus.mbn   | Bin 2031188 -> 2031620 bytes
>  qcom/vpu-2.0/venus.mdt   | Bin 8068 -> 0 bytes
>  21 files changed, 2 insertions(+), 16 deletions(-)
>  delete mode 100644 qcom/vpu-2.0/venus.b00
>  delete mode 100644 qcom/vpu-2.0/venus.b01
>  delete mode 100644 qcom/vpu-2.0/venus.b02
>  delete mode 100644 qcom/vpu-2.0/venus.b03
>  delete mode 100644 qcom/vpu-2.0/venus.b04
>  delete mode 100644 qcom/vpu-2.0/venus.b05
>  delete mode 100644 qcom/vpu-2.0/venus.b06
>  delete mode 100644 qcom/vpu-2.0/venus.b07
>  delete mode 100644 qcom/vpu-2.0/venus.b08
>  delete mode 100644 qcom/vpu-2.0/venus.b09
>  delete mode 100644 qcom/vpu-2.0/venus.b10
>  delete mode 100644 qcom/vpu-2.0/venus.b19
>  delete mode 100644 qcom/vpu-2.0/venus.mdt

As I wrote in another email thread, it might be better to drop split
files, add only venus.mbn and then use Link: lines to create venus.mdt
-> venus.mbn symlinks.
Nathan Hebert Oct. 24, 2022, 11:13 p.m. UTC | #2 
On Thu, Oct 20, 2022 at 11:02 AM Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> On Thu, 20 Oct 2022 at 20:08, Nathan Hebert <nhebert@chromium.org> wrote:
> >
> > The following changes since commit 48407ffd7adb9511701547068b1e6f0956bd1c94:
> >
> >   cnm: update chips&media wave521c firmware. (2022-10-17 10:20:43 -0400)
> >
> > are available in the Git repository at:
> >
> >   https://github.com/nathan-google/linux-firmware.git
> > update_sc7180_and_sc7280_firmwares_v2
> >
> > for you to fetch changes up to 054271e4164b6b4c069a3aaaeb9bfcde8c850339:
> >
> >   qcom: update venus firmware files for VPU-2.0 (2022-10-20 09:37:35 -0700)
> >
> > ----------------------------------------------------------------
> > Nathan Hebert (3):
> >       qcom: update venus firmware files for v5.4
> >       qcom: remove split SC7280 venus firmware images
> >       qcom: update venus firmware files for VPU-2.0
> >
> >  WHENCE                   |  17 ++---------------
> >  qcom/venus-5.4/venus.b00 | Bin 212 -> 212 bytes
> >  qcom/venus-5.4/venus.b01 | Bin 6808 -> 6808 bytes
> >  qcom/venus-5.4/venus.b02 | Bin 873596 -> 875020 bytes
> >  qcom/venus-5.4/venus.b03 | Bin 33792 -> 33896 bytes
> >  qcom/venus-5.4/venus.mbn | Bin 919708 -> 921236 bytes
> >  qcom/venus-5.4/venus.mdt | Bin 7020 -> 7020 bytes
> >  qcom/vpu-2.0/venus.b00   | Bin 692 -> 0 bytes
> >  qcom/vpu-2.0/venus.b01   | Bin 7376 -> 0 bytes
> >  qcom/vpu-2.0/venus.b02   | Bin 300 -> 0 bytes
> >  qcom/vpu-2.0/venus.b03   | Bin 20 -> 0 bytes
> >  qcom/vpu-2.0/venus.b04   | Bin 20 -> 0 bytes
> >  qcom/vpu-2.0/venus.b05   | Bin 20 -> 0 bytes
> >  qcom/vpu-2.0/venus.b06   | Bin 20 -> 0 bytes
> >  qcom/vpu-2.0/venus.b07   | Bin 24 -> 0 bytes
> >  qcom/vpu-2.0/venus.b08   | Bin 16 -> 0 bytes
> >  qcom/vpu-2.0/venus.b09   | Bin 939184 -> 0 bytes
> >  qcom/vpu-2.0/venus.b10   | Bin 42976 -> 0 bytes
> >  qcom/vpu-2.0/venus.b19   |   1 -
> >  qcom/vpu-2.0/venus.mbn   | Bin 2031188 -> 2031620 bytes
> >  qcom/vpu-2.0/venus.mdt   | Bin 8068 -> 0 bytes
> >  21 files changed, 2 insertions(+), 16 deletions(-)
> >  delete mode 100644 qcom/vpu-2.0/venus.b00
> >  delete mode 100644 qcom/vpu-2.0/venus.b01
> >  delete mode 100644 qcom/vpu-2.0/venus.b02
> >  delete mode 100644 qcom/vpu-2.0/venus.b03
> >  delete mode 100644 qcom/vpu-2.0/venus.b04
> >  delete mode 100644 qcom/vpu-2.0/venus.b05
> >  delete mode 100644 qcom/vpu-2.0/venus.b06
> >  delete mode 100644 qcom/vpu-2.0/venus.b07
> >  delete mode 100644 qcom/vpu-2.0/venus.b08
> >  delete mode 100644 qcom/vpu-2.0/venus.b09
> >  delete mode 100644 qcom/vpu-2.0/venus.b10
> >  delete mode 100644 qcom/vpu-2.0/venus.b19
> >  delete mode 100644 qcom/vpu-2.0/venus.mdt
>
> As I wrote in another email thread, it might be better to drop split
> files, add only venus.mbn and then use Link: lines to create venus.mdt
> -> venus.mbn symlinks.
Thanks for the feedback. There was some concern from Vikash at Qualcomm
about making a change to the venus-5.4 firmware to drop the split files.

Vikash, are you able to comment about whether or not it is a concern to replace
the venus.mdt and split files with a symbolic link to venus.mdn for
the venus-5.4
files? I tested the suggested approach on ChromeOS with kernels based on 5.4
and 5.15 and did not see any issues.

This is a holiday week, so his response may be slow.

>
> --
> With best wishes
> Dmitry
Rob Clark Oct. 25, 2022, 2:24 p.m. UTC | #3 
On Mon, Oct 24, 2022 at 4:13 PM Nathan Hebert <nhebert@chromium.org> wrote:
>
> On Thu, Oct 20, 2022 at 11:02 AM Dmitry Baryshkov
> <dmitry.baryshkov@linaro.org> wrote:
> >
> > On Thu, 20 Oct 2022 at 20:08, Nathan Hebert <nhebert@chromium.org> wrote:
> > >
> > > The following changes since commit 48407ffd7adb9511701547068b1e6f0956bd1c94:
> > >
> > >   cnm: update chips&media wave521c firmware. (2022-10-17 10:20:43 -0400)
> > >
> > > are available in the Git repository at:
> > >
> > >   https://github.com/nathan-google/linux-firmware.git
> > > update_sc7180_and_sc7280_firmwares_v2
> > >
> > > for you to fetch changes up to 054271e4164b6b4c069a3aaaeb9bfcde8c850339:
> > >
> > >   qcom: update venus firmware files for VPU-2.0 (2022-10-20 09:37:35 -0700)
> > >
> > > ----------------------------------------------------------------
> > > Nathan Hebert (3):
> > >       qcom: update venus firmware files for v5.4
> > >       qcom: remove split SC7280 venus firmware images
> > >       qcom: update venus firmware files for VPU-2.0
> > >
> > >  WHENCE                   |  17 ++---------------
> > >  qcom/venus-5.4/venus.b00 | Bin 212 -> 212 bytes
> > >  qcom/venus-5.4/venus.b01 | Bin 6808 -> 6808 bytes
> > >  qcom/venus-5.4/venus.b02 | Bin 873596 -> 875020 bytes
> > >  qcom/venus-5.4/venus.b03 | Bin 33792 -> 33896 bytes
> > >  qcom/venus-5.4/venus.mbn | Bin 919708 -> 921236 bytes
> > >  qcom/venus-5.4/venus.mdt | Bin 7020 -> 7020 bytes
> > >  qcom/vpu-2.0/venus.b00   | Bin 692 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b01   | Bin 7376 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b02   | Bin 300 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b03   | Bin 20 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b04   | Bin 20 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b05   | Bin 20 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b06   | Bin 20 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b07   | Bin 24 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b08   | Bin 16 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b09   | Bin 939184 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b10   | Bin 42976 -> 0 bytes
> > >  qcom/vpu-2.0/venus.b19   |   1 -
> > >  qcom/vpu-2.0/venus.mbn   | Bin 2031188 -> 2031620 bytes
> > >  qcom/vpu-2.0/venus.mdt   | Bin 8068 -> 0 bytes
> > >  21 files changed, 2 insertions(+), 16 deletions(-)
> > >  delete mode 100644 qcom/vpu-2.0/venus.b00
> > >  delete mode 100644 qcom/vpu-2.0/venus.b01
> > >  delete mode 100644 qcom/vpu-2.0/venus.b02
> > >  delete mode 100644 qcom/vpu-2.0/venus.b03
> > >  delete mode 100644 qcom/vpu-2.0/venus.b04
> > >  delete mode 100644 qcom/vpu-2.0/venus.b05
> > >  delete mode 100644 qcom/vpu-2.0/venus.b06
> > >  delete mode 100644 qcom/vpu-2.0/venus.b07
> > >  delete mode 100644 qcom/vpu-2.0/venus.b08
> > >  delete mode 100644 qcom/vpu-2.0/venus.b09
> > >  delete mode 100644 qcom/vpu-2.0/venus.b10
> > >  delete mode 100644 qcom/vpu-2.0/venus.b19
> > >  delete mode 100644 qcom/vpu-2.0/venus.mdt
> >
> > As I wrote in another email thread, it might be better to drop split
> > files, add only venus.mbn and then use Link: lines to create venus.mdt
> > -> venus.mbn symlinks.
> Thanks for the feedback. There was some concern from Vikash at Qualcomm
> about making a change to the venus-5.4 firmware to drop the split files.
>
> Vikash, are you able to comment about whether or not it is a concern to replace
> the venus.mdt and split files with a symbolic link to venus.mdn for
> the venus-5.4
> files? I tested the suggested approach on ChromeOS with kernels based on 5.4
> and 5.15 and did not see any issues.
>
> This is a holiday week, so his response may be slow.
>

The symlink trick that Dmitry suggested would cover the case that
Vikash is concerned about.

BR,
-R
Nathan Hebert Oct. 25, 2022, 8:58 p.m. UTC | #4 
On Tue, Oct 25, 2022 at 7:24 AM Rob Clark <robdclark@gmail.com> wrote:
>
> On Mon, Oct 24, 2022 at 4:13 PM Nathan Hebert <nhebert@chromium.org> wrote:
> >
> > On Thu, Oct 20, 2022 at 11:02 AM Dmitry Baryshkov
> > <dmitry.baryshkov@linaro.org> wrote:
> > >
> > > On Thu, 20 Oct 2022 at 20:08, Nathan Hebert <nhebert@chromium.org> wrote:
> > > >
> > > > The following changes since commit 48407ffd7adb9511701547068b1e6f0956bd1c94:
> > > >
> > > >   cnm: update chips&media wave521c firmware. (2022-10-17 10:20:43 -0400)
> > > >
> > > > are available in the Git repository at:
> > > >
> > > >   https://github.com/nathan-google/linux-firmware.git
> > > > update_sc7180_and_sc7280_firmwares_v2
> > > >
> > > > for you to fetch changes up to 054271e4164b6b4c069a3aaaeb9bfcde8c850339:
> > > >
> > > >   qcom: update venus firmware files for VPU-2.0 (2022-10-20 09:37:35 -0700)
> > > >
> > > > ----------------------------------------------------------------
> > > > Nathan Hebert (3):
> > > >       qcom: update venus firmware files for v5.4
> > > >       qcom: remove split SC7280 venus firmware images
> > > >       qcom: update venus firmware files for VPU-2.0
> > > >
> > > >  WHENCE                   |  17 ++---------------
> > > >  qcom/venus-5.4/venus.b00 | Bin 212 -> 212 bytes
> > > >  qcom/venus-5.4/venus.b01 | Bin 6808 -> 6808 bytes
> > > >  qcom/venus-5.4/venus.b02 | Bin 873596 -> 875020 bytes
> > > >  qcom/venus-5.4/venus.b03 | Bin 33792 -> 33896 bytes
> > > >  qcom/venus-5.4/venus.mbn | Bin 919708 -> 921236 bytes
> > > >  qcom/venus-5.4/venus.mdt | Bin 7020 -> 7020 bytes
> > > >  qcom/vpu-2.0/venus.b00   | Bin 692 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b01   | Bin 7376 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b02   | Bin 300 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b03   | Bin 20 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b04   | Bin 20 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b05   | Bin 20 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b06   | Bin 20 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b07   | Bin 24 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b08   | Bin 16 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b09   | Bin 939184 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b10   | Bin 42976 -> 0 bytes
> > > >  qcom/vpu-2.0/venus.b19   |   1 -
> > > >  qcom/vpu-2.0/venus.mbn   | Bin 2031188 -> 2031620 bytes
> > > >  qcom/vpu-2.0/venus.mdt   | Bin 8068 -> 0 bytes
> > > >  21 files changed, 2 insertions(+), 16 deletions(-)
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b00
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b01
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b02
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b03
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b04
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b05
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b06
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b07
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b08
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b09
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b10
> > > >  delete mode 100644 qcom/vpu-2.0/venus.b19
> > > >  delete mode 100644 qcom/vpu-2.0/venus.mdt
> > >
> > > As I wrote in another email thread, it might be better to drop split
> > > files, add only venus.mbn and then use Link: lines to create venus.mdt
> > > -> venus.mbn symlinks.
> > Thanks for the feedback. There was some concern from Vikash at Qualcomm
> > about making a change to the venus-5.4 firmware to drop the split files.
> >
> > Vikash, are you able to comment about whether or not it is a concern to replace
> > the venus.mdt and split files with a symbolic link to venus.mdn for
> > the venus-5.4
> > files? I tested the suggested approach on ChromeOS with kernels based on 5.4
> > and 5.15 and did not see any issues.
> >
> > This is a holiday week, so his response may be slow.
> >
>
> The symlink trick that Dmitry suggested would cover the case that
> Vikash is concerned about.
>
> BR,
> -R
Thanks Rob, Dmitry, and Vikash for the feedback. I have sent a V3 pull
request that incorporates Dmitry's suggestion to the linux-firmware
mailing list.

BR,
Nathan