From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 2/2] NET: NE2000: Hide dp83902a_priv_data_t into ne2k_private_data
Date: Sun, 11 Dec 2011 02:09:31 +0100 [thread overview]
Message-ID: <1323565772-4498-2-git-send-email-marek.vasut@gmail.com> (raw)
In-Reply-To: <1323565772-4498-1-git-send-email-marek.vasut@gmail.com>
This patch should complete effort to let multiple ne2k cards be used. All
card-specific data shall now be hidden in per-card allocated private_data
structure.
Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
---
drivers/net/ne2000_base.c | 52 +++++++++++++++++++++++---------------------
1 files changed, 27 insertions(+), 25 deletions(-)
diff --git a/drivers/net/ne2000_base.c b/drivers/net/ne2000_base.c
index 35a8581..381b334 100644
--- a/drivers/net/ne2000_base.c
+++ b/drivers/net/ne2000_base.c
@@ -89,8 +89,6 @@ Add SNMP
#include "ne2000.h"
#endif
-static dp83902a_priv_data_t nic; /* just one instance of the card supported */
-
uint32_t ne2k_default_in(uint8_t *addr, uint32_t offset)
{
uint32_t ret;
@@ -110,6 +108,7 @@ struct ne2k_io_accessors {
struct ne2k_private_data {
struct ne2k_io_accessors io;
+ struct dp83902a_priv_data nic;
};
/* forward definition of function used for the uboot interface */
@@ -123,7 +122,8 @@ void uboot_push_tx_done(int key, int val);
static bool
dp83902a_init(struct eth_device *dev)
{
- dp83902a_priv_data_t *dp = &nic;
+ struct ne2k_private_data *pdata = dev->priv;
+ struct dp83902a_priv_data *dp = &pdata->nic;
u8* base;
#if defined(NE2000_BASIC_INIT)
int i;
@@ -139,7 +139,6 @@ dp83902a_init(struct eth_device *dev)
#if defined(NE2000_BASIC_INIT)
unsigned char *enetaddr = dev->enetaddr;
- struct ne2k_private_data *pdata = dev->priv;
const struct ne2k_io_accessors *io = &pdata->io;
/* AX88796L doesn't need */
@@ -167,8 +166,8 @@ dp83902a_init(struct eth_device *dev)
static void
dp83902a_stop(struct eth_device *dev)
{
- dp83902a_priv_data_t *dp = &nic;
struct ne2k_private_data *pdata = dev->priv;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
@@ -190,9 +189,9 @@ dp83902a_stop(struct eth_device *dev)
static void
dp83902a_start(struct eth_device *dev)
{
- dp83902a_priv_data_t *dp = &nic;
- unsigned char *enaddr = dev->enetaddr;
struct ne2k_private_data *pdata = dev->priv;
+ struct dp83902a_priv_data *dp = &pdata->nic;
+ unsigned char *enaddr = dev->enetaddr;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
int i;
@@ -244,7 +243,7 @@ dp83902a_start(struct eth_device *dev)
static void
dp83902a_start_xmit(struct ne2k_private_data *pdata, int start_page, int len)
{
- dp83902a_priv_data_t *dp = (dp83902a_priv_data_t *) &nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
@@ -273,7 +272,7 @@ dp83902a_start_xmit(struct ne2k_private_data *pdata, int start_page, int len)
static void
dp83902a_send(struct ne2k_private_data *pdata, u8 *data, int total_len, u32 key)
{
- struct dp83902a_priv_data *dp = (struct dp83902a_priv_data *) &nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
int len, start_page, pkt_len, i, isr;
@@ -403,7 +402,7 @@ dp83902a_send(struct ne2k_private_data *pdata, u8 *data, int total_len, u32 key)
static void
dp83902a_RxEvent(struct ne2k_private_data *pdata)
{
- struct dp83902a_priv_data *dp = (struct dp83902a_priv_data *) &nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
u8 rcv_hdr[4];
@@ -475,7 +474,7 @@ dp83902a_RxEvent(struct ne2k_private_data *pdata)
static void
dp83902a_recv(struct ne2k_private_data *pdata, u8 *data, int len)
{
- struct dp83902a_priv_data *dp = (struct dp83902a_priv_data *) &nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
int i, mlen;
@@ -541,7 +540,7 @@ dp83902a_recv(struct ne2k_private_data *pdata, u8 *data, int len)
static void
dp83902a_TxEvent(struct ne2k_private_data *pdata)
{
- struct dp83902a_priv_data *dp = (struct dp83902a_priv_data *) &nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
u32 key;
@@ -578,7 +577,7 @@ dp83902a_TxEvent(struct ne2k_private_data *pdata)
static void
dp83902a_ClearCounters(struct ne2k_private_data *pdata)
{
- struct dp83902a_priv_data *dp = (struct dp83902a_priv_data *) &nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
@@ -595,7 +594,7 @@ dp83902a_ClearCounters(struct ne2k_private_data *pdata)
static void
dp83902a_Overflow(struct ne2k_private_data *pdata)
{
- struct dp83902a_priv_data *dp = (struct dp83902a_priv_data *)&nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
u8 isr;
@@ -637,7 +636,7 @@ dp83902a_Overflow(struct ne2k_private_data *pdata)
static void
dp83902a_poll(struct ne2k_private_data *pdata)
{
- struct dp83902a_priv_data *dp = (struct dp83902a_priv_data *) &nic;
+ struct dp83902a_priv_data *dp = &pdata->nic;
const struct ne2k_io_accessors *io = &pdata->io;
u8 *base = dp->base;
u8 isr;
@@ -716,6 +715,9 @@ void uboot_push_tx_done(int key, int val) {
*/
static int ne2k_setup_driver(struct eth_device *dev)
{
+ struct ne2k_private_data *pdata = dev->priv;
+ struct dp83902a_priv_data *dp = &pdata->nic;
+
PRINTK("### ne2k_setup_driver\n");
if (!pbuf) {
@@ -736,13 +738,13 @@ static int ne2k_setup_driver(struct eth_device *dev)
}
#endif
- nic.base = (u8 *) CONFIG_DRIVER_NE2000_BASE;
+ dp->base = (u8 *) CONFIG_DRIVER_NE2000_BASE;
- nic.data = nic.base + DP_DATA;
- nic.tx_buf1 = START_PG;
- nic.tx_buf2 = START_PG2;
- nic.rx_buf_start = RX_START;
- nic.rx_buf_end = RX_END;
+ dp->data = dp->base + DP_DATA;
+ dp->tx_buf1 = START_PG;
+ dp->tx_buf2 = START_PG2;
+ dp->rx_buf_start = RX_START;
+ dp->rx_buf_end = RX_END;
/*
* According to doc/README.enetaddr, drivers shall give priority
@@ -751,7 +753,7 @@ static int ne2k_setup_driver(struct eth_device *dev)
*/
if (!eth_getenv_enetaddr("ethaddr", dev->enetaddr)) {
/* If the MAC address is not in the environment, get it: */
- if (!get_prom(dev->enetaddr, nic.base)) /* get MAC from prom */
+ if (!get_prom(dev->enetaddr, dp->base)) /* get MAC from prom */
dp83902a_init(dev); /* fallback: seeprom */
/* And write it into the environment otherwise eth_write_hwaddr
* returns -1 due to eth_getenv_enetaddr_by_index() failing,
@@ -830,9 +832,6 @@ int ne2k_register_io(uint32_t (*in)(uint8_t *addr, uint32_t offset),
return -ENOMEM;
}
- if (ne2k_setup_driver(dev))
- return -EINVAL;
-
if (in == NULL)
in = ne2k_default_in;
@@ -848,6 +847,9 @@ int ne2k_register_io(uint32_t (*in)(uint8_t *addr, uint32_t offset),
dev->recv = ne2k_recv;
dev->priv = pdata;
+ if (ne2k_setup_driver(dev))
+ return -EINVAL;
+
sprintf(dev->name, "NE2000");
return eth_register(dev);
--
1.7.7.1
next prev parent reply other threads:[~2011-12-11 1:09 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-11 1:09 [U-Boot] [PATCH 1/2] NET: NE2000: Cleanup IO accessors Marek Vasut
2011-12-11 1:09 ` Marek Vasut [this message]
2011-12-11 7:24 ` Mike Frysinger
2011-12-11 13:42 ` Marek Vasut
2011-12-12 5:45 ` Mike Frysinger
2011-12-12 10:17 ` Marek Vasut
2011-12-16 17:33 ` Mike Frysinger
2011-12-16 18:13 ` Marek Vasut
2011-12-16 19:32 ` Mike Frysinger
2011-12-16 20:36 ` Marek Vasut
2012-07-10 21:41 ` Joe Hershberger
2012-07-13 10:50 ` Marek Vasut
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=1323565772-4498-2-git-send-email-marek.vasut@gmail.com \
--to=marek.vasut@gmail.com \
--cc=u-boot@lists.denx.de \
/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