[PATCH 05/36] xf86: cleanup helper code to use a pointer.
Aaron Plattner
aplattner at nvidia.com
Tue Jul 3 17:04:04 PDT 2012
On 07/02/2012 03:12 AM, Dave Airlie wrote:
> From: Dave Airlie <airlied at redhat.com>
>
> This is in preparation for gpu screens in here, just use
> a pScrn pointer to point at the new screen.
>
> suggested by Keith.
>
> Reviewed-by: Keith Packard <keithp at keithp.com>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
> ---
> hw/xfree86/common/xf86Helper.c | 18 ++++++++++--------
> 1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86Helper.c b/hw/xfree86/common/xf86Helper.c
> index 5ef1dab..4f3315b 100644
> --- a/hw/xfree86/common/xf86Helper.c
> +++ b/hw/xfree86/common/xf86Helper.c
> @@ -167,6 +167,7 @@ ScrnInfoPtr
> xf86AllocateScreen(DriverPtr drv, int flags)
> {
> int i;
> + ScrnInfoPtr pScrn;
>
> if (xf86Screens == NULL)
> xf86NumScreens = 0;
> @@ -174,22 +175,23 @@ xf86AllocateScreen(DriverPtr drv, int flags)
> i = xf86NumScreens++;
> xf86Screens = xnfrealloc(xf86Screens, xf86NumScreens * sizeof(ScrnInfoPtr));
> xf86Screens[i] = xnfcalloc(sizeof(ScrnInfoRec), 1);
> - xf86Screens[i]->scrnIndex = i; /* Changes when a screen is removed */
> - xf86Screens[i]->origIndex = i; /* This never changes */
> - xf86Screens[i]->privates = xnfcalloc(sizeof(DevUnion),
> + pScrn = xf86Screens[i];
> + pScrn->scrnIndex = i; /* Changes when a screen is removed */
> + pScrn->origIndex = i; /* This never changes */
> + pScrn->privates = xnfcalloc(sizeof(DevUnion),
> xf86ScrnInfoPrivateCount);
Spacing got messed up here. Otherwise,
Reviewed-by: Aaron Plattner <aplattner at nvidia.com>
-- Aaron
> /*
> * EnableDisableFBAccess now gets initialized in InitOutput()
> - * xf86Screens[i]->EnableDisableFBAccess = xf86EnableDisableFBAccess;
> + * pScrn->EnableDisableFBAccess = xf86EnableDisableFBAccess;
> */
>
> - xf86Screens[i]->drv = drv;
> + pScrn->drv = drv;
> drv->refCount++;
> - xf86Screens[i]->module = DuplicateModule(drv->module, NULL);
> + pScrn->module = DuplicateModule(drv->module, NULL);
>
> - xf86Screens[i]->DriverFunc = drv->driverFunc;
> + pScrn->DriverFunc = drv->driverFunc;
>
> - return xf86Screens[i];
> + return pScrn;
> }
>
> /*
>
More information about the xorg-devel
mailing list