[Bug 1001606] Enable the cache on Kinetis in RAM startup mode

[bugzilla-daemon@bugs.ecos.sourceware.org]

Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=1001606

--- Comment #13 from Jonathan Larmour <jifl@ecoscentric.com> 2012-06-23 06:58:21 BST ---
(In reply to comment #12)
> I want to preserve also integral functionality of the part builder
> CYGHWR_HAL_CORTEXM_KINETIS, including FPU option.
> CYGHWR_HAL_CORTEXM_KINETIS_FPU selects FPU as Kinetis option and
> CYGHWR_HAL_CORTEXM_FPU enables/disables FPU function so they aren't identical.

Okay, I see what you're getting at - so the part can match what the user has,
even if they're not actually using the FPU. That's fine.

> But it's true that they look alike so below is CYGHWR_HAL_CORTEXM_KINETIS_FPU
> reshaped to resemble the other entries of the part builder
> (CYGHWR_HAL_CORTEXM_KINETIS_FLEXNVM can be reshaped likewise).
> 
>         cdl_option CYGHWR_HAL_CORTEXM_KINETIS_FPU {
>             display          "Floating Point Unit part name option"
>             flavor           data
>             no_define
>             legal_values  { "D" "F" }
>             default_value { "D" }
>             requires      { CYGHWR_HAL_CORTEXM_FPU implies
>                             CYGHWR_HAL_CORTEXM_KINETIS_FPU == "F" }

You also need to have the converse:
             requires      { !CYGHWR_HAL_CORTEXM_FPU implies
                             CYGHWR_HAL_CORTEXM_KINETIS_FPU == "D" }

But other than that, that's fine by me.

>             description "Select/indicate whether the part has Floating Point
> Unit. \"F\" - stands for 
>             parts with FPU, while \"D\" for ones without."

Just say "Select".

Thanks!

Jifl

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.