diff for duplicates of <1487774545.5907.17.camel@collabora.com> diff --git a/a/1.txt b/N1/1.txt index 488d2fb..d89f6a0 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,4 +1,4 @@ -Le mercredi 22 février 2017 à 10:10 -0300, Thibault Saunier a écrit : +Le mercredi 22 f?vrier 2017 ? 10:10 -0300, Thibault Saunier a ?crit?: > Hello, > > On 02/22/2017 06:29 AM, Andrzej Hajda wrote: @@ -23,8 +23,8 @@ Le mercredi 22 février 2017 à 10:10 -0300, Thibault Saunier a écrit : > > > - Fix a silly build error that slipped in while rebasing the > > > patches > > > -> > > drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 3 +++ -> > > 1 file changed, 3 insertions(+) +> > > ? drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 3 +++ +> > > ? 1 file changed, 3 insertions(+) > > > > > > diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c > > > b/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c @@ -33,9 +33,9 @@ Le mercredi 22 février 2017 à 10:10 -0300, Thibault Saunier a écrit : > > > +++ b/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c > > > @@ -386,6 +386,9 @@ static int vidioc_try_fmt(struct file *file, > > > void *priv, struct v4l2_format *f) -> > > struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp; -> > > struct s5p_mfc_fmt *fmt; -> > > +> > > ?? struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp; +> > > ?? struct s5p_mfc_fmt *fmt; +> > > ?? > > > + if (f->fmt.pix.field == V4L2_FIELD_ANY) > > > + f->fmt.pix.field = V4L2_FIELD_NONE; > > > + @@ -46,13 +46,13 @@ Le mercredi 22 février 2017 à 10:10 -0300, Thibault Saunier a écrit : > > If the only supported format is none you should set 'field' > > unconditionally to NONE, nothing more. > -> Afaict we support 2 things: +> Afaict we??support 2 things: > -> 1. NONE -> 2. INTERLACE +> ???1. NONE +> ???2. INTERLACE > > Until now we were not checking what was supported or not and -> basically +> basically? > ignoring that info, this patch > keeps the old behaviour making sure to be compliant. > @@ -70,7 +70,7 @@ This looks better to me. > ``` > > instead, it is probably more correct, do you think it is what should -> be +> be? > done here? > > Regards, @@ -81,8 +81,15 @@ This looks better to me. > > Regards > > Andrzej > > -> > > mfc_debug(2, "Type is %d\n", f->type); -> > > if (f->type == V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE) { -> > > fmt = find_format(f, MFC_FMT_DEC); +> > > ?? mfc_debug(2, "Type is %d\n", f->type); +> > > ?? if (f->type == V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE) { +> > > ?? fmt = find_format(f, MFC_FMT_DEC); > -> +> +-------------- next part -------------- +A non-text attachment was scrubbed... +Name: signature.asc +Type: application/pgp-signature +Size: 181 bytes +Desc: This is a digitally signed message part +URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170222/13a1ba63/attachment.sig> diff --git a/a/2.bin b/a/2.bin deleted file mode 100644 index b3ee98a..0000000 --- a/a/2.bin +++ /dev/null @@ -1,7 +0,0 @@ ------BEGIN PGP SIGNATURE----- -Version: GnuPG v2 - -iEYEABECAAYFAlito1EACgkQcVMCLawGqBy+tgCeN4UKQCQlZflnDAgybPdmgx0y -uksAoNNQBZGPedHd6Du/W5ySrwWGBc3B -=9TtN ------END PGP SIGNATURE----- diff --git a/a/2.hdr b/a/2.hdr deleted file mode 100644 index da6d245..0000000 --- a/a/2.hdr +++ /dev/null @@ -1,3 +0,0 @@ -Content-Type: application/pgp-signature; name="signature.asc" -Content-Description: This is a digitally signed message part -Content-Transfer-Encoding: 7bit diff --git a/a/content_digest b/N1/content_digest index 502b026..cd1e667 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,32 +3,13 @@ "ref\020170221192059.29745-4-thibault.saunier@osg.samsung.com\0" "ref\078444dcd-169f-0c16-0e09-6b71d1a502b2@samsung.com\0" "ref\0ed287d5a-687b-b344-3f20-10154071852c@osg.samsung.com\0" - "From\0Nicolas Dufresne <nicolas.dufresne@collabora.com>\0" - "Subject\0Re: [PATCH v5 3/3] [media] s5p-mfc: Check and set 'v4l2_pix_format:field' field in try_fmt\0" + "From\0nicolas.dufresne@collabora.com (Nicolas Dufresne)\0" + "Subject\0[PATCH v5 3/3] [media] s5p-mfc: Check and set 'v4l2_pix_format:field' field in try_fmt\0" "Date\0Wed, 22 Feb 2017 09:42:25 -0500\0" - "To\0Thibault Saunier <thibault.saunier@osg.samsung.com>" - Andrzej Hajda <a.hajda@samsung.com> - " linux-kernel@vger.kernel.org\0" - "Cc\0Mauro Carvalho Chehab <mchehab@kernel.org>" - Marek Szyprowski <m.szyprowski@samsung.com> - Kukjin Kim <kgene@kernel.org> - Mauro Carvalho Chehab <mchehab@s-opensource.com> - Andi Shyti <andi.shyti@samsung.com> - linux-media@vger.kernel.org - Shuah Khan <shuahkh@osg.samsung.com> - Javier Martinez Canillas <javier@osg.samsung.com> - linux-samsung-soc@vger.kernel.org - Krzysztof Kozlowski <krzk@kernel.org> - Inki Dae <inki.dae@samsung.com> - Sylwester Nawrocki <s.nawrocki@samsung.com> - linux-arm-kernel@lists.infradead.org - Ulf Hansson <ulf.hansson@linaro.org> - Jeongtae Park <jtp.park@samsung.com> - Kyungmin Park <kyungmin.park@samsung.com> - " Kamil Debski <kamil@wypas.org>\0" - "\01:1\0" + "To\0linux-arm-kernel@lists.infradead.org\0" + "\00:1\0" "b\0" - "Le mercredi 22 f\303\251vrier 2017 \303\240 10:10 -0300, Thibault Saunier a \303\251crit\302\240:\n" + "Le mercredi 22 f?vrier 2017 ? 10:10 -0300, Thibault Saunier a ?crit?:\n" "> Hello,\n" "> \n" "> On 02/22/2017 06:29 AM, Andrzej Hajda wrote:\n" @@ -53,8 +34,8 @@ "> > > - Fix a silly build error that slipped in while rebasing the\n" "> > > patches\n" "> > > \n" - "> > > \302\240 drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 3 +++\n" - "> > > \302\240 1 file changed, 3 insertions(+)\n" + "> > > ? drivers/media/platform/s5p-mfc/s5p_mfc_dec.c | 3 +++\n" + "> > > ? 1 file changed, 3 insertions(+)\n" "> > > \n" "> > > diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c\n" "> > > b/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c\n" @@ -63,9 +44,9 @@ "> > > +++ b/drivers/media/platform/s5p-mfc/s5p_mfc_dec.c\n" "> > > @@ -386,6 +386,9 @@ static int vidioc_try_fmt(struct file *file,\n" "> > > void *priv, struct v4l2_format *f)\n" - "> > > \302\240\302\240\tstruct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;\n" - "> > > \302\240\302\240\tstruct s5p_mfc_fmt *fmt;\n" - "> > > \302\240\302\240\n" + "> > > ??\tstruct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;\n" + "> > > ??\tstruct s5p_mfc_fmt *fmt;\n" + "> > > ??\n" "> > > +\tif (f->fmt.pix.field == V4L2_FIELD_ANY)\n" "> > > +\t\tf->fmt.pix.field = V4L2_FIELD_NONE;\n" "> > > +\n" @@ -76,13 +57,13 @@ "> > If the only supported format is none you should set 'field'\n" "> > unconditionally to NONE, nothing more.\n" "> \n" - "> Afaict we\302\240\302\240support 2 things:\n" + "> Afaict we??support 2 things:\n" "> \n" - "> \302\240\302\240\302\2401. NONE\n" - "> \302\240\302\240\302\2402. INTERLACE\n" + "> ???1. NONE\n" + "> ???2. INTERLACE\n" "> \n" "> Until now we were not checking what was supported or not and\n" - "> basically\302\240\n" + "> basically?\n" "> ignoring that info, this patch\n" "> keeps the old behaviour making sure to be compliant.\n" "> \n" @@ -100,7 +81,7 @@ "> ```\n" "> \n" "> instead, it is probably more correct, do you think it is what should\n" - "> be\302\240\n" + "> be?\n" "> done here?\n" "> \n" "> Regards,\n" @@ -111,21 +92,17 @@ "> > Regards\n" "> > Andrzej\n" "> > \n" - "> > > \302\240\302\240\tmfc_debug(2, \"Type is %d\\n\", f->type);\n" - "> > > \302\240\302\240\tif (f->type == V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE) {\n" - "> > > \302\240\302\240\t\tfmt = find_format(f, MFC_FMT_DEC);\n" + "> > > ??\tmfc_debug(2, \"Type is %d\\n\", f->type);\n" + "> > > ??\tif (f->type == V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE) {\n" + "> > > ??\t\tfmt = find_format(f, MFC_FMT_DEC);\n" "> \n" - > - "\01:2\0" - "fn\0signature.asc\0" - "d\0This is a digitally signed message part\0" - "b\0" - "-----BEGIN PGP SIGNATURE-----\n" - "Version: GnuPG v2\n" - "\n" - "iEYEABECAAYFAlito1EACgkQcVMCLawGqBy+tgCeN4UKQCQlZflnDAgybPdmgx0y\n" - "uksAoNNQBZGPedHd6Du/W5ySrwWGBc3B\n" - "=9TtN\n" - "-----END PGP SIGNATURE-----\n" + "> \n" + "-------------- next part --------------\n" + "A non-text attachment was scrubbed...\n" + "Name: signature.asc\n" + "Type: application/pgp-signature\n" + "Size: 181 bytes\n" + "Desc: This is a digitally signed message part\n" + URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170222/13a1ba63/attachment.sig> -93fad2f952e2a6c19111e3a2593923f2648497de4757644d20728285ce881ac7 +d5db5264539d294371bc4c70605f1783e737e19b0838a130998b6570de034fd8
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.