* [PATCH] tidspbridge: Rename module from bridgedriver to tidspbridge
@ 2012-01-24 21:25 Joe Perches
2012-01-30 17:34 ` Ramirez Luna, Omar
0 siblings, 1 reply; 3+ messages in thread
From: Joe Perches @ 2012-01-24 21:25 UTC (permalink / raw)
To: linux-arm-kernel
tidspbridge when built as a module is named bridgedriver.
bridgedriver is not a particularly good module name.
tidspbridge is what the source is named. That seems
a more appropriate module name too as it describes
the hardware function better.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/staging/tidspbridge/Makefile | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/tidspbridge/Makefile b/drivers/staging/tidspbridge/Makefile
index fd6a276..8c8c92a 100644
--- a/drivers/staging/tidspbridge/Makefile
+++ b/drivers/staging/tidspbridge/Makefile
@@ -1,4 +1,4 @@
-obj-$(CONFIG_TIDSPBRIDGE) += bridgedriver.o
+obj-$(CONFIG_TIDSPBRIDGE) += tidspbridge.o
libgen = gen/gh.o gen/uuidutil.o
libcore = core/chnl_sm.o core/msg_sm.o core/io_sm.o core/tiomap3430.o \
@@ -13,7 +13,7 @@ libdload = dynload/cload.o dynload/getsection.o dynload/reloc.o \
dynload/tramp.o
libhw = hw/hw_mmu.o
-bridgedriver-y := $(libgen) $(libservices) $(libcore) $(libpmgr) $(librmgr) \
+tidspbridge-y := $(libgen) $(libservices) $(libcore) $(libpmgr) $(librmgr) \
$(libdload) $(libhw)
#Machine dependent
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] tidspbridge: Rename module from bridgedriver to tidspbridge
2012-01-24 21:25 [PATCH] tidspbridge: Rename module from bridgedriver to tidspbridge Joe Perches
@ 2012-01-30 17:34 ` Ramirez Luna, Omar
2012-01-30 17:51 ` Felipe Contreras
0 siblings, 1 reply; 3+ messages in thread
From: Ramirez Luna, Omar @ 2012-01-30 17:34 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Jan 24, 2012 at 3:25 PM, Joe Perches <joe@perches.com> wrote:
> tidspbridge when built as a module is named bridgedriver.
>
> bridgedriver is not a particularly good module name.
>
> tidspbridge is what the source is named. ?That seems
> a more appropriate module name too as it describes
> the hardware function better.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> ?drivers/staging/tidspbridge/Makefile | ? ?4 ++--
> ?1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/tidspbridge/Makefile b/drivers/staging/tidspbridge/Makefile
> index fd6a276..8c8c92a 100644
> --- a/drivers/staging/tidspbridge/Makefile
> +++ b/drivers/staging/tidspbridge/Makefile
> @@ -1,4 +1,4 @@
> -obj-$(CONFIG_TIDSPBRIDGE) ? ? ?+= bridgedriver.o
> +obj-$(CONFIG_TIDSPBRIDGE) ? ? ?+= tidspbridge.o
>
> ?libgen = gen/gh.o gen/uuidutil.o
> ?libcore = core/chnl_sm.o core/msg_sm.o core/io_sm.o core/tiomap3430.o \
> @@ -13,7 +13,7 @@ libdload = dynload/cload.o dynload/getsection.o dynload/reloc.o \
> ? ? ? ? ? ? ? ? dynload/tramp.o
> ?libhw = hw/hw_mmu.o
>
> -bridgedriver-y := $(libgen) $(libservices) $(libcore) $(libpmgr) $(librmgr) \
> +tidspbridge-y := $(libgen) $(libservices) $(libcore) $(libpmgr) $(librmgr) \
> ? ? ? ? ? ? ? ? ? ? ? ?$(libdload) $(libhw)
>
> ?#Machine dependent
>
>
Sound good to me, and unless there is an objection, it also gives
plenty of time for the change to be noticed.
Regards,
Omar
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] tidspbridge: Rename module from bridgedriver to tidspbridge
2012-01-30 17:34 ` Ramirez Luna, Omar
@ 2012-01-30 17:51 ` Felipe Contreras
0 siblings, 0 replies; 3+ messages in thread
From: Felipe Contreras @ 2012-01-30 17:51 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, Jan 30, 2012 at 7:34 PM, Ramirez Luna, Omar <omar.ramirez@ti.com> wrote:
> On Tue, Jan 24, 2012 at 3:25 PM, Joe Perches <joe@perches.com> wrote:
>> tidspbridge when built as a module is named bridgedriver.
>>
>> bridgedriver is not a particularly good module name.
>>
>> tidspbridge is what the source is named. ?That seems
>> a more appropriate module name too as it describes
>> the hardware function better.
>>
>> Signed-off-by: Joe Perches <joe@perches.com>
>> ---
>> ?drivers/staging/tidspbridge/Makefile | ? ?4 ++--
>> ?1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/staging/tidspbridge/Makefile b/drivers/staging/tidspbridge/Makefile
>> index fd6a276..8c8c92a 100644
>> --- a/drivers/staging/tidspbridge/Makefile
>> +++ b/drivers/staging/tidspbridge/Makefile
>> @@ -1,4 +1,4 @@
>> -obj-$(CONFIG_TIDSPBRIDGE) ? ? ?+= bridgedriver.o
>> +obj-$(CONFIG_TIDSPBRIDGE) ? ? ?+= tidspbridge.o
>>
>> ?libgen = gen/gh.o gen/uuidutil.o
>> ?libcore = core/chnl_sm.o core/msg_sm.o core/io_sm.o core/tiomap3430.o \
>> @@ -13,7 +13,7 @@ libdload = dynload/cload.o dynload/getsection.o dynload/reloc.o \
>> ? ? ? ? ? ? ? ? dynload/tramp.o
>> ?libhw = hw/hw_mmu.o
>>
>> -bridgedriver-y := $(libgen) $(libservices) $(libcore) $(libpmgr) $(librmgr) \
>> +tidspbridge-y := $(libgen) $(libservices) $(libcore) $(libpmgr) $(librmgr) \
>> ? ? ? ? ? ? ? ? ? ? ? ?$(libdload) $(libhw)
>>
>> ?#Machine dependent
>>
>>
>
> Sound good to me, and unless there is an objection, it also gives
> plenty of time for the change to be noticed.
Yeah, I agree. I also have thought on doing this.
--
Felipe Contreras
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-01-30 17:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-24 21:25 [PATCH] tidspbridge: Rename module from bridgedriver to tidspbridge Joe Perches
2012-01-30 17:34 ` Ramirez Luna, Omar
2012-01-30 17:51 ` Felipe Contreras
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox