All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bill Gatliff <bgat@billgatliff.com>
To: Mike Frysinger <vapier.adi@gmail.com>
Cc: linux-embedded@vger.kernel.org
Subject: Re: [RFC 0/6] Proposal for a Generic PWM Device API
Date: Wed, 08 Oct 2008 22:46:22 -0500	[thread overview]
Message-ID: <48ED7E8E.3010504@billgatliff.com> (raw)
In-Reply-To: <8bd0f97a0810081932t7e3ce7e4l785a3a841228aba2@mail.gmail.com>

Mike Frysinger wrote:
> 
> the Blackfin timers/pwm's can flip between input and ouput based on
> the configuration register.  everything else (pin/etc...) is
> unchanged.

Interesting!

>> Are you proposing that the API accommodate both input and output devices?
> 
> i dont think we should preclude it from the outset.

I don't think have a problem with that.  At some point, I would need someone
with input/output hardware to test the input-specific parts, however.  Hint,
hint.  ;)

> not really, but i see the existing code you've posted could already
> utilize some of the "get" functions ...

Which parts?  I don't really like keeping the period_ticks and duty_ticks values
around, but in my case I have no choice--- unless I were to read the CPRE, CPRD
and CDTY values from the hardware directly.  Which could be what your proposed
"get" methods would do.

But that still isn't PWM "input" the way you are describing, because my hardware
wouldn't be _reading_ an incoming PWM: it would be just reporting back the
values that it was using to produce its output signal.  Huge difference.

I dunno, I'm still not sure that measuring the characteristics of an incoming
PWM signal isn't a different kind of device from one that produces PWM signals,
at least conceptually.  Which, in a way, makes it out of scope for the proposed
API.  Not saying I can't go along with the idea, I'm just not sure what a user
would expect to happen if they called pwm_get_period_ns() on the SAM9263 PWMC
device.  They certainly aren't going to get a measured value in return!



b.g.
-- 
Bill Gatliff
bgat@billgatliff.com

  reply	other threads:[~2008-10-09  3:46 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-08 16:43 [RFC 0/6] Proposal for a Generic PWM Device API Bill Gatliff
2008-10-08 16:43 ` [RFC 1/6] [PWM] Generic PWM API implementation Bill Gatliff
2008-10-09  8:17   ` Marc Pignat
2008-10-10  3:20   ` Benjamin Herrenschmidt
2008-10-10  4:07     ` Bill Gatliff
2008-10-10 14:07     ` Bill Gatliff
2008-10-13 16:04       ` Scott Wood
2008-10-08 16:43 ` Bill Gatliff
2008-10-08 16:43 ` [RFC 2/6] [PWM] Changes to existing include/linux/pwm.h to adapt to generic PWM API Bill Gatliff
2008-10-08 16:43 ` Bill Gatliff
2008-10-08 16:43 ` [RFC 3/6] [PWM] Documentation Bill Gatliff
2008-10-08 16:43 ` [RFC 4/6] [PWM] Driver for Atmel PWMC peripheral Bill Gatliff
2008-10-08 16:43 ` [RFC 3/6] [PWM] Documentation Bill Gatliff
2008-10-08 16:43 ` [RFC 5/6] [PWM] Install new Atmel PWMC driver in Kconfig, expunge old one Bill Gatliff
2008-10-09  5:21   ` Hans-Christian Egtvedt
2008-10-09 12:16     ` Haavard Skinnemoen
2008-10-09 12:17       ` Hans-Christian Egtvedt
2008-10-09 14:04         ` Bill Gatliff
2008-10-09 13:40     ` Bill Gatliff
2008-10-09 13:44       ` Hans-Christian Egtvedt
2008-10-08 16:43 ` [RFC 4/6] [PWM] Driver for Atmel PWMC peripheral Bill Gatliff
2008-10-08 16:43 ` [RFC 6/6] [PWM] New LED driver and trigger that use PWM API Bill Gatliff
2008-10-08 16:43 ` [RFC 5/6] [PWM] Install new Atmel PWMC driver in Kconfig, expunge old one Bill Gatliff
2008-10-08 16:43 ` [RFC 6/6] [PWM] New LED driver and trigger that use PWM API Bill Gatliff
2008-10-08 19:27 ` [RFC 0/6] Proposal for a Generic PWM Device API Mike Frysinger
2008-10-09  2:23   ` Bill Gatliff
2008-10-09  2:29     ` Bill Gatliff
2008-10-09  2:32     ` Mike Frysinger
2008-10-09  3:46       ` Bill Gatliff [this message]
2008-10-09  4:05         ` Mike Frysinger
2008-10-09  4:18           ` Bill Gatliff
2008-10-09  4:33             ` Mike Frysinger
2008-10-09 21:08 ` Matt Sealey
2008-10-09 21:29   ` Bill Gatliff
2008-10-10  3:20 ` Benjamin Herrenschmidt
2008-10-10  4:06   ` Bill Gatliff
2008-10-10  5:02     ` Benjamin Herrenschmidt
2008-10-10  5:06       ` Jon Smirl
2008-10-10 14:04         ` Bill Gatliff
2008-10-10 14:12           ` Jon Smirl
2008-10-10 20:45           ` Jon Loeliger
2008-10-12  2:32             ` Matt Sealey
2008-10-10  9:00     ` Geert Uytterhoeven
2008-10-10  9:00       ` Geert Uytterhoeven
2008-10-10  9:36       ` Paul Mundt
2008-10-10  9:36         ` Paul Mundt
2008-10-10  9:46         ` David Woodhouse
2008-10-10  9:46           ` David Woodhouse
2008-10-10 13:59           ` Bill Gatliff
2008-10-10 13:59             ` Bill Gatliff
2008-10-10 14:03         ` Bill Gatliff
2008-10-10 14:03           ` Bill Gatliff
2008-10-10 14:32           ` Haavard Skinnemoen
2008-10-10 14:32             ` Haavard Skinnemoen
2008-10-10 17:28           ` Paul Mundt
2008-10-10 19:15             ` Bill Gatliff
2008-10-10 19:15               ` Bill Gatliff
2008-10-10 13:59       ` Bill Gatliff
2008-10-10 13:59         ` Bill Gatliff
2008-10-10 17:40         ` Paul Mundt
2008-10-10 17:40           ` Paul Mundt
2008-10-10 19:42           ` Bill Gatliff
2008-10-13  7:40             ` Geert Uytterhoeven
2008-10-13  7:40               ` Geert Uytterhoeven
  -- strict thread matches above, loose matches on Subject: below --
2008-10-08 16:43 Bill Gatliff

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=48ED7E8E.3010504@billgatliff.com \
    --to=bgat@billgatliff.com \
    --cc=linux-embedded@vger.kernel.org \
    --cc=vapier.adi@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.