Re: CRYPTO_DEV_NX merge problem (Re: linux-next: Tree for July 31)

[Michael Neuling <mikey@neuling.org>]

Michael Neuling <mikey@neuling.org> wrote:
> Seth Jennings <sjenning@linux.vnet.ibm.com> wrote:
> 
> > On 07/31/2012 12:58 AM, Michael Neuling wrote:
> > >> Please do not add anything to linux-next included branches/series that is
> > >> destined for v3.7 until after v3.6-rc1 is released.
> > > 
> > > Looks like there is a merge conflict between:
> > >   commit 1b074ac867a2bd08a6f12f0feed7d91e06941723
> > >   Author: Seth Jennings <sjenning@linux.vnet.ibm.com>
> > >   Subject: powerpc/crypto: rework Kconfig
> > > 
> > > and 
> > >   commit fd297b3a7302ab866306f53c1fd1e97b083fe83e
> > >   Author: Anton Blanchard <anton@samba.org>
> > >   powerpc: Enable pseries hardware RNG and crypto modules
> > > 
> > > Seth made CRYPTO_DEV_NX a bool and Anton set it as a module.
> > > 
> > > Hence I get this with a pseries_defconfig
> > >   arch/powerpc/configs/pseries_defconfig:372:warning: symbol value 'm' invalid for CRYPTO_DEV_NX
> > > 
> > > Seth: any reason this can stay as a tristate?
> > 
> > My patch converted CRYPTO_DEV_NX to a submenu access config
> > and the functionality that used to be enabled by it is now
> > enabled by CRYPTO_DEV_NX_ENCRYPT.
> > 
> > The resolution is to change Anton's patch from
> > 
> > +CONFIG_CRYPTO_DEV_NX=m
> > 
> > to
> > 
> > +CONFIG_CRYPTO_DEV_NX=y
> > +CONFIG_CRYPTO_DEV_NX_ENCRYPT=m
> 
> Seth,  
> 
> OK.  Please post the fix as a patch.  Anton's patch is already in the
> powerpc next tree so it won't be changed/rebased directly.  
> 
> I'm not sure who can carry this, since the crypto tree doesn't have
> Anton's change and the powerpc tree doesn't have Seth's change.

Actually Anton's patch is upstream (not just powerpc tree) so this fix
need to go on top of Seth's original patch.  

Seth, your patch breaks mainline pseries/ppc64_defconfig.  You need to
fix this and send to the crypto tree after you get an ack from
Anton/Benh.

Mikey