From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Hugues Deschenes Subject: Re: [PATCH 03/13] stmmac: add the new Header file for stmmac platform data Date: Thu, 07 Jan 2010 09:14:35 -0500 Message-ID: <4B45EC4B.10602@octasic.com> References: <1262855243-6762-1-git-send-email-peppe.cavallaro@st.com> <1262855243-6762-2-git-send-email-peppe.cavallaro@st.com> <1262855243-6762-3-git-send-email-peppe.cavallaro@st.com> <4B45DE02.5010403@octasic.com> <4B45E1D7.80407@st.com> <4B45E5C3.6060205@octasic.com> <4B45E941.1070002@st.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org To: Giuseppe CAVALLARO Return-path: Received: from mail.octasic.com ([70.54.254.106]:25918 "EHLO MAILEXCH.octasic.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752022Ab0AGOOg (ORCPT ); Thu, 7 Jan 2010 09:14:36 -0500 In-Reply-To: <4B45E941.1070002@st.com> Sender: netdev-owner@vger.kernel.org List-ID: Hi Giuseppe, Giuseppe CAVALLARO wrote: > Another question. Indeed, for stm architectures, within each setup file, > we don't include the linux/stmmac.h (I was wrong before!) but include > linux/stm/platform.h. This own header includes the "linux/stmmac.h". > So using stmmac_plat.h, I should add: #include > "../../../drivers/net/stmmac/stmmac_plat.h" within the > include/linux/stm/platform.h file. Is it ok? What do you think? > > I believe that would work out well. ... Although I wonder if the whole stm subdirectory should be in include/linux, rather than in arch/sh... But that's another story... It is there because you maintain both sh and arm-base SOCs right? thanks, jh