public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Kumba <kumba@gentoo.org>
To: Greg KH <greg@kroah.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Can I nest kobjects to create directories under parent directories for my driver?
Date: Wed, 02 Mar 2011 16:49:14 -0500	[thread overview]
Message-ID: <4D6EBB5A.7080307@gentoo.org> (raw)
In-Reply-To: <20110302203823.GA18879@kroah.com>

On 03/02/2011 15:38, Greg KH wrote:
>
> That's nice, but it really isn't a good idea to do things this deep.
> Userspace tools will not be able to find things properly (like libudev)
> and your kernel code will get very messy (as you have found out.)
>
> Why not look at how the other rtc drivers work, they don't nest things
> this deep and you need to sick with the standard userspace api and not
> invent a new one if at all possible.
>
> But to answer your question, you can do this, but I strongly discourage
> it.

 From the way sysfs seems to work, I can see that it encourages as little 
nesting as possible.  So this is probably the way I will go and just call the 
entries "regs_ctrla" through "regs_ctrl4b".  It still conveys the relevant 
information and is easily doable through the existing userspace framework.

How does one nest kobjects, though?  Are ksets the only real way?  Or is it 
doable through linking kobjects to each other?  If the latter is possible, how 
does one keep the reference to the device structure intact?  I figure the right 
use of backcasting (through container_of) is one way, but it definitely seems 
like I am treading new territory with such an approach.

I did learn one useful thing through this -- the use of a common show/store 
function for each register (using strncmp to look up which bit was accessed).  I 
was originally doing a single show/store pair for each, which added major code 
bloat and resulted in a ~80kb+ module.

Thanks for the feedback!  Thought my message had gotten lost!

-- 
Joshua Kinard
Gentoo/MIPS
kumba@gentoo.org

"The past tempts us, the present confuses us, the future frightens us.  And our 
lives slip away, moment by moment, lost in that vast, terrible in-between."

--Emperor Turhan, Centauri Republic

  reply	other threads:[~2011-03-02 21:49 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-01  3:31 Can I nest kobjects to create directories under parent directories for my driver? Kumba
2011-03-02 20:38 ` Greg KH
2011-03-02 21:49   ` Kumba [this message]
2011-03-03  6:19     ` Greg KH

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=4D6EBB5A.7080307@gentoo.org \
    --to=kumba@gentoo.org \
    --cc=greg@kroah.com \
    --cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox