Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-24 Thread Shannon Nelson
On 6/24/19 1:53 PM, Jakub Kicinski wrote: On Thu, 20 Jun 2019 13:24:08 -0700, Shannon Nelson wrote: The ionic device has a small set of PCI registers, including a device control and data space, and a large set of message commands. Signed-off-by: Shannon Nelson struct ionic { struct p

Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-24 Thread Shannon Nelson
On 6/24/19 1:13 PM, Jakub Kicinski wrote: On Fri, 21 Jun 2019 15:22:22 -0700, Shannon Nelson wrote: +static int identity_show(struct seq_file *seq, void *v) +{ + struct ionic *ionic = seq->private; + struct identity *ident = &ionic->ident; + struct ionic_dev *idev = &ionic->ide

Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-24 Thread Jakub Kicinski
On Thu, 20 Jun 2019 13:24:08 -0700, Shannon Nelson wrote: > The ionic device has a small set of PCI registers, including a > device control and data space, and a large set of message > commands. > > Signed-off-by: Shannon Nelson > struct ionic { > struct pci_dev *pdev; > struct devi

Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-24 Thread Jakub Kicinski
On Fri, 21 Jun 2019 15:22:22 -0700, Shannon Nelson wrote: > >> +static int identity_show(struct seq_file *seq, void *v) > >> +{ > >> + struct ionic *ionic = seq->private; > >> + struct identity *ident = &ionic->ident; > >> + struct ionic_dev *idev = &ionic->idev; > >> + > >> + seq_printf(seq, "

Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-21 Thread Shannon Nelson
On 6/20/19 2:54 PM, Andrew Lunn wrote: On Thu, Jun 20, 2019 at 01:24:08PM -0700, Shannon Nelson wrote: + err = ionic_debugfs_add_dev(ionic); + if (err) { + dev_err(dev, "Cannot add device debugfs: %d , aborting\n", err); + goto err_out_clear_drvdata; +

Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-21 Thread kbuild test robot
Hi Shannon, I love your patch! Yet something to improve: [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Shannon-Nelson/ionic-Add-basic-framework-for-IONIC-Network-device-driver/20190621-110046 config: arm-allyesconfig (attached as .config) compiler:

Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-21 Thread kbuild test robot
Hi Shannon, I love your patch! Yet something to improve: [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Shannon-Nelson/ionic-Add-basic-framework-for-IONIC-Network-device-driver/20190621-110046 config: i386-allmodconfig (attached as .config) compiler:

Re: [PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-20 Thread Andrew Lunn
On Thu, Jun 20, 2019 at 01:24:08PM -0700, Shannon Nelson wrote: > + err = ionic_debugfs_add_dev(ionic); > + if (err) { > + dev_err(dev, "Cannot add device debugfs: %d , aborting\n", err); > + goto err_out_clear_drvdata; > + } Hi Shannon debugfs should not fail,

[PATCH net-next 02/18] ionic: Add hardware init and device commands

2019-06-20 Thread Shannon Nelson
The ionic device has a small set of PCI registers, including a device control and data space, and a large set of message commands. Signed-off-by: Shannon Nelson --- drivers/net/ethernet/pensando/ionic/Makefile |2 +- drivers/net/ethernet/pensando/ionic/ionic.h | 21 + .../net/ethernet/p