mdacon: align code in mda_detect properly
authorJiri Slaby <jslaby@suse.cz>
Wed, 14 Jun 2017 15:40:55 +0000 (17:40 +0200)
committerBartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Wed, 14 Jun 2017 15:40:55 +0000 (17:40 +0200)
This is just a whitespace cleanup. The code was a mess having multiple
commands on one line like:
scr_writew(0xAA55, p); if (scr_readw(p) == 0xAA55) count++;

Indent that properly and make it nicer for reading.

Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Cc: Tomi Valkeinen <tomi.valkeinen@ti.com>
Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
drivers/video/console/mdacon.c

index ec192a1..424af6b 100644 (file)
@@ -208,10 +208,17 @@ static int mda_detect(void)
        p = (u16 *) mda_vram_base;
        q = (u16 *) (mda_vram_base + 0x01000);
 
-       p_save = scr_readw(p); q_save = scr_readw(q);
+       p_save = scr_readw(p);
+       q_save = scr_readw(q);
+
+       scr_writew(0xAA55, p);
+       if (scr_readw(p) == 0xAA55)
+               count++;
+
+       scr_writew(0x55AA, p);
+       if (scr_readw(p) == 0x55AA)
+               count++;
 
-       scr_writew(0xAA55, p); if (scr_readw(p) == 0xAA55) count++;
-       scr_writew(0x55AA, p); if (scr_readw(p) == 0x55AA) count++;
        scr_writew(p_save, p);
 
        if (count != 2) {
@@ -220,13 +227,18 @@ static int mda_detect(void)
 
        /* check if we have 4K or 8K */
 
-       scr_writew(0xA55A, q); scr_writew(0x0000, p);
-       if (scr_readw(q) == 0xA55A) count++;
+       scr_writew(0xA55A, q);
+       scr_writew(0x0000, p);
+       if (scr_readw(q) == 0xA55A)
+               count++;
        
-       scr_writew(0x5AA5, q); scr_writew(0x0000, p);
-       if (scr_readw(q) == 0x5AA5) count++;
+       scr_writew(0x5AA5, q);
+       scr_writew(0x0000, p);
+       if (scr_readw(q) == 0x5AA5)
+               count++;
 
-       scr_writew(p_save, p); scr_writew(q_save, q);
+       scr_writew(p_save, p);
+       scr_writew(q_save, q);
        
        if (count == 4) {
                mda_vram_len = 0x02000;
@@ -240,14 +252,12 @@ static int mda_detect(void)
        /* Edward: These two mess `tests' mess up my cursor on bootup */
 
        /* cursor low register */
-       if (! test_mda_b(0x66, 0x0f)) {
+       if (!test_mda_b(0x66, 0x0f))
                return 0;
-       }
 
        /* cursor low register */
-       if (! test_mda_b(0x99, 0x0f)) {
+       if (!test_mda_b(0x99, 0x0f))
                return 0;
-       }
 #endif
 
        /* See if the card is a Hercules, by checking whether the vsync
@@ -257,25 +267,25 @@ static int mda_detect(void)
        
        p_save = q_save = inb_p(mda_status_port) & MDA_STATUS_VSYNC;
 
-       for (count=0; count < 50000 && p_save == q_save; count++) {
+       for (count = 0; count < 50000 && p_save == q_save; count++) {
                q_save = inb(mda_status_port) & MDA_STATUS_VSYNC;
                udelay(2);
        }
 
        if (p_save != q_save) {
                switch (inb_p(mda_status_port) & 0x70) {
-                       case 0x10:
-                               mda_type = TYPE_HERCPLUS;
-                               mda_type_name = "HerculesPlus";
-                               break;
-                       case 0x50:
-                               mda_type = TYPE_HERCCOLOR;
-                               mda_type_name = "HerculesColor";
-                               break;
-                       default:
-                               mda_type = TYPE_HERC;
-                               mda_type_name = "Hercules";
-                               break;
+               case 0x10:
+                       mda_type = TYPE_HERCPLUS;
+                       mda_type_name = "HerculesPlus";
+                       break;
+               case 0x50:
+                       mda_type = TYPE_HERCCOLOR;
+                       mda_type_name = "HerculesColor";
+                       break;
+               default:
+                       mda_type = TYPE_HERC;
+                       mda_type_name = "Hercules";
+                       break;
                }
        }