From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [PATCH 1/2] clps711x/ceiva: port to use physmap flash From: Artem Bityutskiy To: Dmitry Eremin-Solenikov In-Reply-To: References: <1307015648-17505-1-git-send-email-dbaryshkov@gmail.com> <1307075905.4405.130.camel@localhost> <1307077842.4405.133.camel@localhost> Content-Type: text/plain; charset="UTF-8" Date: Fri, 03 Jun 2011 08:31:09 +0300 Message-ID: <1307079069.4405.142.camel@localhost> Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Cc: Russell King , linux-mtd@lists.infradead.org, David Woodhouse , linux-arm-kernel@lists.infradead.org Reply-To: Artem.Bityutskiy@nokia.com List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, 2011-06-03 at 09:17 +0400, Dmitry Eremin-Solenikov wrote: > On 6/3/11, Artem Bityutskiy wrote: > > On Fri, 2011-06-03 at 09:02 +0400, Dmitry Eremin-Solenikov wrote: > >> On 6/3/11, Artem Bityutskiy wrote: > >> > On Thu, 2011-06-02 at 15:54 +0400, Dmitry Eremin-Solenikov wrote: > >> >> Port ceiva machine to use physmap flash instead of custom mapping > >> >> driver > >> >> (which even can't compile cleanly with current kernels). All info > >> >> is gathered from drivers/mtd/maps/ceiva.c. Compile tested only. > >> >> > >> >> Signed-off-by: Dmitry Eremin-Solenikov > >> >> --- > >> >> arch/arm/mach-clps711x/ceiva.c | 60 > >> >> ++++++++++++++++++++++++++++++++++++++++ > >> >> 1 files changed, 60 insertions(+), 0 deletions(-) > >> > > >> > Is ceiva used and not dead? May be we could kill it completely? > >> > >> No functional changes to board file since start of git history. Also > >> I don't know how useful can be contemporary Linux on 4 MiB of RAM > > > > I also noticed that no one is interested in this board for long time. So > > I suggest to remove ceiva support altogether via some arm tree. > > Anyway the patch removing mtd support can be pushed via mtd tree > (there is really no point on mtd/maps/ceiva.c), and removing ceiva > depends on Russell's decision. If you care about conflicts and how to merge all that - you can remove ceiva via the arm tree, and at the same time modify the driver in the mtd tree, and we'll just put a not to the commit message that during the merge this change has to be discarded. Linus did a similar thing this merge window for one board which mtd modified and arm tree removed. -- Best Regards, Artem Bityutskiy (Артём Битюцкий)