Mailinglist Archive: radeonhd (622 mails)
| < Previous | Next > |
Re: [radeonhd] Patch for get DPI
- From: Egbert Eich <eich@xxxxxxx>
- Date: Wed, 3 Oct 2007 15:09:00 +0200
- Message-id: <18179.37996.334169.332892@xxxxxxxxxxxxxxxxxxx>
Swappp writes:
> Swappp пишет:
> > This patch help xf86SetDpi access to DDC data. May be not completely correct, in VESA driver for that used vbe module...
> >
>
> Also without this patch driver have small memory leak. In RHDI2CFunc calling xf86DoEDID_DDC2 function, which allocated memory
> for xf86MonPtr structure, and at end execution rhdTestDDC all pointers on this memory area destroyed, but memory not freed.
Right. This function is scheduled to go anyhow.
Cheers,
Egbert.
> --- git/xf86-video-radeonhd/src/rhd_driver.c 2007-09-29 17:52:15.000000000 +0400
> +++ xf86-video-radeonhd/src/rhd_driver.c 2007-09-29 17:50:00.000000000 +0400
> @@ -1577,7 +1577,8 @@
> if (data.monitor) {
> xf86DrvMsg(pScrn->scrnIndex, X_INFO, "Found DDC on line %i:\n",
> i - 1);
> - xf86PrintEDID(data.monitor);
> + pScrn->monitor->DDC = data.monitor;
> + xf86PrintEDID(pScrn->monitor->DDC);
> } else {
> xf86DrvMsgVerb(pScrn->scrnIndex, 7,X_INFO,
> "No DDC data found on line %i\n",i - 1);
--
To unsubscribe, e-mail: radeonhd+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: radeonhd+help@xxxxxxxxxxxx
> Swappp пишет:
> > This patch help xf86SetDpi access to DDC data. May be not completely correct, in VESA driver for that used vbe module...
> >
>
> Also without this patch driver have small memory leak. In RHDI2CFunc calling xf86DoEDID_DDC2 function, which allocated memory
> for xf86MonPtr structure, and at end execution rhdTestDDC all pointers on this memory area destroyed, but memory not freed.
Right. This function is scheduled to go anyhow.
Cheers,
Egbert.
> --- git/xf86-video-radeonhd/src/rhd_driver.c 2007-09-29 17:52:15.000000000 +0400
> +++ xf86-video-radeonhd/src/rhd_driver.c 2007-09-29 17:50:00.000000000 +0400
> @@ -1577,7 +1577,8 @@
> if (data.monitor) {
> xf86DrvMsg(pScrn->scrnIndex, X_INFO, "Found DDC on line %i:\n",
> i - 1);
> - xf86PrintEDID(data.monitor);
> + pScrn->monitor->DDC = data.monitor;
> + xf86PrintEDID(pScrn->monitor->DDC);
> } else {
> xf86DrvMsgVerb(pScrn->scrnIndex, 7,X_INFO,
> "No DDC data found on line %i\n",i - 1);
--
To unsubscribe, e-mail: radeonhd+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: radeonhd+help@xxxxxxxxxxxx
| < Previous | Next > |