Message ID | 1461894969-3856-1-git-send-email-van.freenix@gmail.com |
---|---|
State | Changes Requested |
Delegated to: | Stefano Babic |
Headers | show |
On Fri, Apr 29, 2016 at 09:56:09AM +0800, Peng Fan wrote: > This commit "2bb014820c49a63902103bac710bc86b5772e843" > do some clean up to use the code in lib/time.c. > But usec2ticks is still being used by security related job ring code. > Bring back the function to avoid build break. > > Signed-off-by: Peng Fan <van.freenix@gmail.com> > Cc: Stefano Babic <sbabic@denx.de> .... some option not enabled by default somewhere? I build test the world frequently..
Hi Tom, On Mon, May 02, 2016 at 07:41:35PM -0400, Tom Rini wrote: >On Fri, Apr 29, 2016 at 09:56:09AM +0800, Peng Fan wrote: > >> This commit "2bb014820c49a63902103bac710bc86b5772e843" >> do some clean up to use the code in lib/time.c. >> But usec2ticks is still being used by security related job ring code. >> Bring back the function to avoid build break. >> >> Signed-off-by: Peng Fan <van.freenix@gmail.com> >> Cc: Stefano Babic <sbabic@denx.de> > >.... some option not enabled by default somewhere? I build test the >world frequently.. Yeah. Security related options not enabled by default. If enabled these for i.MX6, #define CONFIG_SECURE_BOOT #define CONFIG_SYS_FSL_SEC_COMPAT 4 /* HAB version */ #define CONFIG_FSL_CAAM #define CONFIG_CMD_DEKBLOB #define CONFIG_SYS_FSL_SEC_LE Met the errors: drivers/crypto/fsl/jr.c:333: undefined reference to `usec2ticks' Regards, Peng. > >-- >Tom
On Tue, May 03, 2016 at 09:17:51AM +0800, Peng Fan wrote: > Hi Tom, > > On Mon, May 02, 2016 at 07:41:35PM -0400, Tom Rini wrote: > >On Fri, Apr 29, 2016 at 09:56:09AM +0800, Peng Fan wrote: > > > >> This commit "2bb014820c49a63902103bac710bc86b5772e843" > >> do some clean up to use the code in lib/time.c. > >> But usec2ticks is still being used by security related job ring code. > >> Bring back the function to avoid build break. > >> > >> Signed-off-by: Peng Fan <van.freenix@gmail.com> > >> Cc: Stefano Babic <sbabic@denx.de> > > > >.... some option not enabled by default somewhere? I build test the > >world frequently.. > > > Yeah. > > Security related options not enabled by default. > > If enabled these for i.MX6, > #define CONFIG_SECURE_BOOT > #define CONFIG_SYS_FSL_SEC_COMPAT 4 /* HAB version */ > #define CONFIG_FSL_CAAM > #define CONFIG_CMD_DEKBLOB > #define CONFIG_SYS_FSL_SEC_LE > > Met the errors: > > drivers/crypto/fsl/jr.c:333: undefined reference to `usec2ticks' I'd like to see this stuff moved over to Kconfig and a sample config enabled so things don't break in the future, thanks!
Hi Tom, On Mon, May 02, 2016 at 10:04:47PM -0400, Tom Rini wrote: >On Tue, May 03, 2016 at 09:17:51AM +0800, Peng Fan wrote: >> Hi Tom, >> >> On Mon, May 02, 2016 at 07:41:35PM -0400, Tom Rini wrote: >> >On Fri, Apr 29, 2016 at 09:56:09AM +0800, Peng Fan wrote: >> > >> >> This commit "2bb014820c49a63902103bac710bc86b5772e843" >> >> do some clean up to use the code in lib/time.c. >> >> But usec2ticks is still being used by security related job ring code. >> >> Bring back the function to avoid build break. >> >> >> >> Signed-off-by: Peng Fan <van.freenix@gmail.com> >> >> Cc: Stefano Babic <sbabic@denx.de> >> > >> >.... some option not enabled by default somewhere? I build test the >> >world frequently.. >> >> >> Yeah. >> >> Security related options not enabled by default. >> >> If enabled these for i.MX6, >> #define CONFIG_SECURE_BOOT >> #define CONFIG_SYS_FSL_SEC_COMPAT 4 /* HAB version */ >> #define CONFIG_FSL_CAAM >> #define CONFIG_CMD_DEKBLOB >> #define CONFIG_SYS_FSL_SEC_LE >> >> Met the errors: >> >> drivers/crypto/fsl/jr.c:333: undefined reference to `usec2ticks' > >I'd like to see this stuff moved over to Kconfig and a sample config >enabled so things don't break in the future, thanks! Ok. Will add this support. Will you directly pick up this patch or go through i.MX tree? Thanks, Peng. > >-- >Tom
Hi Stefano, Kindly ping.. Thanks, Peng. On Fri, Apr 29, 2016 at 09:56:09AM +0800, Peng Fan wrote: >This commit "2bb014820c49a63902103bac710bc86b5772e843" >do some clean up to use the code in lib/time.c. >But usec2ticks is still being used by security related job ring code. >Bring back the function to avoid build break. > >Signed-off-by: Peng Fan <van.freenix@gmail.com> >Cc: Stefano Babic <sbabic@denx.de> >--- > arch/arm/imx-common/timer.c | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > >diff --git a/arch/arm/imx-common/timer.c b/arch/arm/imx-common/timer.c >index 92c7218..bde24af 100644 >--- a/arch/arm/imx-common/timer.c >+++ b/arch/arm/imx-common/timer.c >@@ -124,3 +124,20 @@ ulong get_tbclk(void) > { > return gpt_get_clk(); > } >+ >+/* >+ * This function is intended for SHORT delays only. >+ * It will overflow at around 10 seconds @ 400MHz, >+ * or 20 seconds @ 200MHz. >+ */ >+unsigned long usec2ticks(unsigned long usec) >+{ >+ ulong ticks; >+ >+ if (usec < 1000) >+ ticks = ((usec * (get_tbclk()/1000)) + 500) / 1000; >+ else >+ ticks = ((usec / 10) * (get_tbclk() / 100000)); >+ >+ return ticks; >+} >-- >2.6.2 >
Hi Peng, On 10/05/2016 11:30, Peng Fan wrote: > Hi Stefano, > > Kindly ping.. > I have maybe misunderstood the thread and I have supposed you want to send a V2 with the stuff related to secure moved to Kconfig, as requested by Tom. For that reason, you find your patch tagged with "Changes requested" in patchwork. Anyway, this can be done in a follow up patch. I will completely agree if this is done separately. Best regards, Stefano > Thanks, > Peng. > On Fri, Apr 29, 2016 at 09:56:09AM +0800, Peng Fan wrote: >> This commit "2bb014820c49a63902103bac710bc86b5772e843" >> do some clean up to use the code in lib/time.c. >> But usec2ticks is still being used by security related job ring code. >> Bring back the function to avoid build break. >> >> Signed-off-by: Peng Fan <van.freenix@gmail.com> >> Cc: Stefano Babic <sbabic@denx.de> >> --- >> arch/arm/imx-common/timer.c | 17 +++++++++++++++++ >> 1 file changed, 17 insertions(+) >> >> diff --git a/arch/arm/imx-common/timer.c b/arch/arm/imx-common/timer.c >> index 92c7218..bde24af 100644 >> --- a/arch/arm/imx-common/timer.c >> +++ b/arch/arm/imx-common/timer.c >> @@ -124,3 +124,20 @@ ulong get_tbclk(void) >> { >> return gpt_get_clk(); >> } >> + >> +/* >> + * This function is intended for SHORT delays only. >> + * It will overflow at around 10 seconds @ 400MHz, >> + * or 20 seconds @ 200MHz. >> + */ >> +unsigned long usec2ticks(unsigned long usec) >> +{ >> + ulong ticks; >> + >> + if (usec < 1000) >> + ticks = ((usec * (get_tbclk()/1000)) + 500) / 1000; >> + else >> + ticks = ((usec / 10) * (get_tbclk() / 100000)); >> + >> + return ticks; >> +} >> -- >> 2.6.2 >>
diff --git a/arch/arm/imx-common/timer.c b/arch/arm/imx-common/timer.c index 92c7218..bde24af 100644 --- a/arch/arm/imx-common/timer.c +++ b/arch/arm/imx-common/timer.c @@ -124,3 +124,20 @@ ulong get_tbclk(void) { return gpt_get_clk(); } + +/* + * This function is intended for SHORT delays only. + * It will overflow at around 10 seconds @ 400MHz, + * or 20 seconds @ 200MHz. + */ +unsigned long usec2ticks(unsigned long usec) +{ + ulong ticks; + + if (usec < 1000) + ticks = ((usec * (get_tbclk()/1000)) + 500) / 1000; + else + ticks = ((usec / 10) * (get_tbclk() / 100000)); + + return ticks; +}
This commit "2bb014820c49a63902103bac710bc86b5772e843" do some clean up to use the code in lib/time.c. But usec2ticks is still being used by security related job ring code. Bring back the function to avoid build break. Signed-off-by: Peng Fan <van.freenix@gmail.com> Cc: Stefano Babic <sbabic@denx.de> --- arch/arm/imx-common/timer.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)