Re: [PATCH 2/6 V4] OMAP4: Adding the temperature sensor register set bit fields

2011-08-31 Thread Paul Walmsley
On Wed, 31 Aug 2011, Keerthy wrote:

 OMAP4460 specific temperature sensor register bit fields are added.
 Existing OMAP4 entries are renamed to OMAP4430.
 
 Signed-off-by: Keerthy j-keer...@ti.com
 Cc: t...@atomide.com

At least one of these bitfields are incorrect.  Please double-check them.

 ---
  .../include/mach/ctrl_module_core_44xx.h   |   70 
 
  1 files changed, 57 insertions(+), 13 deletions(-)
 
 diff --git a/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h 
 b/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h
 index 2f7ac70..725c1e1 100644
 --- a/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h
 +++ b/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h
 @@ -256,19 +256,63 @@
  #define OMAP4_LDOSRAMCORE_ACTMODE_VSET_OUT_SHIFT 0
  #define OMAP4_LDOSRAMCORE_ACTMODE_VSET_OUT_MASK  (0x1f  0)
  
 -/* TEMP_SENSOR */
 -#define OMAP4_BGAP_TEMPSOFF_SHIFT12
 -#define OMAP4_BGAP_TEMPSOFF_MASK (1  12)
 -#define OMAP4_BGAP_TSHUT_SHIFT   11
 -#define OMAP4_BGAP_TSHUT_MASK(1  11)
 -#define OMAP4_BGAP_TEMP_SENSOR_CONTCONV_SHIFT10
 -#define OMAP4_BGAP_TEMP_SENSOR_CONTCONV_MASK (1  10)
 -#define OMAP4_BGAP_TEMP_SENSOR_SOC_SHIFT 9
 -#define OMAP4_BGAP_TEMP_SENSOR_SOC_MASK  (1  9)
 -#define OMAP4_BGAP_TEMP_SENSOR_EOCZ_SHIFT8
 -#define OMAP4_BGAP_TEMP_SENSOR_EOCZ_MASK (1  8)
 -#define OMAP4_BGAP_TEMP_SENSOR_DTEMP_SHIFT   0
 -#define OMAP4_BGAP_TEMP_SENSOR_DTEMP_MASK(0xff  0)
 +/* TEMP_SENSOR OMAP4430 */
 +#define OMAP4430_BGAP_TEMPSOFF_SHIFT 12
 +#define OMAP4430_BGAP_TEMPSOFF_MASK  (1  12)
 +#define OMAP4430_BGAP_TSHUT_SHIFT11
 +#define OMAP4430_BGAP_TSHUT_MASK (1  11)
 +#define OMAP4430_BGAP_TEMP_SENSOR_CONTCONV_SHIFT 10
 +#define OMAP4430_BGAP_TEMP_SENSOR_CONTCONV_MASK  (1  10)
 +#define OMAP4430_BGAP_TEMP_SENSOR_SOC_SHIFT  9
 +#define OMAP4430_BGAP_TEMP_SENSOR_SOC_MASK   (1  9)
 +#define OMAP4430_BGAP_TEMP_SENSOR_EOCZ_SHIFT 8
 +#define OMAP4430_BGAP_TEMP_SENSOR_EOCZ_MASK  (1  8)
 +#define OMAP4430_BGAP_TEMP_SENSOR_DTEMP_SHIFT0
 +#define OMAP4430_BGAP_TEMP_SENSOR_DTEMP_MASK (0x3ff  0)

For example, this bitfield is only eight bits wide on 4430.

 +
 +/* TEMP_SENSOR OMAP4460 */
 +#define OMAP4460_BGAP_TEMPSOFF_SHIFT 13
 +#define OMAP4460_BGAP_TEMPSOFF_MASK  (1  13)
 +#define OMAP4460_BGAP_TEMP_SENSOR_SOC_SHIFT  11
 +#define OMAP4460_BGAP_TEMP_SENSOR_SOC_MASK   (1  11)
 +#define OMAP4460_BGAP_TEMP_SENSOR_EOCZ_SHIFT 10
 +#define OMAP4460_BGAP_TEMP_SENSOR_EOCZ_MASK  (1  10)
 +#define OMAP4460_BGAP_TEMP_SENSOR_DTEMP_SHIFT0
 +#define OMAP4460_BGAP_TEMP_SENSOR_DTEMP_MASK (0x3ff  0)
 +
 +/* BANDGAP_CTRL */
 +#define OMAP4460_SINGLE_MODE_SHIFT   31
 +#define OMAP4460_SINGLE_MODE_MASK(1  31)
 +#define OMAP4460_MASK_HOT_SHIFT  1
 +#define OMAP4460_MASK_HOT_MASK   (1  1)
 +#define OMAP4460_MASK_COLD_SHIFT 0
 +#define OMAP4460_MASK_COLD_MASK  (1  0)
 +
 +/* BANDGAP_COUNTER */
 +#define OMAP4460_COUNTER_SHIFT   0
 +#define OMAP4460_COUNTER_MASK(0xff  0)
 +
 +/* BANDGAP_THRESHOLD */
 +#define OMAP4460_T_HOT_SHIFT 16
 +#define OMAP4460_T_HOT_MASK  (0x3ff  16)
 +#define OMAP4460_T_COLD_SHIFT0
 +#define OMAP4460_T_COLD_MASK (0x3ff  0)
 +
 +/* TSHUT_THRESHOLD */
 +#define OMAP4460_TSHUT_HOT_SHIFT 16
 +#define OMAP4460_TSHUT_HOT_MASK  (0x3ff  16)
 +#define OMAP4460_TSHUT_COLD_SHIFT0
 +#define OMAP4460_TSHUT_COLD_MASK (0x3ff  0)
 +
 +/* BANDGAP_STATUS */
 +#define OMAP4460_CLEAN_STOP_SHIFT3
 +#define OMAP4460_CLEAN_STOP_MASK (1  3)
 +#define OMAP4460_BGAP_ALERT_SHIFT2
 +#define OMAP4460_BGAP_ALERT_MASK (1  2)
 +#define OMAP4460_HOT_FLAG_SHIFT  1
 +#define OMAP4460_HOT_FLAG_MASK   (1  1)
 +#define OMAP4460_COLD_FLAG_SHIFT 0
 +#define OMAP4460_COLD_FLAG_MASK  (1  0)
  
  /* DPLL_NWELL_TRIM_0 */
  #define OMAP4_DPLL_ABE_NWELL_TRIM_MUX_CTRL_SHIFT 29
 -- 
 1.7.0.4
 
 --
 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  

Re: [PATCH 2/6 V4] OMAP4: Adding the temperature sensor register set bit fields

2011-08-31 Thread J, KEERTHY
On Thu, Sep 1, 2011 at 5:34 AM, Paul Walmsley p...@pwsan.com wrote:
 On Wed, 31 Aug 2011, Keerthy wrote:

 OMAP4460 specific temperature sensor register bit fields are added.
 Existing OMAP4 entries are renamed to OMAP4430.

 Signed-off-by: Keerthy j-keer...@ti.com
 Cc: t...@atomide.com

 At least one of these bitfields are incorrect.  Please double-check them.

 ---
  .../include/mach/ctrl_module_core_44xx.h           |   70 
 
  1 files changed, 57 insertions(+), 13 deletions(-)

 diff --git a/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h 
 b/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h
 index 2f7ac70..725c1e1 100644
 --- a/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h
 +++ b/arch/arm/mach-omap2/include/mach/ctrl_module_core_44xx.h
 @@ -256,19 +256,63 @@
  #define OMAP4_LDOSRAMCORE_ACTMODE_VSET_OUT_SHIFT     0
  #define OMAP4_LDOSRAMCORE_ACTMODE_VSET_OUT_MASK              (0x1f  0)

 -/* TEMP_SENSOR */
 -#define OMAP4_BGAP_TEMPSOFF_SHIFT                    12
 -#define OMAP4_BGAP_TEMPSOFF_MASK                     (1  12)
 -#define OMAP4_BGAP_TSHUT_SHIFT                               11
 -#define OMAP4_BGAP_TSHUT_MASK                                (1  11)
 -#define OMAP4_BGAP_TEMP_SENSOR_CONTCONV_SHIFT                10
 -#define OMAP4_BGAP_TEMP_SENSOR_CONTCONV_MASK         (1  10)
 -#define OMAP4_BGAP_TEMP_SENSOR_SOC_SHIFT             9
 -#define OMAP4_BGAP_TEMP_SENSOR_SOC_MASK                      (1  9)
 -#define OMAP4_BGAP_TEMP_SENSOR_EOCZ_SHIFT            8
 -#define OMAP4_BGAP_TEMP_SENSOR_EOCZ_MASK             (1  8)
 -#define OMAP4_BGAP_TEMP_SENSOR_DTEMP_SHIFT           0
 -#define OMAP4_BGAP_TEMP_SENSOR_DTEMP_MASK            (0xff  0)
 +/* TEMP_SENSOR OMAP4430 */
 +#define OMAP4430_BGAP_TEMPSOFF_SHIFT                 12
 +#define OMAP4430_BGAP_TEMPSOFF_MASK                  (1  12)
 +#define OMAP4430_BGAP_TSHUT_SHIFT                            11
 +#define OMAP4430_BGAP_TSHUT_MASK                             (1  11)
 +#define OMAP4430_BGAP_TEMP_SENSOR_CONTCONV_SHIFT             10
 +#define OMAP4430_BGAP_TEMP_SENSOR_CONTCONV_MASK              (1  10)
 +#define OMAP4430_BGAP_TEMP_SENSOR_SOC_SHIFT          9
 +#define OMAP4430_BGAP_TEMP_SENSOR_SOC_MASK                   (1  9)
 +#define OMAP4430_BGAP_TEMP_SENSOR_EOCZ_SHIFT         8
 +#define OMAP4430_BGAP_TEMP_SENSOR_EOCZ_MASK          (1  8)
 +#define OMAP4430_BGAP_TEMP_SENSOR_DTEMP_SHIFT                0
 +#define OMAP4430_BGAP_TEMP_SENSOR_DTEMP_MASK         (0x3ff  0)

 For example, this bitfield is only eight bits wide on 4430.

