search

adafruit / Adafruit_Seesaw

Arduino library driver for seesaw multi-use chip
93 stars 64 forks source link

Macro NEO_TRELLIS_XY fix #74

Closed FedericoBusero closed 2 years ago

FedericoBusero commented 2 years ago

Macro NEO_TRELLIS_XY(x,y) was defined as ((y)NEO_TRELLIS_NUM_ROWS + (x)) but it should be ((y)NEO_TRELLIS_NUM_COLS + (x))

As NEO_TRELLIS_NUM_ROWS and NEO_TRELLIS_NUM_COLS are both identical (4), it doesn't make a difference, but in fact it only works by accident

So, this code will not result in any change, as long as NEO_TRELLIS_NUM_ROWS and NEO_TRELLIS_NUM_COLS remain 4

ladyada commented 2 years ago

thanks!