All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ryan Mallon <rmallon@gmail.com>
To: "Murali K. Vemuri" <vemuri.muralikrishna@gmail.com>
Cc: Greg KH <greg@kroah.com>, linux-kernel@vger.kernel.org
Subject: Re: kernel panic with simple driver
Date: Wed, 10 Aug 2011 12:35:24 +1000	[thread overview]
Message-ID: <4E41EE6C.9000200@gmail.com> (raw)
In-Reply-To: <CAME+iucHOG+cvB2y9uLxwtLtas+E0nEfekgmN7gfj7YsT3kj+g@mail.gmail.com>

On 10/08/11 12:29, Murali K. Vemuri wrote:
> On Wed, Aug 10, 2011 at 11:16 AM, Greg KH<greg@kroah.com>  wrote:
>> On Wed, Aug 10, 2011 at 10:33:28AM +0900, Murali K. Vemuri wrote:
>>> There is no concurrent access to the timer. The design is that:
>>> 1.Driver provides an IOCTL for start / stop
>>> 2. when the driver receives START IOCTL, it toggles some GPIOs to ON / OFF.
>>> 3. the GPIOs will be ON for 500 MSec and OFF for 500 MSec.
>>> 4. Two successive START IOCTLs will not be honored.
>>> 5. There is only one application that uses these IOCTLs
>>> 6. When I receive a STOP IOCTL, I am doing :
>>> if (timer_pending (&my_timer))
>>> del_timer(&my_timer);
>> What kind of driver is this?  For what type of hardware?
>>
>> Can't you control the gpios from userspace with out any need to write a
>> kernel driver?
>>
> This driver is meant for controlling some LEDs. The CPU is OMAP 3530
> and the OS is Android.
>  From the user space, I could not control the GPIOs directly, and thus
> I ended up supporting in the form of a simple driver.
> I agree that these are better done from the user space, but as much as
> I google'd studied, I could not find any better way to implement this.
>
> If anyone has more info, that is also highly appreciated.

If you have CONFIG_GPIO_SYSFS enabled then you can access the gpios 
directly via sysfs. See Documentation/gpio.txt for details.

~Ryan


      reply	other threads:[~2011-08-10  2:36 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-10  0:16 kernel panic with simple driver Murali K. Vemuri
2011-08-10  0:42 ` Ryan Mallon
2011-08-10  1:33   ` Murali K. Vemuri
2011-08-10  2:16     ` Greg KH
2011-08-10  2:29       ` Murali K. Vemuri
2011-08-10  2:35         ` Ryan Mallon [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=4E41EE6C.9000200@gmail.com \
    --to=rmallon@gmail.com \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=vemuri.muralikrishna@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.