starfive-tech/u-boot.git/drivers/net/rtl8139.c, branch rtthread_AMP

dm: treewide: Rename dev_get_platdata() to dev_get_plat()

2020-12-13T23:51:09+00:00

Rename this to be consistent with the change from 'platdata'. Signed-off-by: Simon Glass

dm: treewide: Rename 'platdata' variables to just 'plat'

2020-12-13T23:51:08+00:00

We use 'priv' for private data but often use 'platdata' for platform data. We can't really use 'pdata' since that is ambiguous (it could mean private or platform data). Rename some of the latter variables to end with 'plat' for consistency. Signed-off-by: Simon Glass

dm: treewide: Rename auto_alloc_size members to be shorter

2020-12-13T15:00:25+00:00

This construct is quite long-winded. In earlier days it made some sense since auto-allocation was a strange concept. But with driver model now used pretty universally, we can shorten this to 'auto'. This reduces verbosity and makes it easier to read. Coincidentally it also ensures that every declaration is on one line, thus making dtoc's job easier. Signed-off-by: Simon Glass

treewide: convert bd_t to struct bd_info by coccinelle

2020-07-17T13:30:13+00:00

The Linux coding style guide (Documentation/process/coding-style.rst) clearly says: It's a **mistake** to use typedef for structures and pointers. Besides, using typedef for structures is annoying when you try to make headers self-contained. Let's say you have the following function declaration in a header: void foo(bd_t *bd); This is not self-contained since bd_t is not defined. To tell the compiler what 'bd_t' is, you need to include #include void foo(bd_t *bd); Then, the include direcective pulls in more bloat needlessly. If you use 'struct bd_info' instead, it is enough to put a forward declaration as follows: struct bd_info; void foo(struct bd_info *bd); Right, typedef'ing bd_t is a mistake. I used coccinelle to generate this commit. The semantic patch that makes this change is as follows: @@ typedef bd_t; @@ -bd_t +struct bd_info Signed-off-by: Masahiro Yamada

net: rtl8139: Add DM support

2020-06-12T17:17:23+00:00

Add support for driver model to the driver. Signed-off-by: Marek Vasut Cc: Joe Hershberger

net: rtl8139: Read HW address from EEPROM only on probe

2020-06-12T17:17:23+00:00

Do not re-read the HW address from the EEPROM on every start of transfer, otherwise the user will not be able to adjust ethaddr as needed. Read the address only once, when the card is detected. Signed-off-by: Marek Vasut Cc: Joe Hershberger

net: rtl8139: Use PCI_DEVICE() to define PCI device compat list

2020-06-12T17:17:23+00:00

Use this macro to fully fill the PCI device ID table. This is mandatory for the DM PCI support, which checks all the fields. Signed-off-by: Marek Vasut Cc: Joe Hershberger

net: rtl8139: Split out common and non-DM functions

2020-06-12T17:17:23+00:00

Split the driver into common and non-DM functionality, so that the DM support can later re-use the common code, while we retain the non-DM code until all the platforms are converted. Signed-off-by: Marek Vasut Cc: Joe Hershberger

net: rtl8139: Pass private data into rtl8139_eeprom_delay()

2020-06-12T17:17:23+00:00

Instead of always calling rtl8139_eeprom_delay() with priv->ioaddr, call it with priv and let the function access priv->ioaddr. This reduces code duplication and has no impact, since the compiler will inline this as needed anyway. Signed-off-by: Marek Vasut Cc: Joe Hershberger

net: rtl8139: Introduce device private data

2020-06-12T17:17:23+00:00

Introduce rtl8139_pdata, which is a super-structure around eth_device and tracks per-device state, here the device IO address, PCI BDF, RX and TX ring position. Pass this structure around instead of the old non-DM eth_device in preparation for DM conversion. Signed-off-by: Marek Vasut Cc: Joe Hershberger