public inbox for ecos-patches@sourceware.org
 help / color / mirror / Atom feed
From: bugzilla-daemon@bugs.ecos.sourceware.org
To: ecos-patches@ecos.sourceware.org
Subject: [Bug 1001219] Ethernet driver for STM32 connectivity line with port on MMstm32f107 board.
Date: Wed, 12 Oct 2011 21:15:00 -0000	[thread overview]
Message-ID: <20111012211512.5444C2F78010@mail.ecoscentric.com> (raw)
In-Reply-To: <bug-1001219-104@http.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=1001219

--- Comment #16 from Ilija Kocho <ilijak@siva.com.mk> 2011-10-12 22:15:09 BST ---
(In reply to comment #14)
> Hello Ilija,
> 
> (In reply to comment #12)
> > (In reply to comment #6)
> > > Created an attachment (id=1395)
 --> (http://bugs.ecos.sourceware.org/attachment.cgi?id=1395) [details]
[details] [details]
> > > Platform for Propox MMstm32f107 module
> > 
> > First to remind you to my remark in comment #4
> > 
> > At present the user will see F103ZE which is unlikely to be the target device.
> Please consider that such situation will always happen if further target will
> be introduced.
> 
> > Platform may require the "STM variany in use" (CYGHWR_HAL_CORTEXM_STM32) to be
> > a specific device, or a set of eligible devices. If you opt for second case
> > just make sure that configtool resolves with the one found on Propox board.
> IMHO Either platform should provide proper STM32 variant or this option should
> be remove - nobody is interested in this option.
> 
> Please review this option considering targets existing STM32F100/1/2/3 further
> like my STM32F105/7 and possible STM32F2xx, STM32F4xx and give me "order" how I
> should introduce STM32F105/7 family.
> 

I am not in a position to "order" you but I can give you some hints. Here is a
hal_cortexm_stm32_mmstm32f107.cdl diff snippet.

     implements  CYGINT_DEVS_ETH_CORTEXM_STM32_REMAP_PINS
     implements  CYGINT_DEVS_SPI_CORTEXM_STM32_BUS3_REMAP_PINS
+
+    requires {
+        (CYGHWR_HAL_CORTEXM_STM32 == "F107VB") ||
+        (CYGHWR_HAL_CORTEXM_STM32 == "F107VC") ||
+        (CYGHWR_HAL_CORTEXM_STM32 == "F105VB") ||
+        (CYGHWR_HAL_CORTEXM_STM32 == "F105VC")
+    }
+
     requires      { is_active(CYGPKG_DEVS_ETH_PHY) implies
                     (1 == CYGHWR_DEVS_ETH_PHY_DP8384X) }

You can at your option put a single or multiple devices in "requires".
Configtool will resolve (conflict) with first one. Other listed devices shall
be "allowed" and rest of devices (off the list) shall raise conflict.

Note: If you opt for multiple devices they should be ones that the controller
on Propox board can emulate.

Regarding F2 devices let's leave them for the time being. It will be another
platform anyway.

> 
> > Why JTAG startup? AFAIK Connectivity line desn't feature FSMC (or am I out of
> > date?). I guess SRAM startup should work (but test it first).
> You are absolute right. JTAG startup doesn't have sens and I remove it.
> 
> > Regarding ROM startup there's one issue: RAM region is required in order to
> > build RedBoot. True, there's not much use of RedBoot on device with 64KiB RAM
> > but somebody would like to try. Solution is simple rename SRAM region (nor
> > section) into RAM.
> I follow other cortex-m target like lm3s where is:
> - only one startup ROM
> - only SRAM section

I am refering to SRAM memory region, not .sram section.

> 
> BTW. I've built RedBoot successful with only SRAM section. 

You probably do it without GDB support. Then I was unclear, sorry, I should
have said "RAM is required by GDB support".

When I reconsider, this is a small memory device, perhaps we could live without
it.

> 
> If it's possible I would like to shrink amount of startups only to ROM.

ROM startup is enough.


P.S.
As a general advice, please take care about backward compatibility - it is very
important. If able, please do regressions on "STM3210E EVAL board" (or
equivalent). If you don't have hardware at least check the building process.

Ilija

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

  parent reply	other threads:[~2011-10-12 21:15 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-03 22:11 [Bug 1001219] New: " bugzilla-daemon
2011-05-04  7:41 ` [Bug 1001219] " bugzilla-daemon
2011-05-12  9:18 ` bugzilla-daemon
2011-10-08 14:02 ` bugzilla-daemon
2011-10-08 14:27 ` bugzilla-daemon
2011-10-10 20:51 ` bugzilla-daemon
2011-10-10 20:54 ` bugzilla-daemon
2011-10-10 20:56 ` bugzilla-daemon
2011-10-10 21:13 ` bugzilla-daemon
2011-10-10 21:50 ` bugzilla-daemon
2011-10-11 19:53 ` bugzilla-daemon
2011-10-11 20:39 ` bugzilla-daemon
2011-10-11 21:08 ` bugzilla-daemon
2011-10-11 21:45 ` bugzilla-daemon
2011-10-12 11:25 ` bugzilla-daemon
2011-10-12 20:34 ` bugzilla-daemon
2011-10-12 21:15 ` bugzilla-daemon [this message]
2011-10-16 14:12 ` bugzilla-daemon
2011-10-16 14:12 ` bugzilla-daemon
2011-10-22 12:22 ` bugzilla-daemon
2011-10-23 21:27 ` bugzilla-daemon
2011-10-24 15:55 ` bugzilla-daemon
2011-12-04 21:22 ` bugzilla-daemon
2011-12-04 21:23 ` bugzilla-daemon
2011-12-04 21:24 ` bugzilla-daemon
2011-12-04 21:25 ` bugzilla-daemon
2011-12-04 21:35 ` bugzilla-daemon
2011-12-21  9:33 ` bugzilla-daemon
2011-12-26 21:47 ` bugzilla-daemon
2012-01-02  0:35 ` bugzilla-daemon
2012-01-06 23:53 ` bugzilla-daemon
2012-01-06 23:56 ` bugzilla-daemon
2012-01-06 23:57 ` bugzilla-daemon
2012-01-06 23:57 ` bugzilla-daemon
2012-01-07  0:02 ` bugzilla-daemon
2012-01-07 22:31 ` bugzilla-daemon
2012-01-09 14:19 ` bugzilla-daemon
2012-02-02 21:44 ` bugzilla-daemon
2012-02-03 13:20 ` bugzilla-daemon
2012-12-10 22:33 ` bugzilla-daemon
2012-12-21 20:02 ` bugzilla-daemon
2012-12-24 17:31 ` bugzilla-daemon
2012-12-24 21:46 ` bugzilla-daemon
2012-12-25 11:34 ` bugzilla-daemon
2012-12-27 17:45 ` bugzilla-daemon
2012-12-28 14:15 ` bugzilla-daemon
2013-01-01 21:09 ` bugzilla-daemon
2013-01-01 21:10 ` bugzilla-daemon
2013-01-01 21:11 ` bugzilla-daemon
2013-01-01 21:17 ` bugzilla-daemon
2013-01-01 21:22 ` bugzilla-daemon
2013-01-15 21:26 ` bugzilla-daemon
2013-01-15 21:27 ` bugzilla-daemon
2013-01-15 21:29 ` bugzilla-daemon
2013-01-20 21:47 ` bugzilla-daemon
2013-02-07 15:06 ` bugzilla-daemon
2013-02-07 18:05 ` bugzilla-daemon
2013-02-16 21:55 ` bugzilla-daemon
2013-04-01 17:28 ` bugzilla-daemon
2013-04-01 18:25 ` bugzilla-daemon
2013-04-06 22:43 ` bugzilla-daemon
2013-04-08 18:48 ` bugzilla-daemon
2013-04-09 11:03 ` bugzilla-daemon
2013-04-09 12:14 ` bugzilla-daemon
2013-04-12 11:12 ` bugzilla-daemon
2013-04-14 23:40 ` bugzilla-daemon
2013-04-17 21:34 ` bugzilla-daemon
2013-04-17 21:48 ` bugzilla-daemon
2017-02-15  7:40 ` bugzilla-daemon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20111012211512.5444C2F78010@mail.ecoscentric.com \
    --to=bugzilla-daemon@bugs.ecos.sourceware.org \
    --cc=ecos-patches@ecos.sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).