Changeset c2250702 in mainline for uspace/drv/fb/kfb/port.c

Timestamp:

2019-12-16T13:31:42Z (5 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

fb420e48

Parents:

65160d7

git-author:

Jiri Svoboda <jiri@…> (2019-12-15 13:23:28)

git-committer:

Jiri Svoboda <jiri@…> (2019-12-16 13:31:42)

Message:

Clip KFB render operations to KFB interior

File:

• uspace/drv/fb/kfb/port.c (modified) (6 diffs)

uspace/drv/fb/kfb/port.c

@@ -68 +68 @@
 
         sysarg_t paddr;
-        sysarg_t width;
-        sysarg_t height;
+        gfx_rect_t rect;
         size_t offset;
         size_t scanline;
@@ -154 +153 @@
 {
         kfb_t *kfb = (kfb_t *) arg;
+        gfx_rect_t crect;
         gfx_coord_t x, y;
 
-        // XXX We should handle p0.x > p1.x and p0.y > p1.y
-
-        for (y = rect->p0.y; y < rect->p1.y; y++) {
-                for (x = rect->p0.x; x < rect->p1.x; x++) {
+        /* Make sure we have a sorted, clipped rectangle */
+        gfx_rect_clip(rect, &kfb->rect, &crect);
+
+        for (y = crect.p0.y; y < crect.p1.y; y++) {
+                for (x = crect.p0.x; x < crect.p1.x; x++) {
                         kfb->pixel2visual(kfb->addr + FB_POS(kfb, x, y),
                             kfb->color);
@@ -242 +243 @@
         gfx_rect_t srect;
         gfx_rect_t drect;
+        gfx_rect_t skfbrect;
+        gfx_rect_t crect;
         gfx_coord2_t offs;
         gfx_coord2_t bmdim;
@@ -251 +254 @@
         pixel_t color;
 
+        /* Clip source rectangle to bitmap bounds */
+
         if (srect0 != NULL)
-                srect = *srect0;
+                gfx_rect_clip(srect0, &kfbbm->rect, &srect);
         else
                 srect = kfbbm->rect;
@@ -272 +277 @@
         pbm.data = kfbbm->alloc.pixels;
 
-        for (pos.y = srect.p0.y; pos.y < srect.p1.y; pos.y++) {
-                for (pos.x = srect.p0.x; pos.x < srect.p1.x; pos.x++) {
+        /* Transform KFB bounding rectangle back to bitmap coordinate system */
+        gfx_rect_rtranslate(&offs, &kfb->rect, &skfbrect);
+
+        /*
+         * Make sure we have a sorted source rectangle, clipped so that
+         * destination lies within KFB bounding rectangle
+         */
+        gfx_rect_clip(&srect, &skfbrect, &crect);
+
+        for (pos.y = crect.p0.y; pos.y < crect.p1.y; pos.y++) {
+                for (pos.x = crect.p0.x; pos.x < crect.p1.x; pos.x++) {
                         gfx_coord2_subtract(&pos, &kfbbm->rect.p0, &sp);
                         gfx_coord2_add(&pos, &offs, &dp);
@@ -429 +443 @@
         kfb->fun = fun;
 
-        kfb->width = width;
-        kfb->height = height;
+        kfb->rect.p0.x = 0;
+        kfb->rect.p0.y = 0;
+        kfb->rect.p1.x = width;
+        kfb->rect.p1.y = height;
+
         kfb->paddr = paddr;
         kfb->offset = offset;