McBSP support for 34XX
cover McBSP support for 34XX
Hi,
Here is a patch that adds support for McBSP on 34XX. It has been verified in digital loopback mode.
The patch includes the following features.
1. McBSP enabled in defconfig
2. Supporting all the LP features.
3. Support of DMA chaining mode and one time configuration for entire transfer to improve performance.
4. Independent configurations and transfer of Rx and Tx
5. MCBSP data transfer in 8, 16 and 32 bit mode.
6. instead of mcbsp client configuring all the register, it allows clients to configure parameters like clk polarity, clk src and other parameters.
Regards
Chandra Shekhar
Quoted 5 lines
Hi,
Here is a patch that adds support for McBSP on 34XX. It has been verified in digital loopback mode.
Hi,
I tested the set with N810 ASoC audio and still plays :-)
Some comments. First, commit messages should more descriptive than "McBSP support for 34XX" for all of them. Like:
1. Add support for OMAP34xx McBSP ports 3-4
2. Fix register access width for OMAP34xx (this should be -rc patch?)
3. ...
Quoted 3 lines
The patch includes the following features.
1. McBSP enabled in defconfig
This should go as a separate patch since it is for 3430dsp only.
Quoted 6 lines
2. Supporting all the LP features.
3. Support of DMA chaining mode and one time configuration for entire transfer to improve performance.
4. Independent configurations and transfer of Rx and Tx
5. MCBSP data transfer in 8, 16 and 32 bit mode.
I think we should have some expected use case for these before adding the features?
Quoted 4 lines
6. instead of mcbsp client configuring all the register, it allows clients to configure parameters like clk polarity, clk src and other parameters.
This would be good direction. E.g. it's bit hacky how configuration is now done in sound/soc/omap/omap-mcbsp.c where McBSP register cache is have to use because all of the registers are written in once with omap_mcbsp_config.
Jarkko
thanks for the comments
i will make the necessary changes (for your and tony's comments) and resend it asap.
----- Original Message ----- From: "Jarkko Nikula" <[email protected]>
To: "ext Chandra shekhar" <[email protected]>
Cc: <[email protected]>
Sent: Wednesday, June 18, 2008 12:46 PM
Subject: Re: [RFC/PATCH 0/4] McBSP support for 34XX
Quoted 41 lines
Quoted 5 lines
Hi,
Here is a patch that adds support for McBSP on 34XX. It has been verified in digital loopback mode.
Hi,
I tested the set with N810 ASoC audio and still plays :-)
Some comments. First, commit messages should more descriptive than "McBSP support for 34XX" for all of them. Like:
1. Add support for OMAP34xx McBSP ports 3-4
2. Fix register access width for OMAP34xx (this should be -rc patch?)
3. ...Quoted 3 lines
The patch includes the following features.
1. McBSP enabled in defconfig
This should go as a separate patch since it is for 3430dsp only.
Quoted 6 lines
2. Supporting all the LP features.
3. Support of DMA chaining mode and one time configuration for entire transfer to improve performance.
4. Independent configurations and transfer of Rx and Tx
5. MCBSP data transfer in 8, 16 and 32 bit mode.I think we should have some expected use case for these before adding the features?
Quoted 4 lines
6. instead of mcbsp client configuring all the register, it allows clients to configure parameters like clk polarity, clk src and other parameters.
This would be good direction. E.g. it's bit hacky how configuration is now done in
sound/soc/omap/omap-mcbsp.cwhere McBSP register cache is have to use because all of the registers are written in once withomap_mcbsp_config.Jarkko