On Mon, Feb 17, 2014 at 04:50:42PM +0400, Ivan Kluchnikov wrote:

Good Morning,

> +             llist_for_each_entry(cur_trx, &trx->bts->trx_list, list) {
> +                     if (cur_trx->arfcn == arfcn) {

I wondered if we want to compare the access reference as well? We do not
enforce that the ARFCNs of a TRX are distinct. And besides that I would
like you to move the look-up code to a separate function. That way we can
at least write a unit test in the future.


Reply via email to