add video firmware for Kaanapali

add video firmware for Kaanapali

[Dikshita Agarwal]

Hi,

The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:

  Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)

are available in the Git repository at:

  https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
video-fw-kaanapali

for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:

  qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)

----------------------------------------------------------------
Dikshita Agarwal (1):
      qcom: vpu: add video firmware for Kaanapali

 WHENCE                   |   5 +++++
 qcom/vpu/vpu40_p2_s7.mbn | Bin 0 -> 2474696 bytes
 2 files changed, 5 insertions(+)
 create mode 100644 qcom/vpu/vpu40_p2_s7.mbn

Thanks,
Dikshita

Re: add video firmware for Kaanapali

[Dmitry Baryshkov]

On Wed, Oct 01, 2025 at 06:36:11PM +0530, Dikshita Agarwal wrote:
> Hi,
> 
> The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:
> 
>   Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)
> 
> are available in the Git repository at:
> 
>   https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
> video-fw-kaanapali
> 
> for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:
> 
>   qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)

+Link: qcom/vpu/vpu40_p2.mbn -> vpu40_p2_s7.mbn

Can we please drop that link? FWIW the driver for that platform is not
yet merged.

> 
> ----------------------------------------------------------------
> Dikshita Agarwal (1):
>       qcom: vpu: add video firmware for Kaanapali
> 
>  WHENCE                   |   5 +++++
>  qcom/vpu/vpu40_p2_s7.mbn | Bin 0 -> 2474696 bytes
>  2 files changed, 5 insertions(+)
>  create mode 100644 qcom/vpu/vpu40_p2_s7.mbn
> 
> Thanks,
> Dikshita

-- 
With best wishes
Dmitry

Re: add video firmware for Kaanapali

[Josh Boyer]

On Wed, Oct 1, 2025 at 8:48 PM Dmitry Baryshkov
<dmitry.baryshkov@oss.qualcomm.com> wrote:
>
> On Wed, Oct 01, 2025 at 06:36:11PM +0530, Dikshita Agarwal wrote:
> > Hi,
> >
> > The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:
> >
> >   Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)
> >
> > are available in the Git repository at:
> >
> >   https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
> > video-fw-kaanapali
> >
> > for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:
> >
> >   qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)
>
> +Link: qcom/vpu/vpu40_p2.mbn -> vpu40_p2_s7.mbn
>
> Can we please drop that link? FWIW the driver for that platform is not
> yet merged.

This got merged yesterday.

https://gitlab.com/kernel-firmware/linux-firmware/-/merge_requests/717

josh

Re: add video firmware for Kaanapali

[Mario Limonciello]

On 10/2/25 7:19 AM, Josh Boyer wrote:
> On Wed, Oct 1, 2025 at 8:48 PM Dmitry Baryshkov
> <dmitry.baryshkov@oss.qualcomm.com> wrote:
>>
>> On Wed, Oct 01, 2025 at 06:36:11PM +0530, Dikshita Agarwal wrote:
>>> Hi,
>>>
>>> The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:
>>>
>>>    Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)
>>>
>>> are available in the Git repository at:
>>>
>>>    https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
>>> video-fw-kaanapali
>>>
>>> for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:
>>>
>>>    qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)
>>
>> +Link: qcom/vpu/vpu40_p2.mbn -> vpu40_p2_s7.mbn
>>
>> Can we please drop that link? FWIW the driver for that platform is not
>> yet merged.
> 
> This got merged yesterday.
> 
> https://gitlab.com/kernel-firmware/linux-firmware/-/merge_requests/717
> 
> josh

Please send up a pull request to drop the link if it's not correct.

Re: add video firmware for Kaanapali

[Dmitry Baryshkov]

On Thu, Oct 02, 2025 at 08:29:47AM -0500, Mario Limonciello wrote:
> On 10/2/25 7:19 AM, Josh Boyer wrote:
> > On Wed, Oct 1, 2025 at 8:48 PM Dmitry Baryshkov
> > <dmitry.baryshkov@oss.qualcomm.com> wrote:
> > > 
> > > On Wed, Oct 01, 2025 at 06:36:11PM +0530, Dikshita Agarwal wrote:
> > > > Hi,
> > > > 
> > > > The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:
> > > > 
> > > >    Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)
> > > > 
> > > > are available in the Git repository at:
> > > > 
> > > >    https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
> > > > video-fw-kaanapali
> > > > 
> > > > for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:
> > > > 
> > > >    qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)
> > > 
> > > +Link: qcom/vpu/vpu40_p2.mbn -> vpu40_p2_s7.mbn
> > > 
> > > Can we please drop that link? FWIW the driver for that platform is not
> > > yet merged.
> > 
> > This got merged yesterday.
> > 
> > https://gitlab.com/kernel-firmware/linux-firmware/-/merge_requests/717
> > 
> > josh
> 
> Please send up a pull request to drop the link if it's not correct.

I'd let Dikshita respond and I'll send a patch if she or Vikash don't
object.

-- 
With best wishes
Dmitry

Re: add video firmware for Kaanapali

[Dikshita Agarwal]

On 10/2/2025 6:18 AM, Dmitry Baryshkov wrote:
> On Wed, Oct 01, 2025 at 06:36:11PM +0530, Dikshita Agarwal wrote:
>> Hi,
>>
>> The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:
>>
>>   Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)
>>
>> are available in the Git repository at:
>>
>>   https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
>> video-fw-kaanapali
>>
>> for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:
>>
>>   qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)
> 
> +Link: qcom/vpu/vpu40_p2.mbn -> vpu40_p2_s7.mbn
> 
> Can we please drop that link? FWIW the driver for that platform is not
> yet merged.

