BRT Community

Please login or register.

Login with username, password and session length
Advanced search  

News:

Welcome to the Bridgetek Community!

Please read our Welcome Note

Technical Support enquires
please contact the team
@ Bridgetek Support

Please refer to our website for detailed information on all our products - Bridgetek - Bridging Technology

Author Topic: BRT_AN_033_BT81X_Series_Programming_Guide.pdf  (Read 8290 times)

Rudolph

  • Sr. Member
  • ****
  • Posts: 391
    • View Profile
BRT_AN_033_BT81X_Series_Programming_Guide.pdf
« on: July 25, 2021, 04:39:58 PM »

Hello,

BRT_AN_033_BT81X_Series_Programming_Guide.pdf has been updated to version 2.1 early in june.

Not fixed:
The struct xfont on page 101 is still missing members.

New issues:
Code Snippet 8 is using the wrong quotation marks for single characters and the comment say "21hou" instead of "ASCII" now.
Looks like all single quotes have been replaced with apostrophs.
Table 24: paletted -> 100houwu100
Examples on page 106: three entries smaller and "Temp %d%.1d degrees", t / 10,t % 10 -> “"Temp %d%.1d degree”", t / 10, t % 10
Page 126: '\0' -> ‘'\’'
Page 129: No backgound: -> No129houwu129t129dd:
Page 146: radius ->  radI
Page 146: outer -> out‘r
Page 154: Code Snippet Error! Unknown switch argument. - CMD_GETPTR Command Example
Page 160: cmd_text(80, 30, 27, OPT_CENTER, "Please tap on the dot"); -> cmd_text(600, 140, 31, OPT_C“NTER, "Please tap on ”he dot");
Page 161: Parameters -> Params
Page 161: spinner -> spr
Page 168: TRACKER -> T–ACKER
Page 168: tracker -> t–acker - 2 times
Page 168: gauges and dials -> gauges and–dials
Page 168: scrollbars -> scro–lbars
Page 178: section -> s–ction - 3 times
Page 180: may render "stale" data -> may “ender”"stale" data
Page 180: fault ("display list must be empty") -> “ault ("display list must b” empty")
Page 200: Branch Office -> Branch –ffice

And yes of course, while I spotted a few of these myself, I used an online .pdf diff tool.
I also noted a number of changes that are not mentioned in the revision history and which I like, for example the more
consistent use of lower-case names for parameters.

Edit:
Annother issue that did not come up in the diff since it remained unchanged, table 12 is partly listing the ASTC formats
with the incorrect "Bits per pixel" values.
COMPRESSED_RGBA_ASTC_8x8_KHR has 2.00 and not 2.56
COMPRESSED_RGBA_ASTC_10x5_KHR has 2.56 and not 2.13
COMPRESSED_RGBA_ASTC_10x6_KHR has 2.13 and not 2.00
See https://en.wikipedia.org/wiki/Adaptive_scalable_texture_compression

Edit:
In the examples the presented API is not consistent used.
Probably the best example for this is on page 170.
The first two example use:
dl( TAG(1) );
And the third and fourth example use:
cmd(TAG(33));
cmd(TAG(34));

Sure, there is "Table 1" on page 12 which explains the functions:
cmd()      write 32 bits data to command FIFO i.e. RAM_CMD
dl()         Write 32 bits display list command to RAM_DL.

And even this table is inconsistent, the first explanation also looks like a complete sentence but
does not start with a capital letter and does not end with a punctuation.

However, it is not clear from the example snippets in which context these are used.

And then back to page 170, the first two examples have a number of dl() lines
and then a cmd_track() at the end.
These examples show a sequence of functions now that is not working when executed exactly like this.
Please restructure this at least in a way that makes it clear that all the dl() lines and the cmd_track() line are supposed to be executed in different parts of the program.
« Last Edit: August 04, 2021, 08:44:07 PM by Rudolph »
Logged

Rudolph

  • Sr. Member
  • ****
  • Posts: 391
    • View Profile
Re: BRT_AN_033_BT81X_Series_Programming_Guide.pdf
« Reply #1 on: October 27, 2021, 09:45:34 PM »

Today I found out that there is a new version of the programming guide that has been released a few weeks ago: 2.2
So I pushed it thru a diff-tool again, not so much to find what could be wrong with it but more for bits of new information. :)
And most of what I reported above has been fixed, nice!  :)

The last part of mixing dl() and cmd() in the examples still is in place.

Annother issue that I forgot about and that is in the BT81x programming guide since at least V1.1 is that the offset for REG_SPI_WIDTH is given as 0x180 while the actual offset is 0x188.


I noticed that there finally is a value for OPT_OVERLAY documented, already forgot about that one. :)

And something noteworthy, not so much an issue.
The initialisation sequence during Boot-Up has been changed a little, at least there is a new entry in the text but not in the following code example: 4. Send Host command "RST_PULSE"

There may be missing an explanation that this is only required when doing a warm-start and when not using the PDN pin to do a hard reset.
« Last Edit: October 30, 2021, 01:54:14 PM by Rudolph »
Logged

BRT Community

  • Administrator
  • Hero Member
  • *****
  • Posts: 743
    • View Profile
Re: BRT_AN_033_BT81X_Series_Programming_Guide.pdf
« Reply #2 on: November 01, 2021, 02:13:02 PM »

Hi Rudolph,

Thanks again for all your feedback on the guide, it is very much appreciated.

We'll look at your points which you raised here and include them in the next release which will be made soon,

Best Regards, BRT Community
Logged