All of lore.kernel.org
 help / color / mirror / Atom feed
From: JD Louw <jd.louw@mweb.co.za>
To: Theunis Potgieter <theunis.potgieter@gmail.com>
Cc: linux-media@vger.kernel.org
Subject: Re: [PATCH] Compro S350 GPIO change
Date: Mon, 18 Jan 2010 21:42:50 +0200	[thread overview]
Message-ID: <1263843770.3042.11.camel@Core2Duo> (raw)
In-Reply-To: <23582ca1001172352k21d5ceua2067b7eba020369@mail.gmail.com>

On Mon, 2010-01-18 at 09:52 +0200, Theunis Potgieter wrote:
> 2010/1/17 JD Louw <jd.louw@mweb.co.za>:
> > Hi,
> >
> > This patch enables LNB power on newer revision d1 Compro S350 and S300
> > DVB-S cards. While I don't have these cards to test with I'm confident
> > that this works. See
> > http://thread.gmane.org/gmane.linux.drivers.video-input-infrastructure/7471 and http://thread.gmane.org/gmane.linux.drivers.video-input-infrastructure/14296
> > and new windows driver as reference.
> >
> > Signed-off-by: JD Louw <jd.louw@mweb.co.za>
> >
> > diff -r 59e746a1c5d1 linux/drivers/media/video/saa7134/saa7134-cards.c
> > --- a/linux/drivers/media/video/saa7134/saa7134-cards.c Wed Dec 30
> > 09:10:33 2009 -0200
> > +++ b/linux/drivers/media/video/saa7134/saa7134-cards.c Sun Jan 17
> > 14:51:07 2010 +0200
> > @@ -7037,8 +7037,8 @@ int saa7134_board_init1(struct saa7134_d
> >                break;
> >        case SAA7134_BOARD_VIDEOMATE_S350:
> >                dev->has_remote = SAA7134_REMOTE_GPIO;
> > -               saa_andorl(SAA7134_GPIO_GPMODE0 >> 2,   0x00008000, 0x00008000);
> > -               saa_andorl(SAA7134_GPIO_GPSTATUS0 >> 2, 0x00008000, 0x00008000);
> > +               saa_andorl(SAA7134_GPIO_GPMODE0 >> 2,   0x0000C000, 0x0000C000);
> > +               saa_andorl(SAA7134_GPIO_GPSTATUS0 >> 2, 0x0000C000, 0x0000C000);
> >                break;
> >        }
> >        return 0;
> >
> >
> > --
> Hi Jan,
> 
> This does not fix the problem where the card is suppose to suspend and
> the Voltage drops to 0V? Do you still require the windows registry
> reference for this part?

Hi,

No, the aim of the patch is just to get the basic driver support for all
S300/S350 revisions. A windows regspy capture on a d1 revision card may
still be useful in double checking the GPIO values, but I already know
which GPIO controls LNB voltage:

GPIO15 = modulator reset
GPIO14 = LNB power



      reply	other threads:[~2010-01-18 19:43 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-17 12:57 [PATCH] Compro S350 GPIO change JD Louw
2010-01-18  7:52 ` Theunis Potgieter
2010-01-18 19:42   ` JD Louw [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=1263843770.3042.11.camel@Core2Duo \
    --to=jd.louw@mweb.co.za \
    --cc=linux-media@vger.kernel.org \
    --cc=theunis.potgieter@gmail.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.