Skip to content
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

Add support for Nokia 5 and Nokia 6 variants #462

Closed

Conversation

andrewgigena
Copy link
Contributor

These changes are based on downstream DTS files. I have tested the changes on my Nokia 5 with qcom,msm-id value <QCOM_ID_MSM8937 0x2000>. I do not personally own the other devices, so the changes for those variants have not been tested. I believe it is valid to provide support for these devices even though I cannot test them personally. Feedback or verification from someone with access to those devices would be very useful.

Comment on lines +7 to +9
qcom,msm-id = <QCOM_ID_MSM8937 0x0>, // Variant: FIH_D1_EVB
<QCOM_ID_MSM8937 0x1000>, // Variant: FIH_D1_EVT
<QCOM_ID_MSM8937 0x2000>; // Variant: FIH_D1_DVT

Choose a reason for hiding this comment

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

Suggested change
qcom,msm-id = <QCOM_ID_MSM8937 0x0>, // Variant: FIH_D1_EVB
<QCOM_ID_MSM8937 0x1000>, // Variant: FIH_D1_EVT
<QCOM_ID_MSM8937 0x2000>; // Variant: FIH_D1_DVT
qcom,msm-id = <QCOM_ID_MSM8937 0x0>, // Variant: FIH_D1_EVB
<QCOM_ID_MSM8937 0x1000>, // Variant: FIH_D1_EVT
<QCOM_ID_MSM8937 0x2000>; // Variant: FIH_D1_DVT

Comment on lines +7 to +9
qcom,msm-id = <QCOM_ID_MSM8937 0x0>, // Variant: FIH_PLE_EVT1 / FIH_D1C_EVT1
<QCOM_ID_MSM8937 0x2000>, // Variant: FIH_PLE_DVT1 / FIH_D1C_DVT1
<QCOM_ID_MSM8937 0x3000>; // Variant: FIH_PLE_PVT / FIH_D1C_PVT
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
qcom,msm-id = <QCOM_ID_MSM8937 0x0>, // Variant: FIH_PLE_EVT1 / FIH_D1C_EVT1
<QCOM_ID_MSM8937 0x2000>, // Variant: FIH_PLE_DVT1 / FIH_D1C_DVT1
<QCOM_ID_MSM8937 0x3000>; // Variant: FIH_PLE_PVT / FIH_D1C_PVT
qcom,msm-id = <QCOM_ID_MSM8937 0x0>, // Variant: FIH_PLE_EVT1 / FIH_D1C_EVT1
<QCOM_ID_MSM8937 0x2000>, // Variant: FIH_PLE_DVT1 / FIH_D1C_DVT1
<QCOM_ID_MSM8937 0x3000>; // Variant: FIH_PLE_PVT / FIH_D1C_PVT


up {
lk2nd,code = <KEY_VOLUMEUP>;
gpios = <&pmic_pon GPIO_PMIC_RESIN 0>;

Choose a reason for hiding this comment

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

Interesting.

Comment on lines +7 to +8
qcom,msm-id = <QCOM_ID_MSM8937 0x0>, // Variant: FIH_D1_EVB
<QCOM_ID_MSM8937 0x1000>, // Variant: FIH_D1_EVT
Copy link
Member

Choose a reason for hiding this comment

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

In this case if they're clearly labeled as EVB and EVT (and for ple, also DVT since there is PVT) I'd argue they don't actually exist in the wild. According to common terms, those would be

EVB -> Evaluation board
EVT -> Engineering Validation Testing
DVT -> Design Validation Testing
PVT -> Production Validation Testing

PVT is usually the production device right before mass production, and DVT is usually close to PVT software-wise I believe. In this case I'd guess they decided to not respin stuff after DVT for nd1 but had to have some last minute changes for PVT in ple.

In any case I'd argue those don't exist in the wild and we usually skip prototypes in lk2nd (unless of course you actually have one in your hands)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh ok, I didn't know the devices were prototypes neither know the terminology, I just assumed they were variants because the ND1 have 4 models (TA-1053, TA-1024, TA-1044, TA-1027) for different markets. In that case it trims down my mainlining process because the EVB for my device has a Synaptics touchscreen but the DVT and my device has a Novatek touchscreen.

In the case of the PLE the change between DVT -> PVT was a single voltage output of one led, probably it was burnt.

Thanks for the clarifications, I will close the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants