[PATCH V3 RESEND] staging: sm750: Fix lynxfb_ops_imageblit() if image->depth != 1

Subsystems: staging - silicon motion sm750 frame buffer driver, staging subsystem, the rest

STALE4141d REVIEWED: 3 (3M)

1 review trailer (1 from subsystem maintainers).

3 messages, 3 authors, 2015-05-10 · open the first message on its own page

[PATCH V3 RESEND] staging: sm750: Fix lynxfb_ops_imageblit() if image->depth != 1

From: Huacai Chen <hidden>
Date: 2015-05-08 09:04:28

If image->depth != 1, lynxfb_ops_imageblit() should fallback to call
cfb_imageblit(), not return directly. Otherwise it can't display the
boot logo.

Cc: Teddy Wang <teddy.wang@siliconmotion.com>
Acked-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Signed-off-by: Huacai Chen <redacted>
---
 V2: Coding style ajustment.
 V3: Add TODO comments.

 drivers/staging/sm750fb/TODO    |    1 +
 drivers/staging/sm750fb/sm750.c |    3 +++
 2 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/staging/sm750fb/TODO b/drivers/staging/sm750fb/TODO
index bc16172..f8698498 100644
--- a/drivers/staging/sm750fb/TODO
+++ b/drivers/staging/sm750fb/TODO
@@ -2,6 +2,7 @@ TODO:
 - lots of clechpatch cleanup
 - use kernel coding style
 - refine the code and remove unused code
+- Implement hardware acceleration for imageblit if image->depth > 1
 - check on hardware effects of removal of USE_HW_I2C and USE_DVICHIP (these two
 	are supposed to be sample code which is given here if someone wants to
 	use those functionalities)
diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm750.c
index 3c7ea95..6f78de3 100644
--- a/drivers/staging/sm750fb/sm750.c
+++ b/drivers/staging/sm750fb/sm750.c
@@ -279,7 +279,10 @@ static void lynxfb_ops_imageblit(struct fb_info *info,
 		}
 		goto _do_work;
 	}
+	/* TODO: Implement hardware acceleration for image->depth > 1 */
+	cfb_imageblit(info, image);
 	return;
+
 _do_work:
 	/*
 	 * If not use spin_lock, system will die if user load driver
-- 
1.7.7.3



Re: [PATCH V3 RESEND] staging: sm750: Fix lynxfb_ops_imageblit() if image->depth != 1

From: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Date: 2015-05-08 09:55:54

On Fri, May 08, 2015 at 05:04:28PM +0800, Huacai Chen wrote:
If image->depth != 1, lynxfb_ops_imageblit() should fallback to call
cfb_imageblit(), not return directly. Otherwise it can't display the
boot logo.
why resending? 

regards
sudip

Re: [PATCH V3 RESEND] staging: sm750: Fix lynxfb_ops_imageblit() if image->depth != 1

From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: 2015-05-10 13:17:39

On Fri, May 08, 2015 at 05:04:28PM +0800, Huacai Chen wrote:
If image->depth != 1, lynxfb_ops_imageblit() should fallback to call
cfb_imageblit(), not return directly. Otherwise it can't display the
boot logo.

Cc: Teddy Wang <teddy.wang@siliconmotion.com>
Acked-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Signed-off-by: Huacai Chen <redacted>
---
 V2: Coding style ajustment.
 V3: Add TODO comments.

 drivers/staging/sm750fb/TODO    |    1 +
 drivers/staging/sm750fb/sm750.c |    3 +++
 2 files changed, 4 insertions(+), 0 deletions(-)
Doesn't apply without fuzz, so something isn't synced up properly here.
Please refresh it against my tree and resend.

thanks,

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help