From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Subject: Re: [PATCH v6] ACPI / serial: Add peripheral PnP IDs enumeration support Date: Thu, 4 Apr 2013 11:23:35 -0700 Message-ID: <20130404182335.GC24711@kroah.com> References: <20130403161431.GB21383@kroah.com> <1AE640813FDE7649BE1B193DEA596E88015CC891@SHSMSX101.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1AE640813FDE7649BE1B193DEA596E88015CC891@SHSMSX101.ccr.corp.intel.com> Sender: linux-serial-owner@vger.kernel.org To: "Zheng, Lv" Cc: "Wysocki, Rafael J" , "Brown, Len" , "linux-acpi@vger.kernel.org" , "linux-serial@vger.kernel.org" , "Zhang, Rui" List-Id: linux-acpi@vger.kernel.org On Thu, Apr 04, 2013 at 03:43:43PM +0000, Zheng, Lv wrote: > Hi, > > > > > + return snprintf(buf, PAGE_SIZE, "uart\n"); > > > > > > What is the "uart" part for? Isn't that kind of obvious? > > > > It is the default "peripheral_type" content. > > Before sending next revision out, let me ask a question here: > > Shall I leave this empty as my previous revisions? > > Hmm, if this is empty by default then the udev rules may require additional handlings. > Can I use "NONE" here for the user space users? No, just don't create the sysfs file at all for that type of device. greg k-h