-----Original Message----- From: Greg KH gregkh@linuxfoundation.org Sent: Thursday, October 1, 2020 4:02 AM To: Ertman, David M david.m.ertman@intel.com Cc: alsa-devel@alsa-project.org; tiwai@suse.de; broonie@kernel.org; pierre- louis.bossart@linux.intel.com; Sridharan, Ranjani ranjani.sridharan@intel.com; jgg@nvidia.com; parav@nvidia.com; Patil, Kiran kiran.patil@intel.com; Ranjani Sridharan ranjani.sridharan@linux.intel.com; Fred Oh fred.oh@linux.intel.com; Saleem, Shiraz shiraz.saleem@intel.com; Parav Pandit parav@mellanox.com; Williams, Dan J dan.j.williams@intel.com Subject: Re: [PATCH 1/6] Add ancillary bus support
On Wed, Sep 30, 2020 at 03:50:46PM -0700, Dave Ertman wrote:
diff --git a/drivers/bus/Makefile b/drivers/bus/Makefile index 397e35392bff..1fd238094543 100644 --- a/drivers/bus/Makefile +++ b/drivers/bus/Makefile @@ -3,6 +3,9 @@ # Makefile for the bus drivers. #
+#Ancillary bus driver
Nit, you need a space :)
Fixed for next version 😊
+obj-$(CONFIG_ANCILLARY_BUS) += ancillary.o
And why not put this in drivers/base/? Why way over here in drivers/bus?
My only reason was that this is a bus, and I wagered that buses should go into the bus subdirectory. 😊
thanks,
greg k-h
-DaveE