linux-firmware.lore.kernel.org archive mirror
 help / color / mirror / Atom feed
From: gtk_ruiwang <gtk_ruiwang@mediatek.com>
To: Alexandre Courbot <acourbot@chromium.org>
Cc: <linux-firmware@kernel.org>,
	Linux Media Mailing List <linux-media@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	"moderated list:ARM/Mediatek SoC support"
	<linux-arm-kernel@lists.infradead.org>,
	"moderated list:ARM/Mediatek SoC support"
	<linux-mediatek@lists.infradead.org>,
	<srv_heupstream@mediatek.com>, Tomasz Figa <tfiga@chromium.org>,
	<PoChun.Lin@mediatek.com>,
	Longfei Wang <longfei.wang@mediatek.com>,
	Yunfei Dong <yunfei.dong@mediatek.com>,
	Maoguang Meng <maoguang.meng@mediatek.com>
Subject: Re: [PATCH] mediatek: move MT8173 VPU FW to subfolder
Date: Wed, 26 Feb 2020 13:31:41 +0800	[thread overview]
Message-ID: <1582695101.6276.21.camel@mhfsdcap03> (raw)
In-Reply-To: <CAPBb6MWcoSjvxj5tHvWTiQWqY5CdgTemCmyhmio0yZMEq-4VWQ@mail.gmail.com>

Dear Alex,

we have submit a new patch to linux-firmware, and update WHENCE file.

please kindly help to review it.

Thanks
Best Regards

On Wed, 2020-02-26 at 13:45 +0900, Alexandre Courbot wrote:
> On Wed, Feb 26, 2020 at 1:20 PM gtk_ruiwang <gtk_ruiwang@mediatek.com> wrote:
> >
> > Dear Alex,
> >
> > Do you mean to create two symbolic link?
> >
> > lrwxrwxrwx  1 gtk_ruiwang gtk_ruiwang      25 Feb 26 11:52 vpu_d.bin ->
> > mediatek/mt8173/vpu_d.bin
> > lrwxrwxrwx  1 gtk_ruiwang gtk_ruiwang      25 Feb 26 11:52 vpu_p.bin ->
> > mediatek/mt8173/vpu_p.bin
> 
> Yes, otherwise older kernels will become unable to use the firmware. I
> don't know whether these are supposed to stay forever or whether we
> can remove them after a while... presumably the former.
> 
> Also the WHENCE file should probably be updated to list the new files.
> 
> >
> > we create them at the linux-firmware root folder.
> >
> > Thanks
> > Best Regards
> >
> > On Wed, 2020-02-26 at 12:41 +0900, Alexandre Courbot wrote:
> > > On Wed, Feb 26, 2020 at 12:38 PM <gtk_ruiwang@mediatek.com> wrote:
> > > >
> > > > From: gtk_ruiwang <gtk_ruiwang@mediatek.com>
> > > >
> > > > Currently vpu_d.bin and vpu_p.bin are at the root of
> > > > linux-firmware git tree, it's not appropriate so we move
> > > > them to subfolder mediatek/mt8173
> > > >
> > > > Release Version: 1.1.4
> > > >
> > > > Signed-off-by: Rui Wang <gtk_ruiwang@mediatek.com>
> > > > ---
> > > >  vpu_d.bin => mediatek/mt8173/vpu_d.bin | Bin
> > > >  vpu_p.bin => mediatek/mt8173/vpu_p.bin | Bin
> > > >  2 files changed, 0 insertions(+), 0 deletions(-)
> > > >  rename vpu_d.bin => mediatek/mt8173/vpu_d.bin (100%)
> > > >  rename vpu_p.bin => mediatek/mt8173/vpu_p.bin (100%)
> > >
> > > This is nice as it removes some stuff from the root, but for
> > > compatibility with older kernels that don't know about the new path
> > > shouldn't we at least temporarily create a symbolic link between the
> > > old location and the new one?
> >


WARNING: multiple messages have this Message-ID (diff)
From: gtk_ruiwang <gtk_ruiwang@mediatek.com>
To: Alexandre Courbot <acourbot@chromium.org>
Cc: Maoguang Meng <maoguang.meng@mediatek.com>,
	srv_heupstream@mediatek.com,
	Yunfei Dong <yunfei.dong@mediatek.com>,
	Longfei Wang <longfei.wang@mediatek.com>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-firmware@kernel.org, Tomasz Figa <tfiga@chromium.org>,
	"moderated list:ARM/Mediatek SoC support"
	<linux-mediatek@lists.infradead.org>,
	PoChun.Lin@mediatek.com,
	"moderated list:ARM/Mediatek SoC support"
	<linux-arm-kernel@lists.infradead.org>,
	Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH] mediatek: move MT8173 VPU FW to subfolder
