Hi Hannes,
On 03/25/2015 11:24 PM, Hannes Petermaier wrote:
+static void console_calc_rowcol(struct console_t *pcons)
+{
+ pcons->cols = pcons->lcdsizex / VIDEO_FONT_WIDTH;
+#if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
+ pcons->rows = (pcons->lcdsizey - BMP_LOGO_HEIGHT);
+ pcons->rows /= VIDEO_FONT_HEIGHT;
+#else
+ pcons->rows = pcons->lcdsizey / VIDEO_FONT_HEIGHT;
+#endif
+}
Okay, i will fixup the description in v4 ... maybe these troubles are coming
from, lets say, sub-optimal english-language practise :-)
In original i speak german.
That's perfectly understandable, don't worry about it :)
+static void console_calc_rowcol_rot(struct console_t *pcons)
+{
+ u32 cols, rows;
+
+ if (pcons->lcdrot == 1 || pcons->lcdrot == 3) {
+ cols = pcons->lcdsizey;
+ rows = pcons->lcdsizex;
+ } else {
+ cols = pcons->lcdsizex;
+ rows = pcons->lcdsizey;
+ }
+
+ pcons->cols = cols / VIDEO_FONT_WIDTH;
+#if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
+ pcons->rows = (rows - BMP_LOGO_HEIGHT);
+ pcons->rows /= VIDEO_FONT_HEIGHT;
+#else
+ pcons->rows = rows / VIDEO_FONT_HEIGHT;
+#endif
This duplication with console_calc_rowcol() exists because the lcdsizey and
lcdsizex data is expected by the functions to be already in pcons. If you
change console_calc_rowcol() to accept both variables as additional
arguments, then console_calc_rowcol() could be reused in
console_calc_rowcol_rot() and we'll get rid of the code duplication.
I'm not sure about what is more uggly or better.
To avoid this duplication and use one function i have to make this function
non-static and make a mix of rotation-code into lcd_console.c - i wouldn't
prefer this.
Maybe the actual way of this (little) duplication is the beautiful one and
gives most readability of the code.
what do you mean about?
Actually I was thinking about keeping the two functions, but have
console_calc_rowcol_rot() use console_calc_rowcol(), like so:
void console_calc_rowcol(struct console_t *pcons, u32 lcdsizex, lcdsizey)
{
pcons->cols = lcdsizex / VIDEO_FONT_WIDTH;
#if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
pcons->rows = (lcdsizey - BMP_LOGO_HEIGHT);
pcons->rows /= VIDEO_FONT_HEIGHT;
#else
pcons->rows = lcdsizey / VIDEO_FONT_HEIGHT;
#endif
}
static void console_calc_rowcol_rot(struct console_t *pcons)
{
u32 cols, rows;
if (pcons->lcdrot == 1 || pcons->lcdrot == 3) {
cols = pcons->lcdsizey;
rows = pcons->lcdsizex;
} else {
cols = pcons->lcdsizex;
rows = pcons->lcdsizey;
}
console_calc_rowcol(&cons, cols, rows);
}
--
Regards,
Nikita Kiryanov
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot