[RFC]PPC driver lock for DMA registers

3 messages, 2 authors, 2009-03-18 · open the first message on its own page

[RFC]PPC driver lock for DMA registers

From: Eddie Dawydiuk <hidden>
Date: 2009-03-18 16:26:39

Hello,

I'm working on SD card and Nand drivers that I would like to eventually submit 
for inclusion in the mainline kernel. This being my first kernel port being 
submitted upstream I was hoping for comments on my proposed design to ensure it 
would be excepted in the mainline kernel(from a design standpoint). We have an 
FPGA on our product that has an interface for transferring data via DMA. The SD 
card driver and Nand flash driver will both be accessing these registers. As a 
result I was planning on writing a dma driver(drivers/dma/ts81xx.c) that 
implements a function dma_op() that gets passed a physical address, virtual 
address, length, and read/write op and returns an int. I'd use a semaphore to 
protect the registers. I'd also implement a function dma_bus_init() for 
allocating a cache coherent buffer, initializing the semaphore, and ioremapping 
the FPGA regs.

I'd really appreciate any feedback or suggestions.

-- 
Best Regards,
________________________________________________________________
  Eddie Dawydiuk, Technologic Systems | voice:  (480) 837-5200
  16525 East Laser Drive 	     | fax:    (480) 837-5300
  Fountain Hills, AZ 85268            | web: www.embeddedARM.com

Re: [RFC]PPC driver lock for DMA registers

From: Grant Likely <hidden>
Date: 2009-03-18 16:40:50

On Wed, Mar 18, 2009 at 10:29 AM, Eddie Dawydiuk [off-list ref] wrote:
Hello,

I'm working on SD card and Nand drivers that I would like to eventually
submit for inclusion in the mainline kernel. This being my first kernel port
being submitted upstream I was hoping for comments on my proposed design to
ensure it would be excepted in the mainline kernel(from a design
standpoint). We have an FPGA on our product that has an interface for
transferring data via DMA. The SD card driver and Nand flash driver will
both be accessing these registers. As a result I was planning on writing a
dma driver(drivers/dma/ts81xx.c) that implements a function dma_op() that
gets passed a physical address, virtual address, length, and read/write op
and returns an int. I'd use a semaphore to protect the registers. I'd also
implement a function dma_bus_init() for allocating a cache coherent buffer,
initializing the semaphore, and ioremapping the FPGA regs.
Nothing that you describe here sounds off base, but the proof is in
the pudding so they say.  Best advice I can offer is to follow the
example of existing powerpc device drivers.  You'll need to describe
your FPGA interface in the .dts file and write an of_platform driver
that binds against it.  Make sure you post your proposed dts
description of the FPGA to the devicetree-discuss@ozlabs.org mailing
list before committing to it.  In you're device drivers .probe() hook
you would create and register the NAND and SD devices with the kernel
with the needed hooks to access the FPGA.

Here are some good examples of of_platform drivers to look at:

drivers/block/xsysace.c
drivers/spi/xilinx_spi.c
drivers/i2c/busses/i2c-ibm_iic.c

Cheers,
g.

-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.

Re: [RFC]PPC driver lock for DMA registers

From: Eddie Dawydiuk <hidden>
Date: 2009-03-18 16:50:42

Hello,
quoted
I'm working on SD card and Nand drivers that I would like to eventually
submit for inclusion in the mainline kernel. This being my first kernel port
being submitted upstream I was hoping for comments on my proposed design to
ensure it would be excepted in the mainline kernel(from a design
standpoint). We have an FPGA on our product that has an interface for
transferring data via DMA. The SD card driver and Nand flash driver will
both be accessing these registers. As a result I was planning on writing a
dma driver(drivers/dma/ts81xx.c) that implements a function dma_op() that
gets passed a physical address, virtual address, length, and read/write op
and returns an int. I'd use a semaphore to protect the registers. I'd also
implement a function dma_bus_init() for allocating a cache coherent buffer,
initializing the semaphore, and ioremapping the FPGA regs.
Nothing that you describe here sounds off base, but the proof is in
the pudding so they say.  Best advice I can offer is to follow the
example of existing powerpc device drivers.  You'll need to describe
your FPGA interface in the .dts file and write an of_platform driver
that binds against it.  Make sure you post your proposed dts
description of the FPGA to the devicetree-discuss@ozlabs.org mailing
list before committing to it.  In you're device drivers .probe() hook
you would create and register the NAND and SD devices with the kernel
with the needed hooks to access the FPGA.

Here are some good examples of of_platform drivers to look at:

drivers/block/xsysace.c
drivers/spi/xilinx_spi.c
drivers/i2c/busses/i2c-ibm_iic.c
Thanks for the suggestions, and the pointer to some reference drivers ;)


-- 
Best Regards,
________________________________________________________________
  Eddie Dawydiuk, Technologic Systems | voice:  (480) 837-5200
  16525 East Laser Drive 	     | fax:    (480) 837-5300
  Fountain Hills, AZ 85268            | web: www.embeddedARM.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help