All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andy Walls <awalls@radix.net>
To: linux-media@vger.kernel.org, stoth@kernellabs.com
Cc: mkrufky@kernellabs.com, hverkuil@xs4all.nl
Subject: cx25840: GPIO settings wrong for HVR-1850 IR Tx
Date: Fri, 27 Nov 2009 16:10:32 -0500	[thread overview]
Message-ID: <1259356232.2353.13.camel@localhost> (raw)

Steve,

This code in cx25840-core.c mucks up HVR-1850 IR Tx:

        /* Drive GPIO2 direction and values for HVR1700
         * where an onboard mux selects the output of demodulator
         * vs the 417. Failure to set this results in no DTV.
         * It's safe to set this across all Hauppauge boards
         * currently, regardless of the board type.
         */
        cx25840_write(client, 0x160, 0x1d);
        cx25840_write(client, 0x164, 0x00);

This changes the IR_TX pin to act as GPIO_20 and defaults it to low,
keeping the IR Transmitter LED illuminated.

Setting register 0x160 to 0x1f makes the pin the IR Tx function again.
I don't want to hack it in there though, as I don't know the
implications for other CX2388[578] boards.  I also perceive the cx23885
module GPIOs could be an easy place to introduce regressions if not
careful.


Steve and Hans,

Any ideas?

I know on the list I had bantered around a configure, enable, set, get
etc v4l2_subdev ops for gpio, but I can't remember the details nor the
requirements.

The cx25840 module really needs a way for the cx23885 bridge driver to
set GPIOs cleanly.

Regards,
Andy


             reply	other threads:[~2009-11-27 21:11 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-27 21:10 Andy Walls [this message]
2009-11-28  6:35 ` cx25840: GPIO settings wrong for HVR-1850 IR Tx Andy Walls
2009-11-28 13:52   ` Steven Toth

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=1259356232.2353.13.camel@localhost \
    --to=awalls@radix.net \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=mkrufky@kernellabs.com \
    --cc=stoth@kernellabs.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.