From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregkh@linuxfoundation.org (Greg Kroah-Hartman) Date: Wed, 6 Jun 2018 10:25:31 +0200 Subject: [PATCH v4 01/14] coresight: cpu_debug: minor module fixups In-Reply-To: <20180605210710.22227-2-kim.phillips@arm.com> References: <20180605210710.22227-1-kim.phillips@arm.com> <20180605210710.22227-2-kim.phillips@arm.com> Message-ID: <20180606082531.GC19727@kroah.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jun 05, 2018 at 04:06:57PM -0500, Kim Phillips wrote: > - provide the name of the module in the Kconfig help section > - define a MODULE_DEVICE_TABLE in order to be autoloaded on boot > > Cc: Mathieu Poirier > Cc: Leo Yan > Cc: Alexander Shishkin > Cc: Randy Dunlap > Cc: Suzuki K Poulose > Cc: Greg Kroah-Hartman > Cc: Russell King > Signed-off-by: Kim Phillips > --- > drivers/hwtracing/coresight/Kconfig | 3 +++ > drivers/hwtracing/coresight/coresight-cpu-debug.c | 2 ++ > 2 files changed, 5 insertions(+) > > diff --git a/drivers/hwtracing/coresight/Kconfig b/drivers/hwtracing/coresight/Kconfig > index ef9cb3c164e1..d1209f5acf76 100644 > --- a/drivers/hwtracing/coresight/Kconfig > +++ b/drivers/hwtracing/coresight/Kconfig > @@ -103,4 +103,7 @@ config CORESIGHT_CPU_DEBUG > properly, please refer Documentation/trace/coresight-cpu-debug.txt > for detailed description and the example for usage. > > + To compile this driver as a module, choose M here: the > + module will be called coresight-cpu-debug. > + > endif > diff --git a/drivers/hwtracing/coresight/coresight-cpu-debug.c b/drivers/hwtracing/coresight/coresight-cpu-debug.c > index 45b2460f3166..1efe9626eb6c 100644 > --- a/drivers/hwtracing/coresight/coresight-cpu-debug.c > +++ b/drivers/hwtracing/coresight/coresight-cpu-debug.c > @@ -671,6 +671,8 @@ static const struct amba_id debug_ids[] = { > { 0, 0 }, > }; > > +MODULE_DEVICE_TABLE(amba, debug_ids); > + > static struct amba_driver debug_driver = { > .drv = { > .name = "coresight-cpu-debug", This should be 2 patches as you are doing two different things here. thanks, greg k-h