| Message ID | 1591708550-14058-1-git-send-email-Anson.Huang@nxp.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Commit | c95c9693b112f312b59c5d100fd09a1349970fab |
| Headers | show |
| Series | soc: imx8m: Correct i.MX8MP UID fuse offset | expand |
On 6/9/2020 4:15 PM, Anson Huang wrote: > Correct i.MX8MP UID fuse offset according to fuse map: > > UID_LOW: 0x420 > UID_HIGH: 0x430 > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> If this patch corrects the imx8mp UID shouldn't have a Fixes tag? > --- > drivers/soc/imx/soc-imx8m.c | 8 ++++++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/soc/imx/soc-imx8m.c b/drivers/soc/imx/soc-imx8m.c > index 7b0759a..0bc8314 100644 > --- a/drivers/soc/imx/soc-imx8m.c > +++ b/drivers/soc/imx/soc-imx8m.c > @@ -22,6 +22,8 @@ > #define OCOTP_UID_LOW 0x410 > #define OCOTP_UID_HIGH 0x420 > > +#define IMX8MP_OCOTP_UID_OFFSET 0x10 > + > /* Same as ANADIG_DIGPROG_IMX7D */ > #define ANADIG_DIGPROG_IMX8MM 0x800 > > @@ -87,6 +89,8 @@ static void __init imx8mm_soc_uid(void) > { > void __iomem *ocotp_base; > struct device_node *np; > + u32 offset = of_machine_is_compatible("fsl,imx8mp") ? > + IMX8MP_OCOTP_UID_OFFSET : 0; > > np = of_find_compatible_node(NULL, NULL, "fsl,imx8mm-ocotp"); > if (!np) > @@ -95,9 +99,9 @@ static void __init imx8mm_soc_uid(void) > ocotp_base = of_iomap(np, 0); > WARN_ON(!ocotp_base); > > - soc_uid = readl_relaxed(ocotp_base + OCOTP_UID_HIGH); > + soc_uid = readl_relaxed(ocotp_base + OCOTP_UID_HIGH + offset); > soc_uid <<= 32; > - soc_uid |= readl_relaxed(ocotp_base + OCOTP_UID_LOW); > + soc_uid |= readl_relaxed(ocotp_base + OCOTP_UID_LOW + offset); > > iounmap(ocotp_base); > of_node_put(np); >
Hi, Luliana > Subject: Re: [PATCH] soc: imx8m: Correct i.MX8MP UID fuse offset > > > > On 6/9/2020 4:15 PM, Anson Huang wrote: > > Correct i.MX8MP UID fuse offset according to fuse map: > > > > UID_LOW: 0x420 > > UID_HIGH: 0x430 > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > If this patch corrects the imx8mp UID shouldn't have a Fixes tag? I thought about this, but I was confused that this file name is changed by commit fc40200ebf82 ("soc: imx: increase build coverage for imx8m soc driver"), so if to add fix tag, should I add the tag to point to first patch of drivers/soc/imx/soc-imx8m.c, or the original commit of supporting 8MP UID in drivers/soc/imx/soc-imx8.c which is no long there. Thanks Anson
Hi, Luliana > Subject: RE: [PATCH] soc: imx8m: Correct i.MX8MP UID fuse offset > > Hi, Luliana > > > > Subject: Re: [PATCH] soc: imx8m: Correct i.MX8MP UID fuse offset > > > > > > > > On 6/9/2020 4:15 PM, Anson Huang wrote: > > > Correct i.MX8MP UID fuse offset according to fuse map: > > > > > > UID_LOW: 0x420 > > > UID_HIGH: 0x430 > > > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > > > If this patch corrects the imx8mp UID shouldn't have a Fixes tag? > > I thought about this, but I was confused that this file name is changed by > commit fc40200ebf82 ("soc: imx: increase build coverage for imx8m soc > driver"), so if to add fix tag, should I add the tag to point to first patch of > drivers/soc/imx/soc-imx8m.c, or the original commit of supporting 8MP UID in > drivers/soc/imx/soc-imx8.c which is no long there. I think it should be commit fc40200ebf82 ("soc: imx: increase build coverage for imx8m soc driver"), will send a V2 patch. Thanks, Anson
diff --git a/drivers/soc/imx/soc-imx8m.c b/drivers/soc/imx/soc-imx8m.c index 7b0759a..0bc8314 100644 --- a/drivers/soc/imx/soc-imx8m.c +++ b/drivers/soc/imx/soc-imx8m.c @@ -22,6 +22,8 @@ #define OCOTP_UID_LOW 0x410 #define OCOTP_UID_HIGH 0x420 +#define IMX8MP_OCOTP_UID_OFFSET 0x10 + /* Same as ANADIG_DIGPROG_IMX7D */ #define ANADIG_DIGPROG_IMX8MM 0x800 @@ -87,6 +89,8 @@ static void __init imx8mm_soc_uid(void) { void __iomem *ocotp_base; struct device_node *np; + u32 offset = of_machine_is_compatible("fsl,imx8mp") ? + IMX8MP_OCOTP_UID_OFFSET : 0; np = of_find_compatible_node(NULL, NULL, "fsl,imx8mm-ocotp"); if (!np) @@ -95,9 +99,9 @@ static void __init imx8mm_soc_uid(void) ocotp_base = of_iomap(np, 0); WARN_ON(!ocotp_base); - soc_uid = readl_relaxed(ocotp_base + OCOTP_UID_HIGH); + soc_uid = readl_relaxed(ocotp_base + OCOTP_UID_HIGH + offset); soc_uid <<= 32; - soc_uid |= readl_relaxed(ocotp_base + OCOTP_UID_LOW); + soc_uid |= readl_relaxed(ocotp_base + OCOTP_UID_LOW + offset); iounmap(ocotp_base); of_node_put(np);
Correct i.MX8MP UID fuse offset according to fuse map: UID_LOW: 0x420 UID_HIGH: 0x430 Signed-off-by: Anson Huang <Anson.Huang@nxp.com> --- drivers/soc/imx/soc-imx8m.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)