Change in osmo-bts[master]: BS Power Control Loop: refactor lchan_bs_pwr_ctrl() to look similar t...

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/.

pespin gerrit-no-reply at lists.osmocom.org
Tue Sep 7 08:47:36 UTC 2021


pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/25334 )

Change subject: BS Power Control Loop: refactor lchan_bs_pwr_ctrl() to look similar to lchan_ms_pwr_ctrl()
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/c/osmo-bts/+/25334/2/src/common/power_control.c 
File src/common/power_control.c:

https://gerrit.osmocom.org/c/osmo-bts/+/25334/2/src/common/power_control.c@351 
PS2, Line 351: 		/* Basic signal transmission / reception formula:
> What was your motivation to move this comment into an else statement? It also applies to the upper R […]
In the previous state there we as an early return, so no logic is changed. I did so to let all paths follow in the function and have a unified print path at the end, same as we do for MS Power Control Loop, to have both functions look alike and make it easier for devs to apply similar improvements on both.



-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/25334
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: Ida76b55f0da722135eb61ed06ea2b3a1862df600
Gerrit-Change-Number: 25334
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-Reviewer: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Comment-Date: Tue, 07 Sep 2021 08:47:36 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210907/0703741b/attachment.htm>


More information about the gerrit-log mailing list