All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rune Petersen <rune@megahurts.dk>
To: linux-media@vger.kernel.org
Subject: Some questions timeout in rc_dev
Date: Sun, 16 Feb 2014 22:54:01 +0100	[thread overview]
Message-ID: <53013379.70403@megahurts.dk> (raw)

Hi,


The intent of the timeout member in the rc_dev struct is a little unclear to me.
In rc-core.h it is described as:
	@timeout: optional time after which device stops sending data.

But if I look at the usage, it is used to detect idle in ir_raw.c which again is 
used by the RC-6 decoder to detect end of RC-6 6A transmissions.

This leaves me with a few questions:
  - Without the timeout (which is optional) the RC-6 decoder will not work
    properly with RC-6 6A transmissions wouldn't that make it required?

  - Why are the timeout set in the individual drivers so varied, shouldn't it
    depend on the encoding rather then the hardware used?
    The timeout set in the drivers ranges from 2750us(redrat3)
    to 1000000us(fintek_cir) and all the way to weird(streamzap)

  - Why is the timeout value controlled by the IR driver, when it us only
    used	by the rc-core.
    Wouldn't it make sense to have the timeout initialized to a sane value
    in a single place?


I would like to get rc to a state where it just works for me without 
modifications, I "just" need to know which changes I can get away without 
breaking it for everybody else =)

As things are right now the RC input feel very sluggish and unresponsive using a 
RC-6 6A remote and a ite-cir receiver.


Rune

             reply	other threads:[~2014-02-16 22:00 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-16 21:54 Rune Petersen [this message]
2014-02-18 14:02 ` Some questions timeout in rc_dev Sean Young
2014-02-18 23:44   ` Rune Petersen
2014-02-19 13:58   ` Mauro Carvalho Chehab

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=53013379.70403@megahurts.dk \
    --to=rune@megahurts.dk \
    --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.