Skip to content

Lineage 18.1 update #49

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

linckandrea
Copy link
Contributor

No description provided.

 * as c43ccf28522e6ca0401e7c64bc7620612313137e

 * the following commits were re-applyed

config: Add missing ims apn's for supported IMS carriers

Taken from stock oem partitions.

This is a combination of all supported carriers for lilac, poplar, poplar_canada and poplar_dsds. It should hopefully cover all maple carriers as well.

Also, hide the IMS apn's from the user. There is no need to show them in the list, and they should not be selectable.

Finally, added KPN Mobiel Internet from stock as well (mcc 204 / mnc 69) as it was missing.

yoshino-common: config: apn: Clean up IMS entries

This seems to fix some general data connection issues for carriers such as Orange.
 * this fix the following warning

build/make/core/Makefile:28: warning: overriding commands for target `out/target/product/lilac/system/product/etc/apns-conf.xml'

build/make/core/base_rules.mk:510: warning: ignoring old commands for target `out/target/product/lilac/system/product/etc/apns-conf.xml'

thanks to @Flamefire for pointing it out
…werhint

 * right now common powerhint is too OP for lilac
@linckandrea linckandrea force-pushed the lineage-18.1_update branch from 8f2dbdd to d345e66 Compare June 9, 2021 06:25
@linckandrea
Copy link
Contributor Author

hold

@@ -137,6 +140,7 @@ endif
OVERRIDE_RS_DRIVER := libRSDriver_adreno.so

### RIL
DEFAULT_APNS_FILE := $(PLATFORM_PATH)/config/telephony/apns-conf.xml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this until LOS fixed their script. See #46 (comment) for details

@derfelot
Copy link
Member

Any updates? If not, I will pick the powerhint commit to an update I want to do later and leave the APN stuff now.

@Flamefire
Copy link
Contributor

I'd remove the 2nd commit and keep the other 2. Tested on my phone with 17.1 and working fine so far.

@derfelot
Copy link
Member

I'll close this and pick the two working ones.

@derfelot derfelot closed this Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants