This is the mail archive of the ecos-bugs@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]

[Bug 1001397] I2C driver for Kinetic microcontrollers


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

--- Comment #5 from Ilija Kocho <ilijak@siva.com.mk> 2011-12-23 14:12:07 GMT ---
(In reply to comment #3)
> In the light of the discussion in bug 1001428, I think the hal bits in this
> patch should be moved under hal/misc/freescale/i2c?

(In reply to comment #3)
> In the light of the discussion in bug 1001428, I think the hal bits in this
> patch should be moved under hal/misc/freescale/i2c?


I assume you are referring to i2c header. They should stay with driver source,
either in driver .c file or (preferably) in a header.
As there is existing devs/i2c directory, I propose following path:
    - devs/i2c/freescale/<freescale_i2c>

where <freescale_i2c> is the name of this particular freescale spi instance
(typically as referred to in ref. man.).

Besides the device structure itself, there is some information provided by HAL
(base address(es), pin configuration or so. They can be be considered variant
or platform dependent and so defined in var_io_devs.h and/or plf_io.h
respectively. If defined in both then plf shall override var.

I hope in future to produce some docs but in meantime pse refer to ENET driver
for example.

-- 
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.


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