I checked the 4430 TRM. It is 8 bits and the others are fine.
I will correct this.


 +
 +/* TEMP_SENSOR OMAP4460 */
 +#define OMAP4460_BGAP_TEMPSOFF_SHIFT                 13
 +#define OMAP4460_BGAP_TEMPSOFF_MASK                  (1  13)
 +#define OMAP4460_BGAP_TEMP_SENSOR_SOC_SHIFT          11
 +#define OMAP4460_BGAP_TEMP_SENSOR_SOC_MASK           (1  11)
 +#define OMAP4460_BGAP_TEMP_SENSOR_EOCZ_SHIFT         10
 +#define OMAP4460_BGAP_TEMP_SENSOR_EOCZ_MASK          (1  10)
 +#define OMAP4460_BGAP_TEMP_SENSOR_DTEMP_SHIFT                0
 +#define OMAP4460_BGAP_TEMP_SENSOR_DTEMP_MASK         (0x3ff  0)
 +
 +/* BANDGAP_CTRL */
 +#define OMAP4460_SINGLE_MODE_SHIFT                   31
 +#define OMAP4460_SINGLE_MODE_MASK                    (1  31)
 +#define OMAP4460_MASK_HOT_SHIFT                              1
 +#define OMAP4460_MASK_HOT_MASK                               (1  1)
 +#define OMAP4460_MASK_COLD_SHIFT                     0
 +#define OMAP4460_MASK_COLD_MASK                              (1  0)
 +
 +/* BANDGAP_COUNTER */
 +#define OMAP4460_COUNTER_SHIFT                               0
 +#define OMAP4460_COUNTER_MASK                                (0xff  0)
 +
 +/* BANDGAP_THRESHOLD */
 +#define OMAP4460_T_HOT_SHIFT                         16
 +#define OMAP4460_T_HOT_MASK                          (0x3ff  16)
 +#define OMAP4460_T_COLD_SHIFT                                0
 +#define OMAP4460_T_COLD_MASK                         (0x3ff  0)
 +
 +/* TSHUT_THRESHOLD */
 +#define OMAP4460_TSHUT_HOT_SHIFT                     16
 +#define OMAP4460_TSHUT_HOT_MASK                              (0x3ff  16)
 +#define OMAP4460_TSHUT_COLD_SHIFT                    0
 +#define OMAP4460_TSHUT_COLD_MASK                     (0x3ff  0)
 +
 +/* BANDGAP_STATUS */
 +#define OMAP4460_CLEAN_STOP_SHIFT                    3
 +#define OMAP4460_CLEAN_STOP_MASK                     (1  3)
 +#define OMAP4460_BGAP_ALERT_SHIFT                    2
 +#define OMAP4460_BGAP_ALERT_MASK                     (1  2)
 +#define OMAP4460_HOT_FLAG_SHIFT                              1
 +#define OMAP4460_HOT_FLAG_MASK                               (1  1)
 +#define OMAP4460_COLD_FLAG_SHIFT                     0
 +#define OMAP4460_COLD_FLAG_MASK                              (1  0)

  /* DPLL_NWELL_TRIM_0 */
  #define OMAP4_DPLL_ABE_NWELL_TRIM_MUX_CTRL_SHIFT     29
 --
 1.7.0.4

 --
 To unsubscribe from