From mboxrd@z Thu Jan  1 00:00:00 1970
From: Prafulla Wadaskar <prafulla@marvell.com>
Date: Tue, 10 Jul 2012 03:18:47 -0700
Subject: [U-Boot] [PATCH 12/20] arm/km: add support for external
 switchconfiguration
In-Reply-To: <m2pq85ngyq.fsf@lamuella.denx.de>
References: <1339063620-8890-1-git-send-email-valentin.longchamp@keymile.com>
	<1339063620-8890-13-git-send-email-valentin.longchamp@keymile.com>
	<F766E4F80769BD478052FB6533FA745D1A2FCAF0AB@SC-VEXCH4.marvell.com>
	<4FE31D14.6080005@keymile.com>
	<F766E4F80769BD478052FB6533FA745D1A2FE301BF@SC-VEXCH4.marvell.com>
	<4FF2C089.4000302@keymile.com> <m2hatoq78u.fsf@lamuella.denx.de>
	<F766E4F80769BD478052FB6533FA745D1A2FE3057B@SC-VEXCH4.marvell.com>
	<m2pq85ngyq.fsf@lamuella.denx.de>
Message-ID: <F766E4F80769BD478052FB6533FA745D1A2FEA71AB@SC-VEXCH4.marvell.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de



> -----Original Message-----
> From: Detlev Zundel [mailto:dzu at denx.de]
> Sent: 09 July 2012 20:02
> To: Prafulla Wadaskar
> Cc: Valentin Longchamp; u-boot at lists.denx.de;
> holger.brunck at keymile.com
> Subject: Re: [U-Boot] [PATCH 12/20] arm/km: add support for external
> switchconfiguration
> 
> Hi Prafulla,
> 
> [...]
> 
> > Hi Detlev
> > Clear NAK for this patch
> 
> Do you mean clear NAK for "the patch" or "for the location of the
> files"?

Dear Detlev

Location, I have not gone through the rest of the patch.

> 
> > Let's put it in drivers/net/phy/
> > FYI: there are several drivers used by just one board,
> > [prafulla at pe-dt061 u-boot-marvell.git (master)]$ vim
> > drivers/net/phy/Makefile
> > [prafulla at pe-dt061 u-boot-marvell.git (master)]$ grep -Irn
> > "CONFIG_PHY_ATHEROS" include/configs/*
> > include/configs/microblaze-generic.h:346:# define CONFIG_PHY_ATHEROS
> 1
> > [prafulla at pe-dt061 u-boot-marvell.git (master)]$ vim
> > drivers/net/phy/Makefile
> > [prafulla at pe-dt061 u-boot-marvell.git (master)]$ grep -Irn
> > "CONFIG_PHY_BROADCOM" include/configs/*
> > include/configs/microblaze-generic.h:347:# define
> CONFIG_PHY_BROADCOM   1
> >
> > Anyways peripheral driver should go in drivers/
> 
> Ok, so if the files are moved to drivers/net/phy, then you will ACK
> the
> patch?  If this is the case, then Valentin please move the files and
> be
> done with it.

Thanks...
Regards...
Prafulla . . .