All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Walls <awalls@md.metrocast.net>
To: Jarod Wilson <jarod@wilsonet.com>
Cc: Jean Delvare <khali@linux-fr.org>,
	Linux Media Mailing List <linux-media@vger.kernel.org>,
	Janne Grunau <j@jannau.net>, Jarod Wilson <jarod@redhat.com>
Subject: Re: [PATCH] hdpvr: enable IR part
Date: Sat, 15 Jan 2011 10:26:24 -0500	[thread overview]
Message-ID: <1295105185.3258.32.camel@localhost> (raw)
In-Reply-To: <cwd2gkgtgyb91bkc0m1dtmnx.1295095844198@email.android.com>

On Sat, 2011-01-15 at 07:50 -0500, Andy Walls wrote:
> Jarod Wilson <jarod@wilsonet.com> wrote:

> >Forgot to mention: I think it was suggested that one could use ir-kbd-i2c
> >for receive and lirc_zilog for transmit, at the same time. With ir-kbd-i2c
> >already loaded, lirc_zilog currently won't bind to anything.


> With my newly hacked lirc_zilog, try using the 'tx_only' parameter
> please.  It's not quite ready yet, but I'd like to know if it can
> bind.

I have now tested this.

Using the 'tx_only' module parameter to lirc_zilog appears to allow
ir-kbd-i2c and lirc_zilog to coexist, for I2C subsystem binding at
least.

It does not appear to matter what order the two modules are loaded. I
tried it both ways.


However, lirc_zilog sharing of Z8 is not fully functional yet.  I need
to change things to have the bridge drivers provide a IR transceiver
mutex to both lirc_zilog and ir-kbd-i2c.  lirc_zilog and ir-kbd-i2c
would use that mutex for exclusive access to the Z8 when needed, if it
was provided by the bridge driver.

I view proper sharing of the Z8 as an important requirement, because of
two use cases:

1. User only wants to use the Z8 for IR Rx.  User doesn't want to fetch
the lirc_zilog required firmware or perform any LIRC setup.

2. User only wants to use the Z8 for IR Tx.  User uses some other
ir-kbd-i2c supported receiver and remote IR Rx.

Maybe use case #2 is too rare to worry about?
However, if one accepts both use cases as valid, then ir-kbd-i2c must
support the Z8, and lirc_zilog must be able to coexist with ir-kbd-i2c.

Proper sharing of the Z8 is, however, lower on my to-do list than fixing
some internal lirc_zilog problems.

Regards,
Andy


  reply	other threads:[~2011-01-15 15:26 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-15 12:50 [PATCH] hdpvr: enable IR part Andy Walls
2011-01-15 15:26 ` Andy Walls [this message]
  -- strict thread matches above, loose matches on Subject: below --
2011-01-14 19:54 Jarod Wilson
2011-01-14 21:44 ` Andy Walls
2011-01-14 22:08   ` Jarod Wilson
2011-01-15  2:30     ` Jarod Wilson
2011-01-15  4:35       ` Andy Walls
2011-01-15  5:37         ` Jarod Wilson
2011-01-15  5:53           ` Jarod Wilson
2011-01-15  6:56           ` Jarod Wilson
2011-01-15 21:56             ` Andy Walls
2011-01-15 22:10               ` Jarod Wilson
2011-01-15 14:41           ` Andy Walls
2011-01-15 21:46           ` Andy Walls

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=1295105185.3258.32.camel@localhost \
    --to=awalls@md.metrocast.net \
    --cc=j@jannau.net \
    --cc=jarod@redhat.com \
    --cc=jarod@wilsonet.com \
    --cc=khali@linux-fr.org \
    --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.