mdacon driver updates

mdacon driver updates

[Boris Bezlaj]

hi

can someone point me to the current maintainer of mdacon.c (Hercules console drv)?
i was unable to find it in MAINTAINERS and the header of the driver is not clear about this..

i would like to make some modifications to the code..small updates etc.

if there is no maintainer, is it ok to post the patch(es) to the lkml ? (when i finish them and testing : )

thanks

Re: mdacon driver updates

[Alan Cox]

> can someone point me to the current maintainer of mdacon.c (Hercules console drv)?
> i was unable to find it in MAINTAINERS and the header of the driver is not clear about this..

It seems to be "whoever last fixed it"

> if there is no maintainer, is it ok to post the patch(es) to the lkml ? (when i finish them and testing : )

Thats probably best

Re: mdacon driver updates

[James Simmons]

> > can someone point me to the current maintainer of mdacon.c (Hercules console drv)?
> > i was unable to find it in MAINTAINERS and the header of the driver is not clear about this..
> 
> It seems to be "whoever last fixed it"

Actually I have a MDA card and having being working on it. I use my MDA as
a second console to debug fbdev drivers. So sned patches my way.
 

Re: mdacon driver updates

[Jan Schubert]

I've posted a small fix to mdacon.c at the end of last year (while 
Marcello was in vacation). It has'nt made it to the kernel yet, nor was 
there any feedback. I've thought, thats the way like open source works...

So, here we go again (against 2.4.17; my MDA is detected, but not 
initialized, see my last posting):

--- drivers/video/mdacon.c.orig    Sun Dec 30 02:44:25 2001
+++ drivers/video/mdacon.c    Sun Dec 30 21:36:50 2001
@@ -24,6 +24,7 @@
  *
  *  Changelog:
  *  Paul G. (03/2001) Fix mdacon= boot prompt to use __setup().
+ *  20011230 Jan.Schubert@GMX.li - consider non-Hercules MDA compatible
  */

 #include <linux/types.h>
@@ -291,6 +292,10 @@
                                break;
                }
        }
+       else {  /* consider non-Hercules as Hercules-compatible */
+               mda_type = TYPE_HERC;
+               mda_type_name = "Hercules compatible (hopefully)";
+       }

        return 1;
 }
@@ -342,9 +347,8 @@
                return NULL;
        }

-       if (mda_type != TYPE_MDA) {
-               mda_initialize();
-       }
+       /* at this point, we found an MDA */
+       mda_initialize();

        /* cursor looks ugly during boot-up, so turn it off */
        mda_set_cursor(mda_vram_len - 1);

Re: mdacon driver updates

[Zwane Mwaikambo]

On Tue, 19 Feb 2002, Jan Schubert wrote:

> I've posted a small fix to mdacon.c at the end of last year (while 
> Marcello was in vacation). It has'nt made it to the kernel yet, nor was 
> there any feedback. I've thought, thats the way like open source works...

James Simmons <jsimmons@transvirtual.com> volunteered to take in patches.

Regards,
	Zwane Mwaikambo

Re: mdacon driver updates

[Paul Gortmaker]

Jan Schubert wrote:
> 
> I've posted a small fix to mdacon.c at the end of last year (while
> Marcello was in vacation). It has'nt made it to the kernel yet, nor was
> there any feedback. I've thought, thats the way like open source works...

Well, you can't be too surprised when everybody doesn't jump to review
driver changes for hardware that dates to the early 80's  :)

I think your change may break true MDA cards (as in an original full
length 8 bit text only card, clustered with enough TTL chips that could
heat a small room...)

If I read the current code right, the p==q case means that TYPE_MDA 
remains set, and these cards don't get mda_initialize().  Only the
TYPE_HERC* cards get mda_initialize().

Your change forces TYPE_HERC and forces mda_initialize() on all 
non Herc cards, which is a departure from existing behaviour.

So, there are two possibilities:

   1) mda_initialize() should really be called hga_initialize() and
      your card is simply broken in some way  :)
or

   2) mda_initialize() is also needed for some non Herc cards too.

What kind of card did you say needed this change?

Paul.

> 
> So, here we go again (against 2.4.17; my MDA is detected, but not
> initialized, see my last posting):
> 
> --- drivers/video/mdacon.c.orig    Sun Dec 30 02:44:25 2001
> +++ drivers/video/mdacon.c    Sun Dec 30 21:36:50 2001
> @@ -24,6 +24,7 @@
>   *
>   *  Changelog:
>   *  Paul G. (03/2001) Fix mdacon= boot prompt to use __setup().
> + *  20011230 Jan.Schubert@GMX.li - consider non-Hercules MDA compatible
>   */
> 
>  #include <linux/types.h>
> @@ -291,6 +292,10 @@
>                                 break;
>                 }
>         }
> +       else {  /* consider non-Hercules as Hercules-compatible */
> +               mda_type = TYPE_HERC;
> +               mda_type_name = "Hercules compatible (hopefully)";
> +       }
> 
>         return 1;
>  }
> @@ -342,9 +347,8 @@
>                 return NULL;
>         }
> 
> -       if (mda_type != TYPE_MDA) {
> -               mda_initialize();
> -       }
> +       /* at this point, we found an MDA */
> +       mda_initialize();
> 
>         /* cursor looks ugly during boot-up, so turn it off */
>         mda_set_cursor(mda_vram_len - 1);
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/