Re: [PATCH v2 2/2] s5p-mfc: Adding MFC v7 and v8 firmwares
On Thu, 2014-06-05 at 09:54 +0530, Arun Kumar K wrote: > Hi Ben, > > On Wed, Jun 4, 2014 at 5:43 PM, Ben Hutchings wrote: > > On Wed, 2014-06-04 at 13:08 +0200, Kamil Debski wrote: > >> Hi Arun, Ben, > >> > >> > >> > From: arunkk.sams...@gmail.com [mailto:arunkk.sams...@gmail.com] On > >> > Behalf Of Arun Kumar K > >> > Sent: Wednesday, June 04, 2014 7:32 AM > >> > > >> > Hi Ben, > >> > > >> > On Wed, Jun 4, 2014 at 4:56 AM, Ben Hutchings [...] > >> > > Just noticed a problem with all the s5p-mfc firmware: you are putting > >> > > it in the s5p-mfc subdirectory, so distribution packages will install > >> > > them in /lib/firmware/s5p-mfc. However, unless I'm mistaken your > >> > > driver requests it with a bare filename (e.g. "s5p-mfc-v7.fw") so it > >> > > won't be found. > >> > > > >> > > >> > Yes this issue is there. > >> > Kamil, are you ok with changing the fw load path to s5p-mfc/s5p-mfc*.fw > >> > in the driver? > >> > >> We have two options here: > >> 1) Change the driver to request s5p-mfc/sp5-mfc-fw* > >> 2) Move the files in the firmware repository > >> > >> I would opt for the second option. The driver was uploaded before the > >> firmware. I see that there are many firmware files in the root folder > >> of the repository > >> > >> Ben, what is your opinion? Which is preferred: creating subdirectories > >> or storing firmware in the root folder? > > > > I prefer subdirectories, but compatibility wins. So I'll apply the > > following patch if no-one objects. > > > > Ben. > > > > --- > > Subject: Move the s5p-mfc firmware out of its subdirectory > > > > The driver does not include this subdirectory name when requesting > > firmware. > > > > Signed-off-by: Ben Hutchings > > --- > > WHENCE | 10 +- > > s5p-mfc/s5p-mfc-v6-v2.fw => s5p-mfc-v6-v2.fw | Bin > > s5p-mfc/s5p-mfc-v6.fw => s5p-mfc-v6.fw | Bin > > s5p-mfc/s5p-mfc-v7.fw => s5p-mfc-v7.fw | Bin > > s5p-mfc/s5p-mfc-v8.fw => s5p-mfc-v8.fw | Bin > > s5p-mfc/s5p-mfc.fw => s5p-mfc.fw | Bin > > 6 files changed, 5 insertions(+), 5 deletions(-) > > rename s5p-mfc/s5p-mfc-v6-v2.fw => s5p-mfc-v6-v2.fw (100%) > > rename s5p-mfc/s5p-mfc-v6.fw => s5p-mfc-v6.fw (100%) > > rename s5p-mfc/s5p-mfc-v7.fw => s5p-mfc-v7.fw (100%) > > rename s5p-mfc/s5p-mfc-v8.fw => s5p-mfc-v8.fw (100%) > > rename s5p-mfc/s5p-mfc.fw => s5p-mfc.fw (100%) [...] > Looks good to me. I've made this change. Ben. -- Ben Hutchings When in doubt, use brute force. - Ken Thompson signature.asc Description: This is a digitally signed message part
Re: [PATCH v2 2/2] s5p-mfc: Adding MFC v7 and v8 firmwares
Hi Ben, On Wed, Jun 4, 2014 at 5:43 PM, Ben Hutchings wrote: > On Wed, 2014-06-04 at 13:08 +0200, Kamil Debski wrote: >> Hi Arun, Ben, >> >> >> > From: arunkk.sams...@gmail.com [mailto:arunkk.sams...@gmail.com] On >> > Behalf Of Arun Kumar K >> > Sent: Wednesday, June 04, 2014 7:32 AM >> > >> > Hi Ben, >> > >> > On Wed, Jun 4, 2014 at 4:56 AM, Ben Hutchings >> > wrote: >> > > On Tue, 2014-06-03 at 20:34 +0100, Ben Hutchings wrote: >> > >> On Wed, 2014-05-21 at 18:11 +0530, Arun Kumar K wrote: >> > >> > Adding the following firmware files for MFC >> > >> > s5p-mfc-v7.fw: Used in exynos 5420 >> > >> > s5p-mfc-v8.fw: Used in exynos 5800 >> > >> > >> > >> > Signed-off-by: Arun Kumar K >> > >> > --- >> > >> > Changes from v1 >> > >> > - None >> > >> > --- >> > >> > WHENCE|2 ++ >> > >> > s5p-mfc/s5p-mfc-v7.fw | Bin 0 -> 382724 bytes >> > >> > s5p-mfc/s5p-mfc-v8.fw | Bin 0 -> 360576 bytes >> > >> > 3 files changed, 2 insertions(+) >> > >> > create mode 100644 s5p-mfc/s5p-mfc-v7.fw create mode 100644 >> > >> > s5p-mfc/s5p-mfc-v8.fw >> > >> [...] >> > >> >> > >> Applied, thanks. >> > > >> > > Just noticed a problem with all the s5p-mfc firmware: you are putting >> > > it in the s5p-mfc subdirectory, so distribution packages will install >> > > them in /lib/firmware/s5p-mfc. However, unless I'm mistaken your >> > > driver requests it with a bare filename (e.g. "s5p-mfc-v7.fw") so it >> > > won't be found. >> > > >> > >> > Yes this issue is there. >> > Kamil, are you ok with changing the fw load path to s5p-mfc/s5p-mfc*.fw >> > in the driver? >> >> We have two options here: >> 1) Change the driver to request s5p-mfc/sp5-mfc-fw* >> 2) Move the files in the firmware repository >> >> I would opt for the second option. The driver was uploaded before the >> firmware. I see that there are many firmware files in the root folder >> of the repository >> >> Ben, what is your opinion? Which is preferred: creating subdirectories >> or storing firmware in the root folder? > > I prefer subdirectories, but compatibility wins. So I'll apply the > following patch if no-one objects. > > Ben. > > --- > Subject: Move the s5p-mfc firmware out of its subdirectory > > The driver does not include this subdirectory name when requesting > firmware. > > Signed-off-by: Ben Hutchings > --- > WHENCE | 10 +- > s5p-mfc/s5p-mfc-v6-v2.fw => s5p-mfc-v6-v2.fw | Bin > s5p-mfc/s5p-mfc-v6.fw => s5p-mfc-v6.fw | Bin > s5p-mfc/s5p-mfc-v7.fw => s5p-mfc-v7.fw | Bin > s5p-mfc/s5p-mfc-v8.fw => s5p-mfc-v8.fw | Bin > s5p-mfc/s5p-mfc.fw => s5p-mfc.fw | Bin > 6 files changed, 5 insertions(+), 5 deletions(-) > rename s5p-mfc/s5p-mfc-v6-v2.fw => s5p-mfc-v6-v2.fw (100%) > rename s5p-mfc/s5p-mfc-v6.fw => s5p-mfc-v6.fw (100%) > rename s5p-mfc/s5p-mfc-v7.fw => s5p-mfc-v7.fw (100%) > rename s5p-mfc/s5p-mfc-v8.fw => s5p-mfc-v8.fw (100%) > rename s5p-mfc/s5p-mfc.fw => s5p-mfc.fw (100%) > > diff --git a/WHENCE b/WHENCE > index 6a64fc2..a172f5f 100644 > --- a/WHENCE > +++ b/WHENCE > @@ -2250,11 +2250,11 @@ Licence: Redistributable. See > LICENCE.atheros_firmware for details > > Driver: s5p-mfc - Samsung MFC video encoder/decoder driver > > -File: s5p-mfc/s5p-mfc.fw > -File: s5p-mfc/s5p-mfc-v6.fw > -File: s5p-mfc/s5p-mfc-v6-v2.fw > -File: s5p-mfc/s5p-mfc-v7.fw > -File: s5p-mfc/s5p-mfc-v8.fw > +File: s5p-mfc.fw > +File: s5p-mfc-v6.fw > +File: s5p-mfc-v6-v2.fw > +File: s5p-mfc-v7.fw > +File: s5p-mfc-v8.fw > > Licence: > Samsung grants permission to use and redistribute aforementioned firmware > diff --git a/s5p-mfc/s5p-mfc-v6-v2.fw b/s5p-mfc-v6-v2.fw > similarity index 100% > rename from s5p-mfc/s5p-mfc-v6-v2.fw > rename to s5p-mfc-v6-v2.fw > diff --git a/s5p-mfc/s5p-mfc-v6.fw b/s5p-mfc-v6.fw > similarity index 100% > rename from s5p-mfc/s5p-mfc-v6.fw > rename to s5p-mfc-v6.fw > diff --git a/s5p-mfc/s5p-mfc-v7.fw b/s5p-mfc-v7.fw > similarity index 100% > rename from s5p-mfc/s5p-mfc-v7.fw > rename to s5p-mfc-v7.fw > diff --git a/s5p-mfc/s5p-mfc-v8.fw b/s5p-mfc-v8.fw > similarity index 100% > rename from s5p-mfc/s5p-mfc-v8.fw > rename to s5p-mfc-v8.fw > diff --git a/s5p-mfc/s5p-mfc.fw b/s5p-mfc.fw > similarity index 100% > rename from s5p-mfc/s5p-mfc.fw > rename to s5p-mfc.fw > > Looks good to me. Regards Arun > -- > Ben Hutchings > Experience is what causes a person to make new mistakes instead of old ones. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 2/2] s5p-mfc: Adding MFC v7 and v8 firmwares
On Wed, 2014-06-04 at 13:08 +0200, Kamil Debski wrote: > Hi Arun, Ben, > > > > From: arunkk.sams...@gmail.com [mailto:arunkk.sams...@gmail.com] On > > Behalf Of Arun Kumar K > > Sent: Wednesday, June 04, 2014 7:32 AM > > > > Hi Ben, > > > > On Wed, Jun 4, 2014 at 4:56 AM, Ben Hutchings > > wrote: > > > On Tue, 2014-06-03 at 20:34 +0100, Ben Hutchings wrote: > > >> On Wed, 2014-05-21 at 18:11 +0530, Arun Kumar K wrote: > > >> > Adding the following firmware files for MFC > > >> > s5p-mfc-v7.fw: Used in exynos 5420 > > >> > s5p-mfc-v8.fw: Used in exynos 5800 > > >> > > > >> > Signed-off-by: Arun Kumar K > > >> > --- > > >> > Changes from v1 > > >> > - None > > >> > --- > > >> > WHENCE|2 ++ > > >> > s5p-mfc/s5p-mfc-v7.fw | Bin 0 -> 382724 bytes > > >> > s5p-mfc/s5p-mfc-v8.fw | Bin 0 -> 360576 bytes > > >> > 3 files changed, 2 insertions(+) > > >> > create mode 100644 s5p-mfc/s5p-mfc-v7.fw create mode 100644 > > >> > s5p-mfc/s5p-mfc-v8.fw > > >> [...] > > >> > > >> Applied, thanks. > > > > > > Just noticed a problem with all the s5p-mfc firmware: you are putting > > > it in the s5p-mfc subdirectory, so distribution packages will install > > > them in /lib/firmware/s5p-mfc. However, unless I'm mistaken your > > > driver requests it with a bare filename (e.g. "s5p-mfc-v7.fw") so it > > > won't be found. > > > > > > > Yes this issue is there. > > Kamil, are you ok with changing the fw load path to s5p-mfc/s5p-mfc*.fw > > in the driver? > > We have two options here: > 1) Change the driver to request s5p-mfc/sp5-mfc-fw* > 2) Move the files in the firmware repository > > I would opt for the second option. The driver was uploaded before the > firmware. I see that there are many firmware files in the root folder > of the repository > > Ben, what is your opinion? Which is preferred: creating subdirectories > or storing firmware in the root folder? I prefer subdirectories, but compatibility wins. So I'll apply the following patch if no-one objects. Ben. --- Subject: Move the s5p-mfc firmware out of its subdirectory The driver does not include this subdirectory name when requesting firmware. Signed-off-by: Ben Hutchings --- WHENCE | 10 +- s5p-mfc/s5p-mfc-v6-v2.fw => s5p-mfc-v6-v2.fw | Bin s5p-mfc/s5p-mfc-v6.fw => s5p-mfc-v6.fw | Bin s5p-mfc/s5p-mfc-v7.fw => s5p-mfc-v7.fw | Bin s5p-mfc/s5p-mfc-v8.fw => s5p-mfc-v8.fw | Bin s5p-mfc/s5p-mfc.fw => s5p-mfc.fw | Bin 6 files changed, 5 insertions(+), 5 deletions(-) rename s5p-mfc/s5p-mfc-v6-v2.fw => s5p-mfc-v6-v2.fw (100%) rename s5p-mfc/s5p-mfc-v6.fw => s5p-mfc-v6.fw (100%) rename s5p-mfc/s5p-mfc-v7.fw => s5p-mfc-v7.fw (100%) rename s5p-mfc/s5p-mfc-v8.fw => s5p-mfc-v8.fw (100%) rename s5p-mfc/s5p-mfc.fw => s5p-mfc.fw (100%) diff --git a/WHENCE b/WHENCE index 6a64fc2..a172f5f 100644 --- a/WHENCE +++ b/WHENCE @@ -2250,11 +2250,11 @@ Licence: Redistributable. See LICENCE.atheros_firmware for details Driver: s5p-mfc - Samsung MFC video encoder/decoder driver -File: s5p-mfc/s5p-mfc.fw -File: s5p-mfc/s5p-mfc-v6.fw -File: s5p-mfc/s5p-mfc-v6-v2.fw -File: s5p-mfc/s5p-mfc-v7.fw -File: s5p-mfc/s5p-mfc-v8.fw +File: s5p-mfc.fw +File: s5p-mfc-v6.fw +File: s5p-mfc-v6-v2.fw +File: s5p-mfc-v7.fw +File: s5p-mfc-v8.fw Licence: Samsung grants permission to use and redistribute aforementioned firmware diff --git a/s5p-mfc/s5p-mfc-v6-v2.fw b/s5p-mfc-v6-v2.fw similarity index 100% rename from s5p-mfc/s5p-mfc-v6-v2.fw rename to s5p-mfc-v6-v2.fw diff --git a/s5p-mfc/s5p-mfc-v6.fw b/s5p-mfc-v6.fw similarity index 100% rename from s5p-mfc/s5p-mfc-v6.fw rename to s5p-mfc-v6.fw diff --git a/s5p-mfc/s5p-mfc-v7.fw b/s5p-mfc-v7.fw similarity index 100% rename from s5p-mfc/s5p-mfc-v7.fw rename to s5p-mfc-v7.fw diff --git a/s5p-mfc/s5p-mfc-v8.fw b/s5p-mfc-v8.fw similarity index 100% rename from s5p-mfc/s5p-mfc-v8.fw rename to s5p-mfc-v8.fw diff --git a/s5p-mfc/s5p-mfc.fw b/s5p-mfc.fw similarity index 100% rename from s5p-mfc/s5p-mfc.fw rename to s5p-mfc.fw -- Ben Hutchings Experience is what causes a person to make new mistakes instead of old ones. signature.asc Description: This is a digitally signed message part
RE: [PATCH v2 2/2] s5p-mfc: Adding MFC v7 and v8 firmwares
Hi Arun, Ben, > From: arunkk.sams...@gmail.com [mailto:arunkk.sams...@gmail.com] On > Behalf Of Arun Kumar K > Sent: Wednesday, June 04, 2014 7:32 AM > > Hi Ben, > > On Wed, Jun 4, 2014 at 4:56 AM, Ben Hutchings > wrote: > > On Tue, 2014-06-03 at 20:34 +0100, Ben Hutchings wrote: > >> On Wed, 2014-05-21 at 18:11 +0530, Arun Kumar K wrote: > >> > Adding the following firmware files for MFC > >> > s5p-mfc-v7.fw: Used in exynos 5420 > >> > s5p-mfc-v8.fw: Used in exynos 5800 > >> > > >> > Signed-off-by: Arun Kumar K > >> > --- > >> > Changes from v1 > >> > - None > >> > --- > >> > WHENCE|2 ++ > >> > s5p-mfc/s5p-mfc-v7.fw | Bin 0 -> 382724 bytes > >> > s5p-mfc/s5p-mfc-v8.fw | Bin 0 -> 360576 bytes > >> > 3 files changed, 2 insertions(+) > >> > create mode 100644 s5p-mfc/s5p-mfc-v7.fw create mode 100644 > >> > s5p-mfc/s5p-mfc-v8.fw > >> [...] > >> > >> Applied, thanks. > > > > Just noticed a problem with all the s5p-mfc firmware: you are putting > > it in the s5p-mfc subdirectory, so distribution packages will install > > them in /lib/firmware/s5p-mfc. However, unless I'm mistaken your > > driver requests it with a bare filename (e.g. "s5p-mfc-v7.fw") so it > > won't be found. > > > > Yes this issue is there. > Kamil, are you ok with changing the fw load path to s5p-mfc/s5p-mfc*.fw > in the driver? We have two options here: 1) Change the driver to request s5p-mfc/sp5-mfc-fw* 2) Move the files in the firmware repository I would opt for the second option. The driver was uploaded before the firmware. I see that there are many firmware files in the root folder of the repository Ben, what is your opinion? Which is preferred: creating subdirectories or storing firmware in the root folder? Best wishes, -- Kamil Debski Samsung R&D Institute Poland > > Regards > Arun > > > Ben. > > > > -- > > Ben Hutchings > > Experience is what causes a person to make new mistakes instead of > old ones. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 2/2] s5p-mfc: Adding MFC v7 and v8 firmwares
Hi Ben, On Wed, Jun 4, 2014 at 4:56 AM, Ben Hutchings wrote: > On Tue, 2014-06-03 at 20:34 +0100, Ben Hutchings wrote: >> On Wed, 2014-05-21 at 18:11 +0530, Arun Kumar K wrote: >> > Adding the following firmware files for MFC >> > s5p-mfc-v7.fw: Used in exynos 5420 >> > s5p-mfc-v8.fw: Used in exynos 5800 >> > >> > Signed-off-by: Arun Kumar K >> > --- >> > Changes from v1 >> > - None >> > --- >> > WHENCE|2 ++ >> > s5p-mfc/s5p-mfc-v7.fw | Bin 0 -> 382724 bytes >> > s5p-mfc/s5p-mfc-v8.fw | Bin 0 -> 360576 bytes >> > 3 files changed, 2 insertions(+) >> > create mode 100644 s5p-mfc/s5p-mfc-v7.fw >> > create mode 100644 s5p-mfc/s5p-mfc-v8.fw >> [...] >> >> Applied, thanks. > > Just noticed a problem with all the s5p-mfc firmware: you are putting it > in the s5p-mfc subdirectory, so distribution packages will install them > in /lib/firmware/s5p-mfc. However, unless I'm mistaken your driver > requests it with a bare filename (e.g. "s5p-mfc-v7.fw") so it won't be > found. > Yes this issue is there. Kamil, are you ok with changing the fw load path to s5p-mfc/s5p-mfc*.fw in the driver? Regards Arun > Ben. > > -- > Ben Hutchings > Experience is what causes a person to make new mistakes instead of old ones. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v2 2/2] s5p-mfc: Adding MFC v7 and v8 firmwares
On Tue, 2014-06-03 at 20:34 +0100, Ben Hutchings wrote: > On Wed, 2014-05-21 at 18:11 +0530, Arun Kumar K wrote: > > Adding the following firmware files for MFC > > s5p-mfc-v7.fw: Used in exynos 5420 > > s5p-mfc-v8.fw: Used in exynos 5800 > > > > Signed-off-by: Arun Kumar K > > --- > > Changes from v1 > > - None > > --- > > WHENCE|2 ++ > > s5p-mfc/s5p-mfc-v7.fw | Bin 0 -> 382724 bytes > > s5p-mfc/s5p-mfc-v8.fw | Bin 0 -> 360576 bytes > > 3 files changed, 2 insertions(+) > > create mode 100644 s5p-mfc/s5p-mfc-v7.fw > > create mode 100644 s5p-mfc/s5p-mfc-v8.fw > [...] > > Applied, thanks. Just noticed a problem with all the s5p-mfc firmware: you are putting it in the s5p-mfc subdirectory, so distribution packages will install them in /lib/firmware/s5p-mfc. However, unless I'm mistaken your driver requests it with a bare filename (e.g. "s5p-mfc-v7.fw") so it won't be found. Ben. -- Ben Hutchings Experience is what causes a person to make new mistakes instead of old ones. signature.asc Description: This is a digitally signed message part
Re: [PATCH v2 2/2] s5p-mfc: Adding MFC v7 and v8 firmwares
On Wed, 2014-05-21 at 18:11 +0530, Arun Kumar K wrote: > Adding the following firmware files for MFC > s5p-mfc-v7.fw: Used in exynos 5420 > s5p-mfc-v8.fw: Used in exynos 5800 > > Signed-off-by: Arun Kumar K > --- > Changes from v1 > - None > --- > WHENCE|2 ++ > s5p-mfc/s5p-mfc-v7.fw | Bin 0 -> 382724 bytes > s5p-mfc/s5p-mfc-v8.fw | Bin 0 -> 360576 bytes > 3 files changed, 2 insertions(+) > create mode 100644 s5p-mfc/s5p-mfc-v7.fw > create mode 100644 s5p-mfc/s5p-mfc-v8.fw [...] Applied, thanks. Ben. -- Ben Hutchings If more than one person is responsible for a bug, no one is at fault. signature.asc Description: This is a digitally signed message part