[PATCH] Continue pcnet probing. Introduce driver's private block

[Dmitry Podgorny]

В Fri, 17 Sep 2010 19:02:38 +0400
Denis Kirjanov <kirjanov at gmail.com> пишет:

> +/* Why do we need to create protos like this? */
> +static int pcnet_dummy_init_netdev(struct pci_dev *pdev, unsigned
> long ioaddr);

Нужны ли вообще прототипы функций? Мне показалось, что удобнее
разместить pcnet_dummy_init_netdev после функции инициализации, но я
тут больше на правах ученика, так что рад выслушать любые
предложения :).

> +	ndev = alloc_etherdev(sizeof(*pp));
> +	SET_NETDEV_DEV(ndev, &pdev->dev);

Почему мы выполняем действия с net_device не закончив инициализацию pci
устройства? Здесь есть какая-то специфика? Я планировал вынести
подобные действия в pcnet_dummy_init_netdev() или здесь я не прав?

>  static void __devexit pcnet_dummy_remove_one(struct pci_dev *pdev)

Будет ли вызвана эта функция, если инициализация устройства завершилась
неудачей и мы возвратили -ENODEV?