Change in osmo-bts[master]: Add OC-2G BTS sources

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Wed Oct 24 09:37:42 UTC 2018


Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/11447 )

Change subject: Add OC-2G BTS sources
......................................................................


Patch Set 1: Code-Review-1

An initial review showed there are *many* parts that are identical to osmo-bts-litecell15, and we generally don't like "copy+paste" programming.  In the end we have multiple compies of essentially the same source files, becoming a maintenance problem over time, when fixes are applied to one copy, but not the other.

I've manually looked through each of the files and looked at the differences between the oc2g and the lc15 counterpart.

== 100% identical to osmo-bts-sysmo or osmo-bts-lc15:

oml_router.c
oml_router.h
misc/oc2gbts_nl.c

== 99% identical to lc15 (just names changed)

hw_misc.h
hw_misc.c
l1_transp.h
l1_trans_hw.c
oc2gbts.c / lc15bts.c
oc2gbts_vty.c / lc15bts_vty.c
oml.c
tch.c
utils.c
utils.h
l1_if.h
misc/oc2gbts_bts.h
misc/oc2gbts_bid.h
misc/oc2gbts_clock.h
misc/oc2gbts_clock.c
misc/oc2gbts_led.c
misc/oc2gbts_swd.c
misc/oc2gbts_swd.h

== 80% identical to lc15 (some small changes)

l1_if.c
misc/oc2gbts_bts.c
misc/oc2gbts_mgr.c
misc/oc2gbts_mgr.h
misc/oc2gbts_mgr_nl.c
misc/oc2gbts_temp.c
misc/oc2gbts_temp.h

== more differences compared to lc15, possibly not intentional? / what to do?

misc/oc2gbts_temp.c
misc/oc2gbts_mgr_vty.c
misc/oc2gbts_misc.c
misc/oc2gbts_par.c
misc/oc2gbts_power.c
misc/oc2gbts_util.c


So we have to discuss how to go about this.  The 100% identical files are easy, one can simply lik the existing implementation.  We could also introduce a "src/nuran-common" directory for common code between the different models.

For those parts with less similarity, this may require some refactoring, so that common/shared code goes to common/shared files and really only those bits that differ are handled in the respective implementations.

Given that osmo-bts-litecell15 also started as copy of osmo-bts-sysmo, there may be room for further unification, but let's not conflate those two discussions.

I think gerrit is not the best location to discuss this, so I'll start a mailing list thread.


-- 
To view, visit https://gerrit.osmocom.org/11447
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I327384fe5ac944dc3996a3f00932d6f1a10d5a35
Gerrit-Change-Number: 11447
Gerrit-PatchSet: 1
Gerrit-Owner: Omar Ramadan <omar.ramadan93 at gmail.com>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Comment-Date: Wed, 24 Oct 2018 09:37:42 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20181024/ecfb16b1/attachment.htm>


More information about the gerrit-log mailing list