public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Fix warning with ndelay
@ 2003-05-23  8:38 Benjamin Herrenschmidt
  2003-05-23  8:39 ` [PATCH] PPC32 Fix warning with ndelay (with patch !) Benjamin Herrenschmidt
  2003-05-23  9:08 ` [PATCH] Fix warning with ndelay Riley Williams
  0 siblings, 2 replies; 3+ messages in thread
From: Benjamin Herrenschmidt @ 2003-05-23  8:38 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: Paul Mackerras, linux-kernel mailing list


-- 
Benjamin Herrenschmidt <benh@kernel.crashing.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] PPC32 Fix warning with ndelay (with patch !)
  2003-05-23  8:38 [PATCH] Fix warning with ndelay Benjamin Herrenschmidt
@ 2003-05-23  8:39 ` Benjamin Herrenschmidt
  2003-05-23  9:08 ` [PATCH] Fix warning with ndelay Riley Williams
  1 sibling, 0 replies; 3+ messages in thread
From: Benjamin Herrenschmidt @ 2003-05-23  8:39 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: Paul Mackerras, linux-kernel mailing list

OOOps, here's the real one...

This patch fixes a warning with recent gcc's when using the new
ndelay() function, by properly defining a large constant as unsigned

===== include/asm-ppc/delay.h 1.5 vs edited =====
--- 1.5/include/asm-ppc/delay.h	Wed Feb 12 05:29:33 2003
+++ edited/include/asm-ppc/delay.h	Fri May 23 10:26:50 2003
@@ -30,8 +30,8 @@
  * (which corresponds to ~3800 bogomips at HZ = 100).
  *  -- paulus
  */
-#define __MAX_UDELAY	(226050910/HZ)	/* maximum udelay argument */
-#define __MAX_NDELAY	(4294967295/HZ)	/* maximum ndelay argument */
+#define __MAX_UDELAY	(226050910UL/HZ)	/* maximum udelay argument */
+#define __MAX_NDELAY	(4294967295UL/HZ)	/* maximum ndelay argument */
 
 extern __inline__ void __udelay(unsigned int x)
 {


^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: [PATCH] Fix warning with ndelay
  2003-05-23  8:38 [PATCH] Fix warning with ndelay Benjamin Herrenschmidt
  2003-05-23  8:39 ` [PATCH] PPC32 Fix warning with ndelay (with patch !) Benjamin Herrenschmidt
@ 2003-05-23  9:08 ` Riley Williams
  1 sibling, 0 replies; 3+ messages in thread
From: Riley Williams @ 2003-05-23  9:08 UTC (permalink / raw)
  To: Benjamin Herrenschmidt, Marcelo Tosatti
  Cc: Paul Mackerras, linux-kernel mailing list

Hi Benjamin.

Did you forget to enclose the patch?

Best wishes from Riley.
---
 * Nothing as pretty as a smile, nothing as ugly as a frown.


 > -----Original Message-----
 > From: linux-kernel-owner@vger.kernel.org
 > [mailto:linux-kernel-owner@vger.kernel.org]On Behalf Of Benjamin
 > Herrenschmidt
 > Sent: Friday, May 23, 2003 9:38 AM
 > To: Marcelo Tosatti
 > Cc: Paul Mackerras; linux-kernel mailing list
 > Subject: [PATCH] Fix warning with ndelay
 > 
 > 
 > 
 > -- 
 > Benjamin Herrenschmidt <benh@kernel.crashing.org>
 > -
 > To unsubscribe from this list: send the line "unsubscribe 
 > linux-kernel" in
 > the body of a message to majordomo@vger.kernel.org
 > More majordomo info at  http://vger.kernel.org/majordomo-info.html
 > Please read the FAQ at  http://www.tux.org/lkml/
 > 
 > ---
 > Incoming mail is certified Virus Free.
 > Checked by AVG anti-virus system (http://www.grisoft.com).
 > Version: 6.0.483 / Virus Database: 279 - Release Date: 19-May-2003
 > 
---
Outgoing mail is certified Virus Free.
Checked by AVG anti-virus system (http://www.grisoft.com).
Version: 6.0.483 / Virus Database: 279 - Release Date: 19-May-2003


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2003-05-23  8:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2003-05-23  8:38 [PATCH] Fix warning with ndelay Benjamin Herrenschmidt
2003-05-23  8:39 ` [PATCH] PPC32 Fix warning with ndelay (with patch !) Benjamin Herrenschmidt
2003-05-23  9:08 ` [PATCH] Fix warning with ndelay Riley Williams

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox