All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sakari Ailus <sakari.ailus@iki.fi>
To: Andreas Nagel <andreasnagel@gmx.net>
Cc: linux-media@vger.kernel.org
Subject: Re: How to configure resizer in ISP pipeline?
Date: Sun, 06 Jan 2013 17:05:04 +0200	[thread overview]
Message-ID: <50E992A0.6010007@iki.fi> (raw)
In-Reply-To: <50E8D6AA.1040305@gmx.net>

Hi Andreas,

Andreas Nagel wrote:
> Hi Sakari,
> 
> thanks for helping.
> 
>>> My "sensor" (TVP5146) already provides YUV data, so I can skip the
>>> previewer. I tried setting the input and output pad of the resizer
>>> subdevice to incoming resolution (input pad) and desired resolution
>>> (output pad). For example: 720x576 --> 352x288. But it didn't work
>>> out quite well.
>> How did it not work quite well? :)
> 
> Not sure, if I recall all the details. I haven't done much in this area
> for a few weeks now.
> Currently, I actually can configure the resizer, but then
> VIDIOC_STREAMON fails with EINVAL when I configure the devnode. Don't
> know why.
> I do connect the resizer source to the resizer output (devnode) and then
> capture from there. I think it is /dev/video6.
> If I leave the resizer out and connect the ccdc source to the ccdc
> output (/dev/video2), capturing works just fine.
> 
> One reason could be, that the resizer isn't supported right now. (You
> remember, I have to use Technexion's TI kernel 2.6.37 with its exotic
> ISP driver. ;-) )

Ouch. There have been many bugfixes and improvements in the omap3isp
driver since it was merged to mainline in this area. I don't know what
TI has backported to their own kernel; it could be something is missing
there.

The fact the ABI and API are incompatible with what's now in mainline
suggests that quite a lot may be missing.

> That's, what one could interpret from this TI wiki page.
> http://processors.wiki.ti.com/index.php/UserGuideOmap35xCaptureDriver_PSP_04.02.00.07#Architecture
> 
> Under the block diagram, there's a note saying, that only the path with
> the continuous line has been validated. So, the dotted lines are ISP
> paths that might not have been validated ("supported"?) yet.

That's very possible.

> (I might add, that all this is part of my master thesis and resizing
> would be a nice-to-have goal, but not a must-have. So I can live with it
> if it won't work.)

Good to hear that... I guess the best starting point to get things
working would definitely be the mainline kernel. The rest is at best
unsupported.

-- 
Kind regards,

Sakari Ailus
sakari.ailus@iki.fi

      reply	other threads:[~2013-01-06 15:01 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-11 14:48 How to configure resizer in ISP pipeline? Andreas Nagel
2013-01-06  1:03 ` Sakari Ailus
2013-01-06  1:43   ` Andreas Nagel
2013-01-06 15:05     ` Sakari Ailus [this message]

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=50E992A0.6010007@iki.fi \
    --to=sakari.ailus@iki.fi \
    --cc=andreasnagel@gmx.net \
    --cc=linux-media@vger.kernel.org \
    /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.