This is the mail archive of the ecos-patches@sourceware.org mailing list for the eCos project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: Update to CYGPKG_IO_ADC

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

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]