From: Sergei Shtylyov <sshtylyov@ru.mvista.com>
To: Rodolfo Giometti <giometti@linux.it>
Cc: jgarzik@pobox.com, netdev@vger.kernel.org,
Linux-MIPS <linux-mips@linux-mips.org>,
Jordan Crouse <jordan.crouse@amd.com>
Subject: Re: [PATCH] au1000_eth.c Power Management, driver registration and module support
Date: Wed, 31 May 2006 19:01:02 +0400 [thread overview]
Message-ID: <447DAFAE.10503@ru.mvista.com> (raw)
In-Reply-To: <20060502150914.GE20543@gundam.enneenne.com>
Hello.
Rodolfo Giometti wrote:
> here:
> http://ftp.enneenne.com/pub/misc/au1100-patches/linux/patch-au1000_eth-pm-and-registration
> the new version of my patch for au1000_eth.c who should implement:
> Also, as suggested by Sergei it:
>
> * uses physical addresses and not KSEG1-based virtual anymore and
> claims/releases the 4-byte MAC enable registers:
>
> wwpc:~# cat /proc/iomem
> 10500000-1050ffff : eth-base
> 10520000-10520003 : eth-mac
>
> * assigns to the Ethernet ports two consecutive MAC addresses:
>
> - dev->dev_addr[4] += 0x10;
> + ((unsigned long) macen_addr);
> + memcpy(ndev->dev_addr, au1000_mac_addr, sizeof(au1000_mac_addr));
> + ndev->dev_addr[5] += 0x01;
>
> Ciao,
Now that this is merged, Rodolfo's patch should probably preempt mine...
but it looks like something was lost during the transition: I failed to see
where SYS_PINFUNC register is actually read (the comment mentioning this was
retained :-) to check whether Ethernet port 1 is enabled (its pins are shared
w/GPIO)...
> Rodolfo
WBR, Sergei
next prev parent reply other threads:[~2006-05-31 15:02 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-05 15:47 Power management for au1000_eth.c Rodolfo Giometti
2006-04-05 22:23 ` Rodolfo Giometti
2006-04-05 22:26 ` [PATCH] Oops! - " Rodolfo Giometti
2006-04-06 14:43 ` Sergei Shtylyov
2006-04-06 15:50 ` Rodolfo Giometti
2006-04-19 18:46 ` [PATCH] au1000_eth.c probe code straightened up Sergei Shtylyov
2006-05-02 15:09 ` [PATCH] au1000_eth.c Power Management, driver registration and module support Rodolfo Giometti
2006-05-03 7:47 ` Rodolfo Giometti
2006-05-31 15:01 ` Sergei Shtylyov [this message]
2006-05-31 15:21 ` Rodolfo Giometti
2006-04-06 17:10 ` Oops! - Re: Power management for au1000_eth.c Jordan Crouse
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=447DAFAE.10503@ru.mvista.com \
--to=sshtylyov@ru.mvista.com \
--cc=giometti@linux.it \
--cc=jgarzik@pobox.com \
--cc=jordan.crouse@amd.com \
--cc=linux-mips@linux-mips.org \
--cc=netdev@vger.kernel.org \
/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 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.