Skip to content

LP8556 EPROM changes and 16h register #135

Open

Description

@setotau

According to datasheet, LP8556 EPROM starts at 98h while in driver it starts with a0h and needs to be extended to match datasheet.
Also, downstream uses 16h register which also according to a datasheet is used for configuring LED strings enablement (https://github.com/pix106/android_kernel_xiaomi_southwest-4.19/blob/abde7f459ee423130b3c93924357569c0cf24cca/arch/arm64/boot/dts/vendor/qcom/xiaomi/mi/clover/clover-overlay.dtsi#L758) so the property named like "led-strings-enable" is needed

Activity

  1. minlexx commented on Sep 19, 2025

    @minlexx
    Member

    downstream uses 16h register

    is downstream wrong then? if according to datasheet that address in not in EEPROM

  2. setotau commented on Sep 19, 2025

    @setotau
    Author

    downstream uses 16h register

    is downstream wrong then? if according to datasheet that address in not in EEPROM

    because of this I want to introduce a separate property for this register instead of extending eprom start to it

  3. setotau commented on Nov 1, 2025

    @setotau
    Author

    EPROM start had been corrected in 07c7efd

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions