Hi Joel, On Mon, Jan 27 2020, Joel Johnson wrote: > Make the board version printed indicate the Pro variant default. > Also adjust static name casing to match what is expected for > EEPROM product name to share string constants. > > --- > > > Baruch - can you confirm expected/desired branding casing? The SolidRun > website and prior to this commit uses "ClearFog", however the EEPROM > checked values use "Clearfog", so I changed to match to be able to use > the same string constant values for consistency.
I am fine with this change. The official name is "ClearFog", but you can also find "Clearfog" in the documentation, for example: https://developer.solid-run.com/knowledge-base/a388-debian/ baruch > Signed-off-by: Joel Johnson <mrj...@lixil.net> > --- > board/solidrun/clearfog/clearfog.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/board/solidrun/clearfog/clearfog.c > b/board/solidrun/clearfog/clearfog.c > index e268ef55a2..9b31902c70 100644 > --- a/board/solidrun/clearfog/clearfog.c > +++ b/board/solidrun/clearfog/clearfog.c > @@ -170,7 +170,7 @@ int board_init(void) > > int checkboard(void) > { > - char *board = "ClearFog"; > + char *board = "Clearfog Pro"; > > cf_read_tlv_data(); > if (strlen(cf_tlv_data.tlv_product_name[0]) > 0) -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - bar...@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -