From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Date: Fri, 16 May 2014 09:08:24 +0000 Subject: Re: [PATCH v6 13/17] ARM: shmobile: marzen: Add to shmobile defconfig Message-Id: <20140516090824.GA21825@verge.net.au> List-Id: References: <1400153532-23675-1-git-send-email-horms+renesas@verge.net.au> <1400153532-23675-14-git-send-email-horms+renesas@verge.net.au> <20140515225027.GA29432@verge.net.au> <20140516000409.GB29432@verge.net.au> <20140516085135.GA20299@verge.net.au> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org On Fri, May 16, 2014 at 11:00:24AM +0200, Geert Uytterhoeven wrote: > On Fri, May 16, 2014 at 10:51 AM, Simon Horman wrote: > >> > --- a/arch/arm/configs/shmobile_defconfig > >> > +++ b/arch/arm/configs/shmobile_defconfig > >> > >> > +CONFIG_USB_RCAR_PHY=y > >> > +CONFIG_USB_RCAR_GEN2_PHY=y > >> > >> While I don't dispute the need for CONFIG_USB_RCAR_GEN2_PHY in > >> shmobile_defconfig, this is not needed for Marzen, right? > > > > The CONFIG_USB_RCAR_GEN2_PHY was moved from above (by savedefconfig). > > Sorry, I had missed that. > > > I decided to include that change due to proximity to the new > > CONFIG_USB_RCAR_PHY line. But perhaps it would have been better not to. > > Nevermind. > > BTW, avoiding this moving around in real changes is another reason for > refreshing the defconfigs on a regular basis ;-) Good point.