Re: [PATCH] 2.5.31 driverfs: patch for your consideration

From: Adam Belay (ambx1@netscape.net)
Date: Sat Aug 17 2002 - 17:32:30 EST


greg@kroah.com wrote:

>On Sat, Aug 17, 2002 at 02:10:34PM +0000, Adam Belay wrote:
>
>>
>>greg@kroah.com wrote:
>>
>>>Um, your email client mangled the patch, dropping tabs and wrapped
>>>lines.
>>>
>>Thanks for pointing this out. I'll send it as an attachment this time.
>> My current client has been causing me a lot of trouble, is there one
>>you would suggest I use?
>>
>
>I like mutt, but that's just my opinion :)
>
>Hm, in looking at your attachments, they will not apply either. All the
>tabs are gone, something's wrong with your originals. Did you cut and
>paste to generate them?
>
I downloaded my patches through the mailing list and applied them:

bash-2.05a$ cat ./driver.patch | patch -p1 -l -d linux
patching file drivers/base/interface.c
bash-2.05a$ cat ./driver2.patch | patch -p1 -l -d linux
patching file drivers/base/base.h
patching file drivers/base/core.c
patching file drivers/base/interface.c

It applies cleanly but . . .

You're right the tabs are gone although when I applied my originals they
weren't. I hate netscape navigator. I gzipped them so netscape can't
mess them up. In the meantime I'm going to download mutt. Thanks for
your help. Let me know if the patch works this time. Also after
looking at the interface code I realized that not just my code used
sprintf. Do you think they should all use snprintf instead or is the
probability of a driver attribute exceeding the one page buffer size so
low that it doesn't matter?

Also I was wondering if you think resource management variables (irq,
io, dma, etc) should live in the device structure like power management
variables do? Global resource management seams interesting to me,
although there already is a proc interface that does list resources, I'm
wondering if the driver model is a good place to put such an interface?

Thanks,
-Adam



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Aug 23 2002 - 22:00:14 EST