All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
To: Simon Horman <horms@verge.net.au>
Cc: Yoshihiro Kaneko <ykaneko0929@gmail.com>,
	linux-media@vger.kernel.org,
	Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
	Magnus Damm <magnus.damm@gmail.com>,
	linux-sh@vger.kernel.org
Subject: Re: [PATCH 3/3] media: soc_camera: rcar_vin: Add NV16 horizontal scaling-up support
Date: Wed, 15 Oct 2014 14:40:06 +0000	[thread overview]
Message-ID: <543E8746.80809@cogentembedded.com> (raw)
In-Reply-To: <20141015045213.GA18646@verge.net.au>

Hello.

On 10/15/2014 08:52 AM, Simon Horman wrote:

>>> From: Koji Matsuoka <koji.matsuoka.xm@renesas.com>

>>> The scaling function had been forbidden for the capture format of
>>> NV16 until now. With this patch, a horizontal scaling-up function
>>> is supported to the capture format of NV16. a vertical scaling-up
>>> by the capture format of NV16 is forbidden for the H/W specification.

>>     s/for/by/?

> How about the following text?

> Up until now scaling has been forbidden for the NV16 capture format. This
> patch adds support for horizontal scaling-up for NV16. Vertical scaling-up
> for NV16 is forbidden for by the H/W specification.

    "For by", hehe? Were you trying to keep every happy? :-)

WBR, Sergei


WARNING: multiple messages have this Message-ID (diff)
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
To: Simon Horman <horms@verge.net.au>
Cc: Yoshihiro Kaneko <ykaneko0929@gmail.com>,
	linux-media@vger.kernel.org,
	Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
	Magnus Damm <magnus.damm@gmail.com>,
	linux-sh@vger.kernel.org
Subject: Re: [PATCH 3/3] media: soc_camera: rcar_vin: Add NV16 horizontal scaling-up support
Date: Wed, 15 Oct 2014 18:40:06 +0400	[thread overview]
Message-ID: <543E8746.80809@cogentembedded.com> (raw)
In-Reply-To: <20141015045213.GA18646@verge.net.au>

Hello.

On 10/15/2014 08:52 AM, Simon Horman wrote:

>>> From: Koji Matsuoka <koji.matsuoka.xm@renesas.com>

>>> The scaling function had been forbidden for the capture format of
>>> NV16 until now. With this patch, a horizontal scaling-up function
>>> is supported to the capture format of NV16. a vertical scaling-up
>>> by the capture format of NV16 is forbidden for the H/W specification.

>>     s/for/by/?

> How about the following text?

> Up until now scaling has been forbidden for the NV16 capture format. This
> patch adds support for horizontal scaling-up for NV16. Vertical scaling-up
> for NV16 is forbidden for by the H/W specification.

    "For by", hehe? Were you trying to keep every happy? :-)

WBR, Sergei


  reply	other threads:[~2014-10-15 14:40 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-14  6:26 [PATCH 0/3] media: soc_camera: rcar_vin: Add scaling support Yoshihiro Kaneko
2014-10-14  6:26 ` Yoshihiro Kaneko
2014-10-14  6:26 ` [PATCH 1/3] " Yoshihiro Kaneko
2014-10-14  6:26   ` Yoshihiro Kaneko
2014-10-14 19:25   ` Geert Uytterhoeven
2014-10-14 19:25     ` Geert Uytterhoeven
2014-10-15  9:42     ` Yoshihiro Kaneko
2014-10-15  9:42       ` Yoshihiro Kaneko
2014-10-14  6:26 ` [PATCH 2/3] media: soc_camera: rcar_vin: Add capture width check for NV16 format Yoshihiro Kaneko
2014-10-14  6:26   ` Yoshihiro Kaneko
2014-10-14  6:26 ` [PATCH 3/3] media: soc_camera: rcar_vin: Add NV16 horizontal scaling-up support Yoshihiro Kaneko
2014-10-14  6:26   ` Yoshihiro Kaneko
2014-10-14 12:57   ` Sergei Shtylyov
2014-10-14 12:57     ` Sergei Shtylyov
2014-10-15  4:52     ` Simon Horman
2014-10-15  4:52       ` Simon Horman
2014-10-15 14:40       ` Sergei Shtylyov [this message]
2014-10-15 14:40         ` Sergei Shtylyov
2014-10-16  4:55         ` Simon Horman
2014-10-16  4:55           ` Simon Horman
2014-10-16  5:18           ` Yoshihiro Kaneko
2014-10-16  5:18             ` Yoshihiro Kaneko
2014-10-15  9:41     ` Yoshihiro Kaneko
2014-10-15  9:41       ` Yoshihiro Kaneko

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=543E8746.80809@cogentembedded.com \
    --to=sergei.shtylyov@cogentembedded.com \
    --cc=g.liakhovetski@gmx.de \
    --cc=horms@verge.net.au \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-sh@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=ykaneko0929@gmail.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 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.