mirror of
https://github.com/AsahiLinux/u-boot
synced 2024-11-24 21:54:01 +00:00
net: ks8851: Clean up chip ID readout
There is only one chip ID in the table of chip IDs for this chip. Read out the chip ID instead and mask off the last "revision" bit to check the chip ID, this works for all chips in the family. Then drop the chip ID passing around. Signed-off-by: Marek Vasut <marex@denx.de> Cc: Eugen Hristev <eugen.hristev@microchip.com> Cc: Joe Hershberger <joe.hershberger@ni.com>
This commit is contained in:
parent
63f22f5998
commit
eb69d8bf3c
1 changed files with 2 additions and 19 deletions
|
@ -19,11 +19,6 @@
|
|||
|
||||
#define RX_BUF_SIZE 2000
|
||||
|
||||
static const struct chip_id chip_ids[] = {
|
||||
{CIDER_ID, "KSZ8851"},
|
||||
{0, NULL},
|
||||
};
|
||||
|
||||
/*
|
||||
* struct ks_net - KS8851 driver private data
|
||||
* @bus_width : i/o bus width.
|
||||
|
@ -333,7 +328,7 @@ static void ks_setup_int(struct eth_device *dev)
|
|||
|
||||
static int ks8851_mll_detect_chip(struct eth_device *dev)
|
||||
{
|
||||
unsigned short val, i;
|
||||
unsigned short val;
|
||||
|
||||
ks_read_config(dev);
|
||||
|
||||
|
@ -350,19 +345,11 @@ static int ks8851_mll_detect_chip(struct eth_device *dev)
|
|||
|
||||
debug("Read back KS8851 id 0x%x\n", val);
|
||||
|
||||
/* only one entry in the table */
|
||||
val &= 0xfff0;
|
||||
for (i = 0; chip_ids[i].id != 0; i++) {
|
||||
if (chip_ids[i].id == val)
|
||||
break;
|
||||
}
|
||||
if (!chip_ids[i].id) {
|
||||
if ((val & 0xfff0) != CIDER_ID) {
|
||||
printf(DRIVERNAME ": Unknown chip ID %04x\n", val);
|
||||
return -1;
|
||||
}
|
||||
|
||||
dev->priv = (void *)&chip_ids[i];
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -406,10 +393,6 @@ static void ks8851_mll_enable(struct eth_device *dev)
|
|||
|
||||
static int ks8851_mll_init(struct eth_device *dev, bd_t *bd)
|
||||
{
|
||||
struct chip_id *id = dev->priv;
|
||||
|
||||
debug(DRIVERNAME ": detected %s controller\n", id->name);
|
||||
|
||||
if (ks_read_selftest(dev)) {
|
||||
printf(DRIVERNAME ": Selftest failed\n");
|
||||
return -1;
|
||||
|
|
Loading…
Reference in a new issue