My intention was to keep the naming consistent in driver with other
platforms which are using _s7 firmwares but driver still won't have this
suffix.

eg: when I upload the patch to rename let's kailua firmware binary name
with adding _s7 suffix, driver would still have the name without the suffix
and soft link will take care of mapping

Any concern here?

Thanks,
Dikshita
> 
>>
>> ----------------------------------------------------------------
>> Dikshita Agarwal (1):
>>       qcom: vpu: add video firmware for Kaanapali
>>
>>  WHENCE                   |   5 +++++
>>  qcom/vpu/vpu40_p2_s7.mbn | Bin 0 -> 2474696 bytes
>>  2 files changed, 5 insertions(+)
>>  create mode 100644 qcom/vpu/vpu40_p2_s7.mbn
>>
>> Thanks,
>> Dikshita
> 

Re: add video firmware for Kaanapali

[Dmitry Baryshkov]

On Fri, 3 Oct 2025 at 09:55, Dikshita Agarwal
<dikshita.agarwal@oss.qualcomm.com> wrote:
>
>
>
> On 10/2/2025 6:18 AM, Dmitry Baryshkov wrote:
> > On Wed, Oct 01, 2025 at 06:36:11PM +0530, Dikshita Agarwal wrote:
> >> Hi,
> >>
> >> The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:
> >>
> >>   Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)
> >>
> >> are available in the Git repository at:
> >>
> >>   https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
> >> video-fw-kaanapali
> >>
> >> for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:
> >>
> >>   qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)
> >
> > +Link: qcom/vpu/vpu40_p2.mbn -> vpu40_p2_s7.mbn
> >
> > Can we please drop that link? FWIW the driver for that platform is not
> > yet merged.
>
> My intention was to keep the naming consistent in driver with other
> platforms which are using _s7 firmwares but driver still won't have this
> suffix.
>
> eg: when I upload the patch to rename let's kailua firmware binary name
> with adding _s7 suffix, driver would still have the name without the suffix
> and soft link will take care of mapping
>
> Any concern here?

There is a difference between having backwards-compatibility links
(yes, we carry them for a long) and just adding a symlink for a new
firmware that didn't exist beforehand. Extra symlinks add extra noise
for no particular reason.

>
> Thanks,
> Dikshita
> >
> >>
> >> ----------------------------------------------------------------
> >> Dikshita Agarwal (1):
> >>       qcom: vpu: add video firmware for Kaanapali
> >>
> >>  WHENCE                   |   5 +++++
> >>  qcom/vpu/vpu40_p2_s7.mbn | Bin 0 -> 2474696 bytes
> >>  2 files changed, 5 insertions(+)
> >>  create mode 100644 qcom/vpu/vpu40_p2_s7.mbn
> >>
> >> Thanks,
> >> Dikshita
> >



-- 
With best wishes
Dmitry

Re: add video firmware for Kaanapali

[Dikshita Agarwal]

On 10/3/2025 7:52 PM, Dmitry Baryshkov wrote:
> On Fri, 3 Oct 2025 at 09:55, Dikshita Agarwal
> <dikshita.agarwal@oss.qualcomm.com> wrote:
>>
>>
>>
>> On 10/2/2025 6:18 AM, Dmitry Baryshkov wrote:
>>> On Wed, Oct 01, 2025 at 06:36:11PM +0530, Dikshita Agarwal wrote:
>>>> Hi,
>>>>
>>>> The following changes since commit ed924f6db3c973f941fd6c6317c0d52664e83e5a:
>>>>
>>>>   Merge branch 'robot/pr-0-1759253351' into 'main' (2025-09-30 17:36:46 +0000)
>>>>
>>>> are available in the Git repository at:
>>>>
>>>>   https://git.codelinaro.org/clo/linux-kernel/linux-firmware.git
>>>> video-fw-kaanapali
>>>>
>>>> for you to fetch changes up to 7c2d4d41e96fdddfa360c0740e8a8ee9b5efb572:
>>>>
>>>>   qcom: vpu: add video firmware for Kaanapali (2025-10-01 18:32:02 +0530)
>>>
>>> +Link: qcom/vpu/vpu40_p2.mbn -> vpu40_p2_s7.mbn
>>>
>>> Can we please drop that link? FWIW the driver for that platform is not
>>> yet merged.
>>
>> My intention was to keep the naming consistent in driver with other
>> platforms which are using _s7 firmwares but driver still won't have this
>> suffix.
>>
>> eg: when I upload the patch to rename let's kailua firmware binary name
>> with adding _s7 suffix, driver would still have the name without the suffix
>> and soft link will take care of mapping
>>
>> Any concern here?
> 
> There is a difference between having backwards-compatibility links
> (yes, we carry them for a long) and just adding a symlink for a new
> firmware that didn't exist beforehand. Extra symlinks add extra noise
> for no particular reason.

I understand, will send the PR to remove the link.

Thanks,
Dikshita
> 
>>
>> Thanks,
>> Dikshita
>>>
>>>>
>>>> ----------------------------------------------------------------
>>>> Dikshita Agarwal (1):
>>>>       qcom: vpu: add video firmware for Kaanapali
>>>>
>>>>  WHENCE                   |   5 +++++
>>>>  qcom/vpu/vpu40_p2_s7.mbn | Bin 0 -> 2474696 bytes
>>>>  2 files changed, 5 insertions(+)
>>>>  create mode 100644 qcom/vpu/vpu40_p2_s7.mbn
>>>>
>>>> Thanks,
>>>> Dikshita
>>>
> 
> 
>