public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Dirk Behme <dirk.behme@de.bosch.com>
Cc: linux-omap-open-source@linux.omap.com
Subject: Re: Ethernet issue in most recent git?
Date: Thu, 19 Jan 2006 13:58:02 -0800	[thread overview]
Message-ID: <20060119215802.GN4425@atomide.com> (raw)
In-Reply-To: <43CFF2C8.6090307@de.bosch.com>

* Dirk Behme <dirk.behme@de.bosch.com> [060119 12:13]:
> Tony Lindgren wrote:
> >>Telnet and ping don't work as well. I imagine that I doesn't change any 
> >>configuration, only kernel update form git. UBoots tftpboot works quite 
> >>well.
> >
> >Same problem here. I'd assume it's the recent "IRQ type flags" patch
> >that does not work properly for omap gpio interrupts.
> 
> Thanks for the hint! Please try quick hack for OSK ethernet in the
> attachment.
> 
> With
> 
> -- cut --
> [PATCH] IRQ type flags
> /include/asm-arm/irq.h
> blob:59975ee43cf139138503288801c7d8758dada859 ->
> blob:7772432d3fd7750506943eaf88ea7eda9e720a7d
> --- include/asm-arm/irq.h
> +++ include/asm-arm/irq.h
> @@ -25,10 +25,14 @@ extern void disable_irq_nosync(unsigned
> extern void disable_irq(unsigned int);
> extern void enable_irq(unsigned int);
> -#define __IRQT_FALEDGE (1 << 0)
> -#define __IRQT_RISEDGE (1 << 1)
> -#define __IRQT_LOWLVL (1 << 2)
> -#define __IRQT_HIGHLVL (1 << 3)
> +/*
> + * These correspond with the SA_TRIGGER_* defines, and therefore the
> + * IRQRESOURCE_IRQ_* defines.
> + */
> +#define __IRQT_RISEDGE (1 << 0)
> +#define __IRQT_FALEDGE (1 << 1)
> +#define __IRQT_HIGHLVL (1 << 2)
> +#define __IRQT_LOWLVL (1 << 3)
> #define IRQT_NOEDGE (0)
> #define IRQT_RISING (__IRQT_RISEDGE)

Ah, that explains!

> 
> http://source.mvista.com/git/gitweb.cgi?p=linux-omap-2.6.git;a=blobdiff;h=7772432d3fd7750506943eaf88ea7eda9e720a7d;hp=59975ee43cf139138503288801c7d8758dada859;hb=9ded96f24c3a5fcbef954e88c443385a1af37eb9;f=include/asm-arm/irq.h
> 
> the meaning of __IRQT_RISEDGE and __IRQT_FALEDGE was inverted :( So I
> think we have to exchange _all_ places in OMAP code where these are used?

We should be able to just remap them in gpio.c. I believe there's a note
there somewhere saying that by coincidence the bits are the same in omap
hw as in irq.h...

Tony

  reply	other threads:[~2006-01-19 21:58 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-19 17:49 Ethernet issue in most recent git? Dirk Behme
2006-01-19 18:11 ` Anderson Briglia
2006-01-19 19:48   ` Tony Lindgren
2006-01-19 18:12 ` Tony Lindgren
2006-01-19 20:12   ` Dirk Behme
2006-01-19 21:58     ` Tony Lindgren [this message]
2006-01-19 23:21       ` Tony Lindgren

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060119215802.GN4425@atomide.com \
    --to=tony@atomide.com \
    --cc=dirk.behme@de.bosch.com \
    --cc=linux-omap-open-source@linux.omap.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox