All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* <, > versus <=, >=
@ 2001-03-21  4:50 Jeff Ozvold
  0 siblings, 0 replies; 5+ messages in thread
From: Jeff Ozvold @ 2001-03-21  4:50 UTC (permalink / raw
  To: linux-hotplug

It seems to me that major changes happen on release boundaries; in Matt's
example, on the 3.00 boundary.  Many compatibility changes happen for
"pre-version X.XX" versus "version X.XX and later."

If that's true, why not use < (less than) and >= (greater than or equal),
since it seems that it would fit the convention that we use when deploying
releases.  It also avoids the problem of modifying earlier releases.  For
example, we could write a rule that uses " < 2.4.0 " and ">= 2.4.0", so that
after 2.4.0 is released, we could release 2.2.19, 2.2.94733, etc, and we
don't have to change the rule (of course, as long as the functionality of
2.2.X doesn't change).  We also avoid the problem of either duplicating or
not considering the "=" case.

A random thought?

-Jeff


_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
http://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel

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

* && versus &
@ 2011-02-21 18:36 Toralf Förster
  2011-02-21 19:21 ` Wim Van Sebroeck
  0 siblings, 1 reply; 5+ messages in thread
From: Toralf Förster @ 2011-02-21 18:36 UTC (permalink / raw
  To: wim; +Cc: linux-watchdog

Hello,

Im wondering about this line in drivers/watchdog/sch311x_wdt.c:

if (sch311x_sio_inb(sio_config_port, 0x30) && 0x01 == 0)

is this intended ?

-- 
MfG/Kind regards
Toralf Förster

pgp finger print: 7B1A 07F4 EC82 0F90 D4C2 8936 872A E508 7DB6 9DA3


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

* || versus |
@ 2011-02-21 18:43 Toralf Förster
  2011-02-21 19:31 ` Wim Van Sebroeck
  0 siblings, 1 reply; 5+ messages in thread
From: Toralf Förster @ 2011-02-21 18:43 UTC (permalink / raw
  To: wim; +Cc: linux-watchdog

I found in drivers/watchdog/w83697ug_wdt.c:

 outb_p(c || 0x01, WDT_EFDR); /* set bit 0 to activate GPIO2 */

-- 
MfG/Kind regards
Toralf Förster

pgp finger print: 7B1A 07F4 EC82 0F90 D4C2 8936 872A E508 7DB6 9DA3


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

* Re: && versus &
  2011-02-21 18:36 && versus & Toralf Förster
@ 2011-02-21 19:21 ` Wim Van Sebroeck
  0 siblings, 0 replies; 5+ messages in thread
From: Wim Van Sebroeck @ 2011-02-21 19:21 UTC (permalink / raw
  To: Toralf Förster; +Cc: linux-watchdog

Hi Toralf,

> Im wondering about this line in drivers/watchdog/sch311x_wdt.c:
> 
> if (sch311x_sio_inb(sio_config_port, 0x30) && 0x01 == 0)
> 
> is this intended ?

This is a typo/bug.

Patch created.

Kind regards and thanks,
Wim.


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

* Re: || versus |
  2011-02-21 18:43 || versus | Toralf Förster
@ 2011-02-21 19:31 ` Wim Van Sebroeck
  0 siblings, 0 replies; 5+ messages in thread
From: Wim Van Sebroeck @ 2011-02-21 19:31 UTC (permalink / raw
  To: Toralf Förster; +Cc: linux-watchdog

Hi Toralf,

> I found in drivers/watchdog/w83697ug_wdt.c:
> 
>  outb_p(c || 0x01, WDT_EFDR); /* set bit 0 to activate GPIO2 */

Also patch created for this.

Kind regards,
Wim.


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

end of thread, other threads:[~2011-02-21 19:31 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-03-21  4:50 <, > versus <=, >= Jeff Ozvold
  -- strict thread matches above, loose matches on Subject: below --
2011-02-21 18:36 && versus & Toralf Förster
2011-02-21 19:21 ` Wim Van Sebroeck
2011-02-21 18:43 || versus | Toralf Förster
2011-02-21 19:31 ` Wim Van Sebroeck

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.