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

Update PocketBook device definitions #6525

Merged
merged 3 commits into from Aug 19, 2020
Merged

Update PocketBook device definitions #6525

merged 3 commits into from Aug 19, 2020

Conversation

roshavagarga
Copy link
Contributor

@roshavagarga roshavagarga commented Aug 19, 2020

Changes:

Reasons:

  • It seems like the 615 Plus is a CIS-only device, there's a manual.
  • 616 is a CIS-only device (manual), while the Basic Lux 2 is a worldwide device that goes under 616W (manual).

Other:

Besides that, this is ready for review, can't test as always.


This change is Reviewable

@Frenzie
Copy link
Member

Frenzie commented Aug 19, 2020

has3BytesWideFrameBuffer

I'd get rid of that one since it's not doing any good, but if I'm not mistaken hasColorScreen should be perfectly fine on any device. The only problem is that things would be rendered in 24-bit and subsequently downconverted to 8-bit. Which wastes memory and processing power but should otherwise be fine.

roshavagarga and others added 2 commits August 19, 2020 09:53
Another infamous roshavagarga typo fix.

Co-authored-by: Frans de Jonge <fransdejonge@gmail.com>
@Frenzie Frenzie added this to the 2020.09 milestone Aug 19, 2020
@Frenzie Frenzie merged commit a9aa63d into koreader:master Aug 19, 2020
@Frenzie Frenzie modified the milestones: 2020.09, 2020.08.1 Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PocketBook 615 Plus doesn't start
2 participants