Skip to content

Tolino Epos 2 support#521

Merged
pazos merged 2 commits into
koreader:masterfrom
hugleo:epos2
Oct 7, 2024
Merged

Tolino Epos 2 support#521
pazos merged 2 commits into
koreader:masterfrom
hugleo:epos2

Conversation

@hugleo

@hugleo hugleo commented Oct 6, 2024

Copy link
Copy Markdown
Contributor

Tolino Epos 2

Device info:
Manufacturer: rakuten kobo inc.
Brand: rakutenkobo
Model: tolino
Device: ntx_6sl
Product: ntx_6sl
Hardware: e80k00
Platform: imx6

test.log

Lights: tolino/ntx

Also adds Tolino Epos 1 to EPD driver


This change is Reviewable

Tolino Epos 2

Device info:
Manufacturer: rakuten kobo inc.
Brand: rakutenkobo
Model: tolino
Device: ntx_6sl
Product: ntx_6sl
Hardware: e80k00
Platform: imx6

test.log

Lights: tolino/ntx


Also adds Tolino Epos 1 to EPD driver
SONY_RP1,
TAGUS_GEA,
TOLINO,
TOLINO_EPOS1,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why?

deviceMap[EinkDevice.SONY_RP1] = SONY_RP1
deviceMap[EinkDevice.TAGUS_GEA] = TAGUS_GEA
deviceMap[EinkDevice.TOLINO] = TOLINO
deviceMap[EinkDevice.TOLINO_EPOS1] = TOLINO_EPOS1

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why?

DeviceInfo.EinkDevice.RIDI_PAPER_3,
DeviceInfo.EinkDevice.TOLINO -> {
DeviceInfo.EinkDevice.TOLINO,
DeviceInfo.EinkDevice.TOLINO_EPOS1 -> {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why?

@pazos pazos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't understand why you added the Epos1 to EPD. It will still be asigned the first driver of the factory that matches, and I think we agree recently that was TOLINO. So it's a bit of noise.

@hugleo

hugleo commented Oct 7, 2024

Copy link
Copy Markdown
Contributor Author

Oh sorry,

I unfold strings here:

    TOLINO = BRAND.contentEquals(rakutenkobo) && MODEL.contentEquals("imx50_rdp")
        || MODEL.contentEquals(ntx_6sl) && 
        (DEVICE.contentEquals("tolino_vision2") || DEVICE.contentEquals(ntx_6sl))

My bad. I got confused for to many || or &&. You are right, is catched here: || MODEL.contentEquals(ntx_6sl).

@hugleo hugleo requested a review from pazos October 7, 2024 12:43

@pazos pazos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you very much ;)

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.

2 participants