Hi Yang,
 
> +
> +unsigned int __ofono_netreg_get_tech(struct ofono_netreg *netreg) {
> +	GSList *o;
> +	unsigned int techs = 0;
> +	struct network_operator_data *opd;
> +
> +	for (o = netreg->operator_list; o; o = o->next) {
> +		opd = o->data;
> +		techs |= opd->techs;
> +	}
> +
> +	return techs;
> +}

Is this function intended to get the current access technology?  

As per the ETSI TS 102 223 specification 4.12 section, when the SAT
issues Provide Local Information proactive command with Access
Technology(0x06) as the command qualifier, we should provide the current
access technology . So, as per my understanding we need to use
netreg->technology. 

> void __ofono_netreg_set_base_station_name(struct ofono_netreg *netreg,
> 						const char *name);
> +unsigned int __ofono_netreg_get_tech(struct ofono_netreg *netreg);
 
Empty line needed between __ofono_netreg_set_base_station_name and
__ofono_netreg_get_tech

Regards,
Jeevaka