All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Andrey Utkin <andrey.utkin@corp.bluecherry.net>,
	Andrey Utkin <andrey.krieger.utkin@gmail.com>
Cc: "Krzysztof Hałasa" <khalasa@piap.pl>,
	"hans.verkuil" <hans.verkuil@cisco.com>,
	"Linux Media" <linux-media@vger.kernel.org>
Subject: Re: [RFC] solo6x10 freeze, even with Oct 31's linux-next... any ideas or help?
Date: Sat, 15 Nov 2014 15:08:05 +0100	[thread overview]
Message-ID: <54675E45.8020603@xs4all.nl> (raw)
In-Reply-To: <CAM_ZknUoNBfnKJW-76FE1tW29O6oFAw+KDYPsViTLw7u-vFXuw@mail.gmail.com>

Hi Andrey,

On 11/15/2014 02:48 PM, Andrey Utkin wrote:
> Thanks to all for the great help so far, but I've got another issue
> with upstream driver.
> 
> In upstream there's no more module parameter for video standard
> (NTSC/PAL). But there's VIDIOC_S_STD handling procedure. But it turns
> out not to work correctly: the frame is offset, so that in the bottom
> there's black horizontal bar.
> The S_STD ioctl call actually makes difference, because without that
> the frame "slides" vertically all the time. But after the call the
> picture is not correct.

That's strange. I know I tested it at the time. I assume it is the PAL
standard that isn't working (as opposed to NTSC)? Or does it just always
fail when you switch between the two standards?

> 
> Such change didn't help:
> https://github.com/krieger-od/linux/commit/55b796c010b622430cb85f5b8d7d14fef6f04fb4
> So, temporarily, I've hardcoded this for exact customer who uses PAL:
> https://github.com/krieger-od/linux/commit/2c26302dfa6d7aa74cf17a89793daecbb89ae93a
> rmmod/modprobe cycle works fine and doesn't make any difference from
> reboot, but still it works correctly only with PAL hardcoded for the
> first-time initialization.
> 
> Any ideas why wouldn't it work to change the mode after the driver load?

Not really. I will have to test this next week (either Monday or Friday) with
my solo board.

> Would it be allowed to add back that kernel module parameter (the one
> passed at module load time)?

No. That's a hack, the S_STD call should just work and we need to figure out
why it fails.

Regards,

	Hans

  reply	other threads:[~2014-11-15 14:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-11 17:46 [RFC] solo6x10 freeze, even with Oct 31's linux-next... any ideas or help? Andrey Utkin
2014-11-11 18:05 ` Hans Verkuil
2014-11-11 18:16   ` Andrey Utkin
2014-11-14  7:29     ` Andrey Utkin
2014-11-27  5:45   ` Ismael Luceno
2014-11-14 11:00 ` Krzysztof Hałasa
2014-11-14 11:42   ` Andrey Utkin
2014-11-15 13:48     ` Andrey Utkin
2014-11-15 14:08       ` Hans Verkuil [this message]
2014-11-15 14:23         ` Andrey Utkin
2014-11-15 20:42       ` Krzysztof Hałasa
2014-11-15 22:12         ` Andrey Utkin
2014-11-27  5:49         ` Ismael Luceno

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=54675E45.8020603@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=andrey.krieger.utkin@gmail.com \
    --cc=andrey.utkin@corp.bluecherry.net \
    --cc=hans.verkuil@cisco.com \
    --cc=khalasa@piap.pl \
    --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.