Re: [PATCH v5 0/6] OMAP: McSPI: Hwmod adaptation + runtime conversion

2011-01-25 Thread Govindraj
On Tue, Jan 25, 2011 at 3:56 AM, Kevin Hilman khil...@ti.com wrote:
 Hi Govindraj,

 Govindraj.R govindraj.r...@ti.com writes:

 Changes invloves:
 
 1) Addition of hwmod data for omap2/3/4.
 2) McSPI driver hwmod adaptation with cleanup of base address
    macros and using omap-device API's.
 3) Runtime Conversion of McSPI driver.

 I accidentally replied to the v4 series about the refresh, but this one
 still needs ar refresh.

 Specifically, this change from Gregory Clement touches some of the same
 code as your patch:

 commit 42ce7fd6319bed8ecb26d656c476365da46b29e9
 Author: Gregory CLEMENT gregory.clem...@free-electrons.com
 Date:   Wed Dec 29 11:52:53 2010 +0100

    spi/omap2_mcspi.c: Force CS to be in inactive state after off-mode 
 transition


Yes correct.

Will rebase and repost in couple of days.

--
Thanks
Govindraj.R



 Kevin


 Changes from v4:
 ---
 1) 4430 hwmod file alignment based on Benoit's comments.
    http://www.spinics.net/lists/arm-kernel/msg111215.html
    4430 Hwmod file now aligned based on:
    http://gitorious.org/omap-pm/linux/blobs/pm-wip/
    hwmods-omap4-full/arch/arm/mach-omap2/omap_hwmod_44xx_data.c

 Changes from v3:
 ---
 1) Updated proper Author for all patches which was missed
    in v2.
 2) Modified 4430 hwmod data file so that mcspi data gets
    updated in proper alphabetical order.
 3) Update omap2/3 hwmod dat files with SYSS_HAS_RESET_STATUS
    flag.

 Changes from v2:
 ---
 1) Fixing minor comments and adding ack from Grant Likely.
       https://patchwork.kernel.org/patch/371321/
       https://patchwork.kernel.org/patch/371331/

 Changes from v1:
 ---
 1) Fixing patch 5/5 comments for hwmod+runtime
   Split the patch 5/5 to hwmod adaptation
   and then runtime conversion
   http://www.mail-archive.com/linux-omap@vger.kernel.org/msg33387.html

 Testing Updates:
 
 Was tested using data transfer test module available at:
 http://dev.omapzoom.org/?p=richo/device_driver_test.git;a=blob;f=mcspi/test_code/
 utils/mcspi_modules/omap_mcspi_datatest.c;
 h=e42ec10c5c844abdde6a7175a268b379fbbdb655;
 hb=5d9a755d50e58de861c5e8991f2f607bc49b5dc3
 This test basically involves MISO -- MOSI lines looped and data transfer 
 test
 done using the above test module.
 System wide suspend and ret/off counts observation, ensured that no 
 behavioral
 difference with and without this patch series.

 Benoit Cousson (1):
   OMAP4: hwmod data: Add McSPI

 Charulatha V (4):
   OMAP2420: hwmod data: Add McSPI
   OMAP2430: hwmod data: Add McSPI
   OMAP3: hwmod data: Add McSPI
   OMAP: devices: Modify McSPI device to adapt to hwmod framework

 Govindraj.R (1):
   OMAP: runtime: McSPI driver runtime conversion

  arch/arm/mach-omap2/devices.c              |  187 ---
  arch/arm/mach-omap2/omap_hwmod_2420_data.c |  156 
  arch/arm/mach-omap2/omap_hwmod_2430_data.c |  219 ++
  arch/arm/mach-omap2/omap_hwmod_3xxx_data.c |  280 
 
  arch/arm/mach-omap2/omap_hwmod_44xx_data.c |  267 ++
  arch/arm/plat-omap/include/plat/mcspi.h    |   11 +
  drivers/spi/omap2_mcspi.c                  |  224 +++---
  7 files changed, 1044 insertions(+), 300 deletions(-)
 --
 To unsubscribe from this list: send the line unsubscribe linux-omap in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/6] OMAP: McSPI: Hwmod adaptation + runtime conversion

2011-01-24 Thread Kevin Hilman
Hi Govindraj,

Govindraj.R govindraj.r...@ti.com writes:

 Changes invloves:
 
 1) Addition of hwmod data for omap2/3/4.
 2) McSPI driver hwmod adaptation with cleanup of base address
macros and using omap-device API's.
 3) Runtime Conversion of McSPI driver.

I accidentally replied to the v4 series about the refresh, but this one
still needs ar refresh.

Specifically, this change from Gregory Clement touches some of the same
code as your patch:

commit 42ce7fd6319bed8ecb26d656c476365da46b29e9
Author: Gregory CLEMENT gregory.clem...@free-electrons.com
Date:   Wed Dec 29 11:52:53 2010 +0100

spi/omap2_mcspi.c: Force CS to be in inactive state after off-mode 
transition


Kevin


 Changes from v4:
 ---
 1) 4430 hwmod file alignment based on Benoit's comments.
http://www.spinics.net/lists/arm-kernel/msg111215.html
4430 Hwmod file now aligned based on:
http://gitorious.org/omap-pm/linux/blobs/pm-wip/
hwmods-omap4-full/arch/arm/mach-omap2/omap_hwmod_44xx_data.c

 Changes from v3:
 ---
 1) Updated proper Author for all patches which was missed
