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
next prev parent 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