[U-Boot] [PATCH 1/2] imx: mx6: add more entry for mxc_ccm_reg
Eric Nelson
eric at nelint.com
Tue Jan 5 19:21:25 CET 2016
Hi Peng,
On 01/03/2016 11:10 PM, Peng Fan wrote:
> Add more entries for structure mxc_ccm_reg.
>
> Signed-off-by: Peng Fan <peng.fan at nxp.com>
> Cc: Stefano Babic <sbabic at denx.de>
> ---
> arch/arm/include/asm/arch-mx6/crm_regs.h | 87 ++++++++++++++++++++++++++++++++
> 1 file changed, 87 insertions(+)
>
> diff --git a/arch/arm/include/asm/arch-mx6/crm_regs.h b/arch/arm/include/asm/arch-mx6/crm_regs.h
> index 13e0a3d..787eb68 100644
> --- a/arch/arm/include/asm/arch-mx6/crm_regs.h
> +++ b/arch/arm/include/asm/arch-mx6/crm_regs.h
> @@ -103,6 +103,93 @@ struct mxc_ccm_reg {
> u32 analog_pfd_528_set;
> u32 analog_pfd_528_clr;
> u32 analog_pfd_528_tog;
I'd like to see some comments here about where in the reference manuals
these registers are defined, since they're not in the CCM section.
Also, I think these should be named to match the RM
i.e. pmu_reg_1p1
> + u32 reg_1p1;
> + u32 reg_1p1_set;
> + u32 reg_1p1_clr;
> + u32 reg_1p1_tog;
> + u32 reg_3p0;
> + u32 reg_3p0_set;
> + u32 reg_3p0_clr;
> + u32 reg_3p0_tog;
> + u32 reg_2p5;
> + u32 reg_2p5_set;
> + u32 reg_2p5_clr;
> + u32 reg_2p5_tog;
> + u32 reg_core;
> + u32 reg_core_set;
> + u32 reg_core_clr;
> + u32 reg_core_tog;
> + u32 ana_misc0;
> + u32 ana_misc0_set;
> + u32 ana_misc0_clr;
> + u32 ana_misc0_tog;
> + u32 ana_misc1;
> + u32 ana_misc1_set;
> + u32 ana_misc1_clr;
> + u32 ana_misc1_tog;
> + u32 ana_misc2;
> + u32 ana_misc2_set;
> + u32 ana_misc2_clr;
... pmu_misc2_tog
> + u32 ana_misc2_tog;
TEMPMON registers
> + u32 tempsense0;
> + u32 tempsense0_set;
> + u32 tempsense0_clr;
> + u32 tempsense0_tog;
> + u32 tempsense1;
> + u32 tempsense1_set;
> + u32 tempsense1_clr;
> + u32 tempsense1_tog;
... and USB_ANALOG here
> + u32 usb1_vbus_detect;
> + u32 usb1_vbus_detect_set;
> + u32 usb1_vbus_detect_clr;
> + u32 usb1_vbus_detect_tog;
> + u32 usb1_chrg_detect;
> + u32 usb1_chrg_detect_set;
> + u32 usb1_chrg_detect_clr;
> + u32 usb1_chrg_detect_tog;
> + u32 usb1_vbus_det_stat;
> + u32 usb1_vbus_det_stat_set;
> + u32 usb1_vbus_det_stat_clr;
> + u32 usb1_vbus_det_stat_tog;
> + u32 usb1_chrg_det_stat;
> + u32 usb1_chrg_det_stat_set;
> + u32 usb1_chrg_det_stat_clr;
> + u32 usb1_chrg_det_stat_tog;
> + u32 usb1_loopback;
> + u32 usb1_loopback_set;
> + u32 usb1_loopback_clr;
> + u32 usb1_loopback_tog;
> + u32 usb1_misc;
> + u32 usb1_misc_set;
> + u32 usb1_misc_clr;
> + u32 usb1_misc_tog;
> + u32 usb2_vbus_detect;
> + u32 usb2_vbus_detect_set;
> + u32 usb2_vbus_detect_clr;
> + u32 usb2_vbus_detect_tog;
> + u32 usb2_chrg_detect;
> + u32 usb2_chrg_detect_set;
> + u32 usb2_chrg_detect_clr;
> + u32 usb2_chrg_detect_tog;
> + u32 usb2_vbus_det_stat;
> + u32 usb2_vbus_det_stat_set;
> + u32 usb2_vbus_det_stat_clr;
> + u32 usb2_vbus_det_stat_tog;
> + u32 usb2_chrg_det_stat;
> + u32 usb2_chrg_det_stat_set;
> + u32 usb2_chrg_det_stat_clr;
> + u32 usb2_chrg_det_stat_tog;
> + u32 usb2_loopback;
> + u32 usb2_loopback_set;
> + u32 usb2_loopback_clr;
> + u32 usb2_loopback_tog;
> + u32 usb2_misc;
> + u32 usb2_misc_set;
> + u32 usb2_misc_clr;
> + u32 usb2_misc_tog;
> + u32 digprog;
> + u32 reserved1[7];
> + u32 digprog_sololite;
> };
> #endif
>
>
Otherwise, I checked these against the i.MX6DQ/SDL/SL and UL manuals
(not i.MX6SX), and
Reviewed-By: Eric Nelson <eric at nelint.com>
More information about the U-Boot
mailing list