All of lore.kernel.org
 help / color / mirror / Atom feed
diff for duplicates of <20170220175331.GC963@e110455-lin.cambridge.arm.com>

diff --git a/a/1.txt b/N1/1.txt
index 601e6d3..3ee11a6 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -37,11 +37,4 @@ Liviu
 | giving me the   |
  \ source code!  /
   ---------------
-    ?\_(?)_/?
--------------- next part --------------
-A non-text attachment was scrubbed...
-Name: 0001-arm-hdlcd-Fix-the-calculation-of-scanout-start-addre.patch
-Type: text/x-diff
-Size: 1635 bytes
-Desc: not available
-URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170220/c264b8a0/attachment.bin>
+    ¯\_(ツ)_/¯
diff --git a/N1/2.hdr b/N1/2.hdr
new file mode 100644
index 0000000..589bdf8
--- /dev/null
+++ b/N1/2.hdr
@@ -0,0 +1,2 @@
+Content-Type: text/x-diff; charset=utf-8
+Content-Disposition: attachment; filename="0001-arm-hdlcd-Fix-the-calculation-of-scanout-start-addre.patch"
diff --git a/N1/2.txt b/N1/2.txt
new file mode 100644
index 0000000..1a25872
--- /dev/null
+++ b/N1/2.txt
@@ -0,0 +1,47 @@
+>From a495d262cb91fdeccb00685646ddb71774c18e7e Mon Sep 17 00:00:00 2001
+From: Liviu Dudau <Liviu.Dudau@arm.com>
+Date: Mon, 20 Feb 2017 17:44:42 +0000
+Subject: [PATCH] arm: hdlcd: Fix the calculation of scanout start address
+
+The framebuffer start address uses the CRTC's x,y position rather
+than the source framebuffer's. Fix that.
+
+Reported-by: Russell King <rmk+kernel@armlinux.org.uk>
+Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>
+---
+ drivers/gpu/drm/arm/hdlcd_crtc.c | 11 +++++------
+ 1 file changed, 5 insertions(+), 6 deletions(-)
+
+diff --git a/drivers/gpu/drm/arm/hdlcd_crtc.c b/drivers/gpu/drm/arm/hdlcd_crtc.c
+index 798a3cc480a2..4c1ab73d9e07 100644
+--- a/drivers/gpu/drm/arm/hdlcd_crtc.c
++++ b/drivers/gpu/drm/arm/hdlcd_crtc.c
+@@ -244,21 +244,20 @@ static void hdlcd_plane_atomic_update(struct drm_plane *plane,
+ 	struct drm_framebuffer *fb = plane->state->fb;
+ 	struct hdlcd_drm_private *hdlcd;
+ 	struct drm_gem_cma_object *gem;
+-	u32 src_w, src_h, dest_w, dest_h;
++	u32 src_x, src_y, dest_w, dest_h;
+ 	dma_addr_t scanout_start;
+ 
+ 	if (!fb)
+ 		return;
+ 
+-	src_w = plane->state->src_w >> 16;
+-	src_h = plane->state->src_h >> 16;
++	src_x = plane->state->src_x >> 16;
++	src_y = plane->state->src_y >> 16;
+ 	dest_w = plane->state->crtc_w;
+ 	dest_h = plane->state->crtc_h;
+ 	gem = drm_fb_cma_get_gem_obj(fb, 0);
+ 	scanout_start = gem->paddr + fb->offsets[0] +
+-		plane->state->crtc_y * fb->pitches[0] +
+-		plane->state->crtc_x *
+-		fb->format->cpp[0];
++		src_y * fb->pitches[0] +
++		src_x *	fb->format->cpp[0];
+ 
+ 	hdlcd = plane->dev->dev_private;
+ 	hdlcd_write(hdlcd, HDLCD_REG_FB_LINE_LENGTH, fb->pitches[0]);
+-- 
+2.11.0
diff --git a/N1/3.hdr b/N1/3.hdr
new file mode 100644
index 0000000..5216513
--- /dev/null
+++ b/N1/3.hdr
@@ -0,0 +1,4 @@
+Content-Type: text/plain; charset="utf-8"
+MIME-Version: 1.0
+Content-Transfer-Encoding: base64
+Content-Disposition: inline
diff --git a/N1/3.txt b/N1/3.txt
new file mode 100644
index 0000000..3b3a377
--- /dev/null
+++ b/N1/3.txt
@@ -0,0 +1,4 @@
+_______________________________________________
+dri-devel mailing list
+dri-devel@lists.freedesktop.org
+https://lists.freedesktop.org/mailman/listinfo/dri-devel
diff --git a/a/content_digest b/N1/content_digest
index bf873f3..f3b6332 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -1,10 +1,13 @@
  "ref\020161118233733.GP1041@n2100.armlinux.org.uk\0"
  "ref\020170220121603.GA1132@n2100.armlinux.org.uk\0"
- "From\0liviu.dudau@arm.com (Liviu Dudau)\0"
- "Subject\0[BUG] hdlcd gets confused about base address\0"
+ "From\0Liviu Dudau <liviu.dudau@arm.com>\0"
+ "Subject\0Re: [BUG] hdlcd gets confused about base address\0"
  "Date\0Mon, 20 Feb 2017 17:53:31 +0000\0"
- "To\0linux-arm-kernel@lists.infradead.org\0"
- "\00:1\0"
+ "To\0Russell King - ARM Linux <linux@armlinux.org.uk>\0"
+ "Cc\0dri-devel@lists.freedesktop.org"
+  Mali DP Maintainers <malidp@foss.arm.com>
+ " linux-arm-kernel@lists.infradead.org\0"
+ "\01:1\0"
  "b\0"
  "Hi Russell,\n"
  "\n"
@@ -45,13 +48,62 @@
  "| giving me the   |\n"
  " \\ source code!  /\n"
  "  ---------------\n"
- "    ?\\_(?)_/?\n"
- "-------------- next part --------------\n"
- "A non-text attachment was scrubbed...\n"
- "Name: 0001-arm-hdlcd-Fix-the-calculation-of-scanout-start-addre.patch\n"
- "Type: text/x-diff\n"
- "Size: 1635 bytes\n"
- "Desc: not available\n"
- URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170220/c264b8a0/attachment.bin>
+ "    \302\257\\_(\343\203\204)_/\302\257"
+ "\01:2\0"
+ "fn\00001-arm-hdlcd-Fix-the-calculation-of-scanout-start-addre.patch\0"
+ "b\0"
+ ">From a495d262cb91fdeccb00685646ddb71774c18e7e Mon Sep 17 00:00:00 2001\n"
+ "From: Liviu Dudau <Liviu.Dudau@arm.com>\n"
+ "Date: Mon, 20 Feb 2017 17:44:42 +0000\n"
+ "Subject: [PATCH] arm: hdlcd: Fix the calculation of scanout start address\n"
+ "\n"
+ "The framebuffer start address uses the CRTC's x,y position rather\n"
+ "than the source framebuffer's. Fix that.\n"
+ "\n"
+ "Reported-by: Russell King <rmk+kernel@armlinux.org.uk>\n"
+ "Signed-off-by: Liviu Dudau <Liviu.Dudau@arm.com>\n"
+ "---\n"
+ " drivers/gpu/drm/arm/hdlcd_crtc.c | 11 +++++------\n"
+ " 1 file changed, 5 insertions(+), 6 deletions(-)\n"
+ "\n"
+ "diff --git a/drivers/gpu/drm/arm/hdlcd_crtc.c b/drivers/gpu/drm/arm/hdlcd_crtc.c\n"
+ "index 798a3cc480a2..4c1ab73d9e07 100644\n"
+ "--- a/drivers/gpu/drm/arm/hdlcd_crtc.c\n"
+ "+++ b/drivers/gpu/drm/arm/hdlcd_crtc.c\n"
+ "@@ -244,21 +244,20 @@ static void hdlcd_plane_atomic_update(struct drm_plane *plane,\n"
+ " \tstruct drm_framebuffer *fb = plane->state->fb;\n"
+ " \tstruct hdlcd_drm_private *hdlcd;\n"
+ " \tstruct drm_gem_cma_object *gem;\n"
+ "-\tu32 src_w, src_h, dest_w, dest_h;\n"
+ "+\tu32 src_x, src_y, dest_w, dest_h;\n"
+ " \tdma_addr_t scanout_start;\n"
+ " \n"
+ " \tif (!fb)\n"
+ " \t\treturn;\n"
+ " \n"
+ "-\tsrc_w = plane->state->src_w >> 16;\n"
+ "-\tsrc_h = plane->state->src_h >> 16;\n"
+ "+\tsrc_x = plane->state->src_x >> 16;\n"
+ "+\tsrc_y = plane->state->src_y >> 16;\n"
+ " \tdest_w = plane->state->crtc_w;\n"
+ " \tdest_h = plane->state->crtc_h;\n"
+ " \tgem = drm_fb_cma_get_gem_obj(fb, 0);\n"
+ " \tscanout_start = gem->paddr + fb->offsets[0] +\n"
+ "-\t\tplane->state->crtc_y * fb->pitches[0] +\n"
+ "-\t\tplane->state->crtc_x *\n"
+ "-\t\tfb->format->cpp[0];\n"
+ "+\t\tsrc_y * fb->pitches[0] +\n"
+ "+\t\tsrc_x *\tfb->format->cpp[0];\n"
+ " \n"
+ " \thdlcd = plane->dev->dev_private;\n"
+ " \thdlcd_write(hdlcd, HDLCD_REG_FB_LINE_LENGTH, fb->pitches[0]);\n"
+ "-- \n"
+ 2.11.0
+ "\01:3\0"
+ "b\0"
+ "_______________________________________________\n"
+ "dri-devel mailing list\n"
+ "dri-devel@lists.freedesktop.org\n"
+ https://lists.freedesktop.org/mailman/listinfo/dri-devel
 
-317219a9c76b86b6b383a70114a378df892ce0a667e12b4d940f6fbfea1e5664
+8c032848072aade9c7eaa64920170b983793d4404cd5e5357baa9dab37b12c20

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.