Linux on ARM based TI OMAP SoCs
 help / color / mirror / Atom feed
* [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: Omar Ramirez Luna; +Cc: devel, Tony Lindgren, linux-omap, 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

* Re: [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: Joe Perches
  Cc: devel, Felipe Contreras, Tony Lindgren, linux-omap,
	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
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [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: Ramirez Luna, Omar
  Cc: Joe Perches, devel, Tony Lindgren, linux-omap, 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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ 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