Date: Wed, 26 Feb 2020 13:31:41 +0800	[thread overview]
Message-ID: <1582695101.6276.21.camel@mhfsdcap03> (raw)
Message-ID: <20200226053141.NK-2P3FaeWLwmsY8qkErNycR69jhap8ZEetWfIqaxFg@z> (raw)
In-Reply-To: <CAPBb6MWcoSjvxj5tHvWTiQWqY5CdgTemCmyhmio0yZMEq-4VWQ@mail.gmail.com>

Dear Alex,

we have submit a new patch to linux-firmware, and update WHENCE file.

please kindly help to review it.

Thanks
Best Regards

On Wed, 2020-02-26 at 13:45 +0900, Alexandre Courbot wrote:
> On Wed, Feb 26, 2020 at 1:20 PM gtk_ruiwang <gtk_ruiwang@mediatek.com> wrote:
> >
> > Dear Alex,
> >
> > Do you mean to create two symbolic link?
> >
> > lrwxrwxrwx  1 gtk_ruiwang gtk_ruiwang      25 Feb 26 11:52 vpu_d.bin ->
> > mediatek/mt8173/vpu_d.bin
> > lrwxrwxrwx  1 gtk_ruiwang gtk_ruiwang      25 Feb 26 11:52 vpu_p.bin ->
> > mediatek/mt8173/vpu_p.bin
> 
> Yes, otherwise older kernels will become unable to use the firmware. I
> don't know whether these are supposed to stay forever or whether we
> can remove them after a while... presumably the former.
> 
> Also the WHENCE file should probably be updated to list the new files.
> 
> >
> > we create them at the linux-firmware root folder.
> >
> > Thanks
> > Best Regards
> >
> > On Wed, 2020-02-26 at 12:41 +0900, Alexandre Courbot wrote:
> > > On Wed, Feb 26, 2020 at 12:38 PM <gtk_ruiwang@mediatek.com> wrote:
> > > >
> > > > From: gtk_ruiwang <gtk_ruiwang@mediatek.com>
> > > >
> > > > Currently vpu_d.bin and vpu_p.bin are at the root of
> > > > linux-firmware git tree, it's not appropriate so we move
> > > > them to subfolder mediatek/mt8173
> > > >
> > > > Release Version: 1.1.4
> > > >
> > > > Signed-off-by: Rui Wang <gtk_ruiwang@mediatek.com>
> > > > ---
> > > >  vpu_d.bin => mediatek/mt8173/vpu_d.bin | Bin
> > > >  vpu_p.bin => mediatek/mt8173/vpu_p.bin | Bin
> > > >  2 files changed, 0 insertions(+), 0 deletions(-)
> > > >  rename vpu_d.bin => mediatek/mt8173/vpu_d.bin (100%)
> > > >  rename vpu_p.bin => mediatek/mt8173/vpu_p.bin (100%)
> > >
> > > This is nice as it removes some stuff from the root, but for
> > > compatibility with older kernels that don't know about the new path
> > > shouldn't we at least temporarily create a symbolic link between the
> > > old location and the new one?
> >

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: gtk_ruiwang <gtk_ruiwang@mediatek.com>
To: Alexandre Courbot <acourbot@chromium.org>
Cc: Maoguang Meng <maoguang.meng@mediatek.com>,
	srv_heupstream@mediatek.com,
	Yunfei Dong <yunfei.dong@mediatek.com>,
	Longfei Wang <longfei.wang@mediatek.com>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-firmware@kernel.org, Tomasz Figa <tfiga@chromium.org>,
	"moderated list:ARM/Mediatek SoC support"
	<linux-mediatek@lists.infradead.org>,
	PoChun.Lin@mediatek.com,
	"moderated list:ARM/Mediatek SoC support"
	<linux-arm-kernel@lists.infradead.org>,
	Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH] mediatek: move MT8173 VPU FW to subfolder
Date: Wed, 26 Feb 2020 13:31:41 +0800	[thread overview]
Message-ID: <1582695101.6276.21.camel@mhfsdcap03> (raw)
Message-ID: <20200226053141.R47zumImx-HLqXv5icAqFWoKExKB0aEkqJbgoz3_HFo@z> (raw)
In-Reply-To: <CAPBb6MWcoSjvxj5tHvWTiQWqY5CdgTemCmyhmio0yZMEq-4VWQ@mail.gmail.com>

Dear Alex,

we have submit a new patch to linux-firmware, and update WHENCE file.

please kindly help to review it.

Thanks
Best Regards

On Wed, 2020-02-26 at 13:45 +0900, Alexandre Courbot wrote:
> On Wed, Feb 26, 2020 at 1:20 PM gtk_ruiwang <gtk_ruiwang@mediatek.com> wrote:
> >
> > Dear Alex,
> >
> > Do you mean to create two symbolic link?
> >
> > lrwxrwxrwx  1 gtk_ruiwang gtk_ruiwang      25 Feb 26 11:52 vpu_d.bin ->
> > mediatek/mt8173/vpu_d.bin
> > lrwxrwxrwx  1 gtk_ruiwang gtk_ruiwang      25 Feb 26 11:52 vpu_p.bin ->
> > mediatek/mt8173/vpu_p.bin
> 
> Yes, otherwise older kernels will become unable to use the firmware. I
> don't know whether these are supposed to stay forever or whether we
> can remove them after a while... presumably the former.
> 
> Also the WHENCE file should probably be updated to list the new files.
> 
> >
> > we create them at the linux-firmware root folder.
> >
> > Thanks
> > Best Regards
> >
> > On Wed, 2020-02-26 at 12:41 +0900, Alexandre Courbot wrote:
> > > On Wed, Feb 26, 2020 at 12:38 PM <gtk_ruiwang@mediatek.com> wrote:
> > > >
> > > > From: gtk_ruiwang <gtk_ruiwang@mediatek.com>
> > > >
> > > > Currently vpu_d.bin and vpu_p.bin are at the root of
> > > > linux-firmware git tree, it's not appropriate so we move
> > > > them to subfolder mediatek/mt8173
> > > >
> > > > Release Version: 1.1.4
> > > >
> > > > Signed-off-by: Rui Wang <gtk_ruiwang@mediatek.com>
> > > > ---
> > > >  vpu_d.bin => mediatek/mt8173/vpu_d.bin | Bin
> > > >  vpu_p.bin => mediatek/mt8173/vpu_p.bin | Bin
> > > >  2 files changed, 0 insertions(+), 0 deletions(-)
> > > >  rename vpu_d.bin => mediatek/mt8173/vpu_d.bin (100%)
> > > >  rename vpu_p.bin => mediatek/mt8173/vpu_p.bin (100%)
> > >
> > > This is nice as it removes some stuff from the root, but for
> > > compatibility with older kernels that don't know about the new path
> > > shouldn't we at least temporarily create a symbolic link between the
> > > old location and the new one?
> >

_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek

  parent reply	other threads:[~2020-02-26  5:31 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-26  3:36 pull request: linux-firmware: move MT8173 VPU FW to subfolder gtk_ruiwang
2020-02-26  3:36 ` gtk_ruiwang
2020-02-26  3:36 ` gtk_ruiwang
2020-02-26  3:36 ` gtk_ruiwang
2020-02-26  3:36 ` gtk_ruiwang
2020-02-26  3:36 ` [PATCH] mediatek: " gtk_ruiwang
2020-02-26  3:36   ` gtk_ruiwang
2020-02-26  3:36   ` gtk_ruiwang
2020-02-26  3:36   ` gtk_ruiwang
2020-02-26  3:36   ` gtk_ruiwang
2020-02-26  3:41   ` Alexandre Courbot
2020-02-26  3:41     ` Alexandre Courbot
2020-02-26  3:41     ` Alexandre Courbot
2020-02-26  3:41     ` Alexandre Courbot
2020-02-26  3:41     ` Alexandre Courbot
2020-02-26  4:19     ` gtk_ruiwang
2020-02-26  4:19       ` gtk_ruiwang
2020-02-26  4:19       ` gtk_ruiwang
2020-02-26  4:19       ` gtk_ruiwang
2020-02-26  4:19       ` gtk_ruiwang
2020-02-26  4:45       ` Alexandre Courbot
2020-02-26  4:45         ` Alexandre Courbot
2020-02-26  4:45         ` Alexandre Courbot
2020-02-26  4:45         ` Alexandre Courbot
2020-02-26  4:45         ` Alexandre Courbot
2020-02-26  5:31         ` gtk_ruiwang [this message]
2020-02-26  5:31           ` gtk_ruiwang
2020-02-26  5:31           ` gtk_ruiwang
2020-02-26  5:31           ` gtk_ruiwang
2020-02-26  5:31           ` gtk_ruiwang
2020-02-26  5:26 pull request: linux-firmware: " gtk_ruiwang
2020-02-26  5:26 ` [PATCH] mediatek: " gtk_ruiwang
2020-02-26  5:26   ` gtk_ruiwang
2020-02-26  5:26   ` gtk_ruiwang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1582695101.6276.21.camel@mhfsdcap03 \
    --to=gtk_ruiwang@mediatek.com \
    --cc=PoChun.Lin@mediatek.com \
    --cc=acourbot@chromium.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-firmware@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=longfei.wang@mediatek.com \
    --cc=maoguang.meng@mediatek.com \
    --cc=srv_heupstream@mediatek.com \
    --cc=tfiga@chromium.org \
    --cc=yunfei.dong@mediatek.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).