netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] moving from pci to dma
@ 2015-01-12 10:02 Quentin Lambert
  2015-01-12 15:27 ` chas williams - CONTRACTOR
  0 siblings, 1 reply; 4+ messages in thread
From: Quentin Lambert @ 2015-01-12 10:02 UTC (permalink / raw)
  To: Chas Williams; +Cc: linux-atm-general, netdev, linux-kernel

This patch replaces the references to the deprecated pci api with the
corresponding dma api.

To ensure that it was possible to access the dev field of pci_dev without
checking for nullity we looked for similar access in the execution
flow.

The most straight forward are "&id->dev" and "pci_get_drvdata(id)" where
id is the variable whose type is pci_dev.

We also found "pci_enable_device(id)" to be satisfying since the call
accesses other field without checking for nullity.

Quentin Lambert (1):
  atm: remove deprecated use of pci api

 drivers/atm/eni.c       | 8 +++++---
 drivers/atm/he.c        | 2 +-
 drivers/atm/lanai.c     | 9 +++++----
 drivers/atm/nicstar.c   | 4 ++--
 drivers/atm/solos-pci.c | 2 +-
 drivers/atm/zatm.c      | 8 +++++---
 6 files changed, 19 insertions(+), 14 deletions(-)

-- 
1.9.1

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] moving from pci to dma
  2015-01-12 10:02 [PATCH] moving from pci to dma Quentin Lambert
@ 2015-01-12 15:27 ` chas williams - CONTRACTOR
  2015-01-12 15:32   ` Quentin Lambert
  0 siblings, 1 reply; 4+ messages in thread
From: chas williams - CONTRACTOR @ 2015-01-12 15:27 UTC (permalink / raw)
  To: Quentin Lambert; +Cc: linux-atm-general, netdev, linux-kernel

There doesn't seem to be a patch for review?

On Mon, 12 Jan 2015 11:02:39 +0100
Quentin Lambert <lambert.quentin@gmail.com> wrote:

> This patch replaces the references to the deprecated pci api with the
> corresponding dma api.
...
> Quentin Lambert (1):
>   atm: remove deprecated use of pci api
> 
>  drivers/atm/eni.c       | 8 +++++---
>  drivers/atm/he.c        | 2 +-
>  drivers/atm/lanai.c     | 9 +++++----
>  drivers/atm/nicstar.c   | 4 ++--
>  drivers/atm/solos-pci.c | 2 +-
>  drivers/atm/zatm.c      | 8 +++++---
>  6 files changed, 19 insertions(+), 14 deletions(-)
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] moving from pci to dma
  2015-01-12 15:27 ` chas williams - CONTRACTOR
@ 2015-01-12 15:32   ` Quentin Lambert
  2015-01-12 15:39     ` chas williams - CONTRACTOR
  0 siblings, 1 reply; 4+ messages in thread
From: Quentin Lambert @ 2015-01-12 15:32 UTC (permalink / raw)
  To: chas williams - CONTRACTOR; +Cc: linux-atm-general, netdev, linux-kernel


On 12/01/2015 16:27, chas williams - CONTRACTOR wrote:
> There doesn't seem to be a patch for review?
>
I made a mistake and forgot to number the mails. I did send them though.
Would you like me to send them again, with the proper subject format?


Quentin Lambert

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] moving from pci to dma
  2015-01-12 15:32   ` Quentin Lambert
@ 2015-01-12 15:39     ` chas williams - CONTRACTOR
  0 siblings, 0 replies; 4+ messages in thread
From: chas williams - CONTRACTOR @ 2015-01-12 15:39 UTC (permalink / raw)
  To: Quentin Lambert; +Cc: linux-atm-general, netdev, linux-kernel

On Mon, 12 Jan 2015 16:32:46 +0100
Quentin Lambert <lambert.quentin@gmail.com> wrote:

> 
> On 12/01/2015 16:27, chas williams - CONTRACTOR wrote:
> > There doesn't seem to be a patch for review?
> >
> I made a mistake and forgot to number the mails. I did send them though.
> Would you like me to send them again, with the proper subject format?
> 
> 
> Quentin Lambert
> 

That would be nice.  Thanks!

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-01-12 15:39 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-12 10:02 [PATCH] moving from pci to dma Quentin Lambert
2015-01-12 15:27 ` chas williams - CONTRACTOR
2015-01-12 15:32   ` Quentin Lambert
2015-01-12 15:39     ` chas williams - CONTRACTOR

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).