in v2. 
 2) Modified 4430 hwmod data file so that mcspi data gets
updated in proper alphabetical order.
 3) Update omap2/3 hwmod dat files with SYSS_HAS_RESET_STATUS
flag.

 Changes from v2:
 ---
 1) Fixing minor comments and adding ack from Grant Likely.
   https://patchwork.kernel.org/patch/371321/
   https://patchwork.kernel.org/patch/371331/

 Changes from v1:
 ---
 1) Fixing patch 5/5 comments for hwmod+runtime
   Split the patch 5/5 to hwmod adaptation
   and then runtime conversion
   http://www.mail-archive.com/linux-omap@vger.kernel.org/msg33387.html

 Testing Updates:
 
 Was tested using data transfer test module available at:
 http://dev.omapzoom.org/?p=richo/device_driver_test.git;a=blob;f=mcspi/test_code/
 utils/mcspi_modules/omap_mcspi_datatest.c;
 h=e42ec10c5c844abdde6a7175a268b379fbbdb655;
 hb=5d9a755d50e58de861c5e8991f2f607bc49b5dc3
 This test basically involves MISO -- MOSI lines looped and data transfer 
 test
 done using the above test module.
 System wide suspend and ret/off counts observation, ensured that no behavioral
 difference with and without this patch series.

 Benoit Cousson (1):
   OMAP4: hwmod data: Add McSPI

 Charulatha V (4):
   OMAP2420: hwmod data: Add McSPI
   OMAP2430: hwmod data: Add McSPI
   OMAP3: hwmod data: Add McSPI
   OMAP: devices: Modify McSPI device to adapt to hwmod framework

 Govindraj.R (1):
   OMAP: runtime: McSPI driver runtime conversion

  arch/arm/mach-omap2/devices.c  |  187 ---
  arch/arm/mach-omap2/omap_hwmod_2420_data.c |  156 
  arch/arm/mach-omap2/omap_hwmod_2430_data.c |  219 ++
  arch/arm/mach-omap2/omap_hwmod_3xxx_data.c |  280 
 
  arch/arm/mach-omap2/omap_hwmod_44xx_data.c |  267 ++
  arch/arm/plat-omap/include/plat/mcspi.h|   11 +
  drivers/spi/omap2_mcspi.c  |  224 +++---
  7 files changed, 1044 insertions(+), 300 deletions(-)
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v5 0/6] OMAP: McSPI: Hwmod adaptation + runtime conversion

2011-01-21 Thread Govindraj.R
Changes invloves:

1) Addition of hwmod data for omap2/3/4.
2) McSPI driver hwmod adaptation with cleanup of base address
   macros and using omap-device API's.
3) Runtime Conversion of McSPI driver.

Changes from v4:
---
1) 4430 hwmod file alignment based on Benoit's comments.
   http://www.spinics.net/lists/arm-kernel/msg111215.html
   4430 Hwmod file now aligned based on:
   http://gitorious.org/omap-pm/linux/blobs/pm-wip/
   hwmods-omap4-full/arch/arm/mach-omap2/omap_hwmod_44xx_data.c

Changes from v3:
---
1) Updated proper Author for all patches which was missed
   in v2. 
2) Modified 4430 hwmod data file so that mcspi data gets
   updated in proper alphabetical order.
3) Update omap2/3 hwmod dat files with SYSS_HAS_RESET_STATUS
   flag.

Changes from v2:
---
1) Fixing minor comments and adding ack from Grant Likely.
https://patchwork.kernel.org/patch/371321/
https://patchwork.kernel.org/patch/371331/

Changes from v1:
---
1) Fixing patch 5/5 comments for hwmod+runtime
  Split the patch 5/5 to hwmod adaptation
  and then runtime conversion
  http://www.mail-archive.com/linux-omap@vger.kernel.org/msg33387.html

Testing Updates:

Was tested using data transfer test module available at:
http://dev.omapzoom.org/?p=richo/device_driver_test.git;a=blob;f=mcspi/test_code/
utils/mcspi_modules/omap_mcspi_datatest.c;
h=e42ec10c5c844abdde6a7175a268b379fbbdb655;
hb=5d9a755d50e58de861c5e8991f2f607bc49b5dc3
This test basically involves MISO -- MOSI lines looped and data transfer test
done using the above test module.
System wide suspend and ret/off counts observation, ensured that no behavioral
difference with and without this patch series.

Benoit Cousson (1):
  OMAP4: hwmod data: Add McSPI

Charulatha V (4):
  OMAP2420: hwmod data: Add McSPI
  OMAP2430: hwmod data: Add McSPI
  OMAP3: hwmod data: Add McSPI
  OMAP: devices: Modify McSPI device to adapt to hwmod framework

Govindraj.R (1):
  OMAP: runtime: McSPI driver runtime conversion

 arch/arm/mach-omap2/devices.c  |  187 ---
 arch/arm/mach-omap2/omap_hwmod_2420_data.c |  156 
 arch/arm/mach-omap2/omap_hwmod_2430_data.c |  219 ++
 arch/arm/mach-omap2/omap_hwmod_3xxx_data.c |  280 
 arch/arm/mach-omap2/omap_hwmod_44xx_data.c |  267 ++
 arch/arm/plat-omap/include/plat/mcspi.h|   11 +
 drivers/spi/omap2_mcspi.c  |  224 +++---
 7 files changed, 1044 insertions(+), 300 deletions(-)

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html