On Tue, 25 May 2010, Christophe Coutand wrote:
On Tue, 25 May 2010, Sergei Gavrikov wrote:
Hi Sergei,
Thanks for your inputs. Attached is the updated patch. Let me know if
more changes are required.
Christophe
Hi Christophe,
I retested this one. The compiler's error went away, thank you. And now
your patch looks great for me.
[snip]
And I had a thought about some CDL wrapping the limit, e.g.
cdl_component CYGSEM_DEVS_ADC_ARM_AT91_SAMPLE_SIZE_LIMIT {
display "Sample size limit"
flavor bool
calculated 1
requires {CYGNUM_IO_ADC_SAMPLE_SIZE >= 10}
requires {CYGNUM_IO_ADC_SAMPLE_SIZE <= 12}
description "
Tell user about the limitation."
}
[snip]
The last thing. When I typed in an example above "Tell user about..." I
thought that you put there (in description) some paraphrase from AT91
datasheet and there is almost mine in CDL :-) Well, you can sent me just
a few descriptive sentences. As you understand it will be then displayed
in `configtool` (the component's description). I will fix it.
Thanks,
Sergei
