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

arm64: dts: qcom: msm8916-samsung-j5-common: Upstreaming #258

Draft
wants to merge 150 commits into
base: wip/msm8916/6.9
Choose a base branch
from

Conversation

wonderfulShrineMaidenOfParadise
Copy link

  • Resort commits
  • Fix the authors
  • Add samsung-j3 to upstream, and include it in msm8916-mainline Drop j3 in upstream
  • Add touchscreen to upstream patches Drop touchscreen in upstream Upstreamed touchscreen partially.
  • Add front flash LED. It works as a normal LED instead of a flash LED at the moment.
  • Add msm8916-samsung-j3-common.dtsi
  • Apply new DT schema

Link:


led-max-microamp = <300000>;
flash-max-microamp = <1500000>;
flash-max-timeout-us = <1835000>;
Copy link
Member

Choose a reason for hiding this comment

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

Curious mostly, where are these values coming from? Or are they just made up?

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Maybe @nergzd723 can share where they are coming from. I failed to find them in a obvious way downstream.

Choose a reason for hiding this comment

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

I believe downstream KTD2692_MOVIE_CURRENT4 and KTD2692_FLASH_CURRENT16 would match those values. I don't remember how I converted it to uA, exactly.

Copy link
Member

Choose a reason for hiding this comment

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

I still don't really understand it, but I guess I'm fine with it as long as you make sure that it works properly :)

arch/arm64/boot/dts/qcom/Makefile Outdated Show resolved Hide resolved
@stephan-gh stephan-gh changed the title arm64: dts: qcom: msm8916-samsung-j5-common: Upstreaming (v6) WIP: arm64: dts: qcom: msm8916-samsung-j5-common: Upstreaming (v6) Nov 26, 2022
@wonderfulShrineMaidenOfParadise wonderfulShrineMaidenOfParadise changed the base branch from msm8916/6.1-rc6 to msm8916/6.1-rc7 November 29, 2022 04:09
@wonderfulShrineMaidenOfParadise wonderfulShrineMaidenOfParadise changed the base branch from msm8916/6.1-rc7 to msm8916/6.1-rc8 December 5, 2022 15:43
@wonderfulShrineMaidenOfParadise wonderfulShrineMaidenOfParadise changed the base branch from msm8916/6.1-rc8 to msm8916/6.1 December 12, 2022 15:54
@ap4ss3rby
Copy link

ap4ss3rby commented Dec 19, 2022

I'm not sure what happened, but testing the modified source from on my SM-J500H (3g variant, codename j5), the touchscreen works, but the modem seems to not work, specifically it doesn't detect my SIM card, but otherwise seems to work fine.

@wonderfulShrineMaidenOfParadise
Copy link
Author

@ap4ss3rby I don't see modem regressions for now. Does it always or sometimes happen?

@ap4ss3rby
Copy link

ap4ss3rby commented Dec 19, 2022

Well I was testing with a nanoSIM, and since the phone has a slot loading microSIM I wasn't pushing the sim into the socket completely, so it seems like its mostly my fault for not inserting it fully (or not using a SIM adapter), otherwise it seems to be working fine (including only recognizing that my phone is 3g only)

v2: Add suffix "-state" to pinctrl.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
Add support for SMB1360 fuel gauge & charger circuit.

Signed-off-by: Vincent Knecht <vincent.knecht@mailoo.org>
v2: Increase timeout to 768
Signed-off-by: Jakob Hauser <jahau@rocketmail.com>
Signed-off-by: Otto Pflüger <otto.pflueger@abscue.de>
Signed-off-by: Otto Pflüger <otto.pflueger@abscue.de>
Signed-off-by: André Apitzsch <git@apitzsch.eu>
v4: split from initial commit after upstreaming
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
Signed-off-by: André Apitzsch <git@apitzsch.eu>
Add support for SMB1360 fuel gauge and charger circuit.

Signed-off-by: André Apitzsch <git@apitzsch.eu>
v2: Increase timeout to 768
Signed-off-by: Jakob Hauser <jahau@rocketmail.com>
Signed-off-by: Adam Słaboń <asaillen@protonmail.com>
Signed-off-by: Adam Słaboń <asaillen@protonmail.com>
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
Signed-off-by: Raymond Hackley <raymondhackley@protonmail.com>
… jack detection (v2)"

This reverts commit d4d1155.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
…(v4)"

This reverts commit 3139bdd.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
This reverts commit 17fc4f5.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
This reverts commit 708d958.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
…(v2)"

This reverts commit 742fd5a.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
…l (v4)"

This reverts commit 65f020f.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
This reverts commit a7dee29.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
J5 and J5X uses KTD2692 Flash LED driver. Add it to the device tree.

Signed-off-by: Markuss Broks <markuss.broks@gmail.com>
[Apply for msm8916-samsung-j5x]
v2: status = "okay"; by default
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
v2: Make reset-gpios GPIO_ACTIVE_LOW
v3: Add &mdss { status = "okay"; };
Signed-off-by: Markuss Broks <markuss.broks@gmail.com>
v4: Move from msm8916-samsung-j5
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
v2: Move &reg_vdd_lcd_3p0 from common dtsi
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
…tection (v2)

v2: Moved from msm8916-samsung-j5x

Headset microphone is broken with this on j5x, headphone buttons are
broken as well. (Needs more investigation...)

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
…e tree

This common dtsi adds support for msm8916 variant of Samsung Galaxy J3
smartphones released in 2016.

Add a device tree for J3 with initial support for:

- GPIO keys
- SDHCI (internal and external storage)
- USB Device Mode
- UART (on USB connector via the SM5703 MUIC)
- WCNSS (WiFi/BT)
- Regulators

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
v2: Use msm8916-samsung-j5-common
v3: Use msm8916-samsung-j3-common
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
J3 has ST LIS2HH12 accelerometer. Add support for it.

v2: Change interrupt to IRQ_TYPE_LEVEL_HIGH
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
v3: Move from msm8916-samsung-j5. Set vdd-supply to l5
Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
Add regulators for the panel on J3. Note that l17 is required.

Signed-off-by: Lin, Meng-Bo <linmengbo0689@protonmail.com>
@wonderfulShrineMaidenOfParadise wonderfulShrineMaidenOfParadise changed the base branch from wip/msm8916/6.8 to wip/msm8916/6.9 June 1, 2024 09:33
@TravMurav TravMurav force-pushed the wip/msm8916/6.9 branch 2 times, most recently from 98939cb to c4d3662 Compare June 5, 2024 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet