Subject: Re: [BK PATCH] add SMBIOS tables to sysfs
From: Michael Brown <Michael_E_Brown@Dell.com>
Reply-To: Michael_E_Brown@Dell.com
To: Greg KH <greg@kroah.com>
Cc: akpm@osdl.org, linux-kernel@vger.kernel.org
In-Reply-To: <20040428033020.GA14078@kroah.com>
References: <1083119269.1203.2821.camel@debian>
	 <20040428033020.GA14078@kroah.com>
Content-Type: text/plain
Organization: Dell Inc
Message-Id: <1083123515.1203.2826.camel@debian>
Mime-Version: 1.0
Date: Tue, 27 Apr 2004 22:38:35 -0500
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1208
Lines: 36

On Tue, 2004-04-27 at 22:30, Greg KH wrote:
> Nice idea and patch.  A few minor comments:

Thank you.

> 
> > /sys/firmware/smbios/smbios/table_entry_point
> > /sys/firmware/smbios/smbios/table
> 
> Why repeat the "smbios" directory?  Is this a limitation in the sysfs
> interface right now?  Or are you going to put more files in the main
> smbios directory some day?

>From what I could tell (please correct me if I am wrong), you can add
only subsystems to the firmware directory. The first "smbios" is for the
subsystem. Then, you can only add devices to the subsystem. The second
"smbios" is for the device.

I really would like to get rid of one myself, but it was not obvious to
me how to do this. No, I do not plan on adding new objects in there.

> 
> > +	snprintf(sdev->kobj.name, 7, "smbios" );
> 
> Try using kobject_set_name() instead, it will do the proper thing if the
> string is bigger than the base kobj.name field.

Ok, thanks.
Thanks,
Michael

-
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/