From: Oleksij Rempel <o.rempel@pengutronix.de>
To: barebox@lists.infradead.org
Subject: [PATCH v1 1/2] ARM: i.MX6 MMDC: Add register offset comments to struct
Date: Thu, 28 Sep 2023 14:11:05 +0200 [thread overview]
Message-ID: <20230928121106.2584743-1-o.rempel@pengutronix.de> (raw)
From: Sascha Hauer <s.hauer@pengutronix.de>
Lying structs over registers is a U-Boot paradigm I'll never understand.
The only thing it does is to successfully prevent a reader from
knowing/verifying the register offset without counting struct members. I
am currently not in the mood of rewriting this, but at least add some
comments with the register offsets as a debugging aid.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
include/mach/imx/imx6-mmdc.h | 100 +++++++++++++++++------------------
1 file changed, 50 insertions(+), 50 deletions(-)
diff --git a/include/mach/imx/imx6-mmdc.h b/include/mach/imx/imx6-mmdc.h
index bf8d41fe58..1df87bf6bd 100644
--- a/include/mach/imx/imx6-mmdc.h
+++ b/include/mach/imx/imx6-mmdc.h
@@ -145,60 +145,60 @@ struct mx6sx_iomux_grp_regs {
*/
#define MX6DQ_IOM_DDR_BASE 0x020e0500
struct mx6dq_iomux_ddr_regs {
- u32 res1[3];
- u32 dram_sdqs5;
- u32 dram_dqm5;
- u32 dram_dqm4;
- u32 dram_sdqs4;
- u32 dram_sdqs3;
- u32 dram_dqm3;
- u32 dram_sdqs2;
- u32 dram_dqm2;
- u32 res2[16];
- u32 dram_cas;
- u32 res3[2];
- u32 dram_ras;
- u32 dram_reset;
- u32 res4[2];
- u32 dram_sdclk_0;
- u32 dram_sdba2;
- u32 dram_sdcke0;
- u32 dram_sdclk_1;
- u32 dram_sdcke1;
- u32 dram_sdodt0;
- u32 dram_sdodt1;
- u32 res5;
- u32 dram_sdqs0;
- u32 dram_dqm0;
- u32 dram_sdqs1;
- u32 dram_dqm1;
- u32 dram_sdqs6;
- u32 dram_dqm6;
- u32 dram_sdqs7;
- u32 dram_dqm7;
+ u32 res1[3]; /* 0x020e0500 */
+ u32 dram_sdqs5; /* 0x020e050c */
+ u32 dram_dqm5; /* 0x020e0510 */
+ u32 dram_dqm4; /* 0x020e0514 */
+ u32 dram_sdqs4; /* 0x020e0518 */
+ u32 dram_sdqs3; /* 0x020e051c */
+ u32 dram_dqm3; /* 0x020e0520 */
+ u32 dram_sdqs2; /* 0x020e0524 */
+ u32 dram_dqm2; /* 0x020e0528 */
+ u32 res2[16]; /* 0x020e052c */
+ u32 dram_cas; /* 0x020e056c */
+ u32 res3[2]; /* 0x020e0570 */
+ u32 dram_ras; /* 0x020e0578 */
+ u32 dram_reset; /* 0x020e057c */
+ u32 res4[2]; /* 0x020e0580 */
+ u32 dram_sdclk_0; /* 0x020e0588 */
+ u32 dram_sdba2; /* 0x020e058c */
+ u32 dram_sdcke0; /* 0x020e0590 */
+ u32 dram_sdclk_1; /* 0x020e0594 */
+ u32 dram_sdcke1; /* 0x020e0598 */
+ u32 dram_sdodt0; /* 0x020e059c */
+ u32 dram_sdodt1; /* 0x020e05a0 */
+ u32 res5; /* 0x020e05a4 */
+ u32 dram_sdqs0; /* 0x020e05a8 */
+ u32 dram_dqm0; /* 0x020e05ac */
+ u32 dram_sdqs1; /* 0x020e05b0 */
+ u32 dram_dqm1; /* 0x020e05b4 */
+ u32 dram_sdqs6; /* 0x020e05b8 */
+ u32 dram_dqm6; /* 0x020e05bc */
+ u32 dram_sdqs7; /* 0x020e05c0 */
+ u32 dram_dqm7; /* 0x020e05c4 */
};
#define MX6DQ_IOM_GRP_BASE 0x020e0700
struct mx6dq_iomux_grp_regs {
- u32 res1[18];
- u32 grp_b7ds;
- u32 grp_addds;
- u32 grp_ddrmode_ctl;
- u32 res2;
- u32 grp_ddrpke;
- u32 res3[6];
- u32 grp_ddrmode;
- u32 res4[3];
- u32 grp_b0ds;
- u32 grp_b1ds;
- u32 grp_ctlds;
- u32 res5;
- u32 grp_b2ds;
- u32 grp_ddr_type;
- u32 grp_b3ds;
- u32 grp_b4ds;
- u32 grp_b5ds;
- u32 grp_b6ds;
+ u32 res1[18]; /* 0x020e0700 */
+ u32 grp_b7ds; /* 0x020e0748 */
+ u32 grp_addds; /* 0x020e074c */
+ u32 grp_ddrmode_ctl; /* 0x020e0750 */
+ u32 res2; /* 0x020e0754 */
+ u32 grp_ddrpke; /* 0x020e0758 */
+ u32 res3[6]; /* 0x020e075c */
+ u32 grp_ddrmode; /* 0x020e0774 */
+ u32 res4[3]; /* 0x020e0778 */
+ u32 grp_b0ds; /* 0x020e0784 */
+ u32 grp_b1ds; /* 0x020e0788 */
+ u32 grp_ctlds; /* 0x020e078c */
+ u32 res5; /* 0x020e0790 */
+ u32 grp_b2ds; /* 0x020e0794 */
+ u32 grp_ddr_type; /* 0x020e0798 */
+ u32 grp_b3ds; /* 0x020e079c */
+ u32 grp_b4ds; /* 0x020e07a0 */
+ u32 grp_b5ds; /* 0x020e07a4 */
+ u32 grp_b6ds; /* 0x020e07a8 */
};
#define MX6SDL_IOM_DDR_BASE 0x020e0400
--
2.39.2
next reply other threads:[~2023-09-28 12:12 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-28 12:11 Oleksij Rempel [this message]
2023-09-28 12:11 ` [PATCH v1 2/2] ARM: skov-imx6: Remove unused board variants Oleksij Rempel
2023-10-04 7:38 ` [PATCH v1 1/2] ARM: i.MX6 MMDC: Add register offset comments to struct Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230928121106.2584743-1-o.rempel@pengutronix.de \
--to=o.rempel@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox