From mboxrd@z Thu Jan 1 00:00:00 1970 From: Frank Dols Date: Fri, 1 Mar 2013 10:05:41 +0000 Subject: [U-Boot] FW: [PATCH 1/2] drivers/net/designware, do an explicit memory access instead of implicit, re-written assignments to use readl() and writel(), all of this as preperation for making the driver able to work in a cached environment (I$D$ support). In-Reply-To: <511879F4.206@st.com> References: <1358863821-31956-1-git-send-email-frank.dols@synopsys.com> <1358863821-31956-2-git-send-email-frank.dols@synopsys.com> <50FF7531.7030609@st.com> <20130123075559.49f94e5f@lilith> <50FFB87F.3060609@st.com> <20130123112107.32bcb731@lilith> <1869199372336F41A75F0B381AC8B48A028B6B@DE02WEMBX1.internal.synopsys.com> <511879F4.206@st.com> Message-ID: <1869199372336F41A75F0B381AC8B48A03C0F3@de02wembxa.internal.synopsys.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de [[ ... to get attention again ... see below ...]] On 2/8/2013 6:22 PM, Frank Dols wrote: > Good afternoon Vipin and Albert, > I where wondering, is there any review/update news on the patches I submitted a short while ago? > Regards, Frank. > Sorry, first excluded " u-boot at lists.denx.de" from this email to prevent from noice on mailing list. > There is no reason to remove the list. This is not noise I believe. Infact by including the list, you are intimating other developers that such work would be available soon Albert, do you think any different. > Sorry, I have to clarify here a bit more. > The descriptors are 16 bytes in length and a cache line is in most architectures more than 16 bytes in length (in our case either 32 or 64). > This means that cached accesses is not an option for these descriptors. Background, two adjacent descriptors as be on one cache line may be owned by different entities (host cpu / network ip). Yes, this is a problem and I can't think of a clean solution. Specially because u-boot (as of today) does not support non-cached memory Albert? > Explicit cache calls that we are added in patch 2/2 are meant for payload of the package. And these are made cache line aligned with patch 1/2. Yes, that is what I thought > Unfortunately we can't align the descriptors on cache line boundaries due to hardware limitations (for architectures with cache line longer than 16 bytes) ! Yes, I know that. The descriptors are 16 bytes and they need to be contiguous in memory PS: I have not added the list but I strongly feel that this mail should also go to the uboot list -Vipin > With kind regards, greetings, Frank. > >