Subject: Re: [PATCH] powerpc: Add aacraid and nvme to powernv_defconfig

From: Benjamin Herrenschmidt <[email-masked]>
Date: Sat, 23 Dec 2017 16:08:40 +1100

On Sat, 2017-12-23 at 11:40 +1100, Alexey Kardashevskiy wrote:
> On 20/12/17 13:14, Benjamin Herrenschmidt wrote:
> > On Wed, 2017-12-20 at 12:59 +1100, Alexey Kardashevskiy wrote:
> > > On 20/12/17 12:51, Benjamin Herrenschmidt wrote:
> > > > These adapters can be found in a number of our systems, so let's
> > > > enable the corresponding drivers by default.
> > > > 
> > > > Signed-off-by: Benjamin Herrenschmidt <[email-masked]>
> > > > ---
> > > > --- a/arch/powerpc/configs/powernv_defconfig	2017-12-19 18:37:24.803470591 -0600
> > > > +++ b/arch/powerpc/configs/powernv_defconfig	2017-12-19 19:47:57.931952417 -0600
> > > > @@ -97,6 +97,7 @@
> > > >  CONFIG_BLK_DEV_RAM=y
> > > >  CONFIG_BLK_DEV_RAM_SIZE=65536
> > > >  CONFIG_VIRTIO_BLK=m
> > > > +CONFIG_BLK_DEV_NVME=y
> > > >  CONFIG_IDE=y
> > > >  CONFIG_BLK_DEV_IDECD=y
> > > >  CONFIG_BLK_DEV_GENERIC=y
> > > > @@ -113,6 +114,7 @@
> > > >  CONFIG_SCSI_CXGB4_ISCSI=m
> > > >  CONFIG_SCSI_BNX2_ISCSI=m
> > > >  CONFIG_BE2ISCSI=m
> > > > +CONFIG_SCSI_AACRAID=y
> > > >  CONFIG_SCSI_MPT2SAS=m
> > > >  CONFIG_SCSI_SYM53C8XX_2=m
> > > >  CONFIG_SCSI_SYM53C8XX_DMA_ADDRESSING_MODE=0
> > > > 
> > > 
> > > "y", not "m"?
> > 
> > Yes. I like my boot devices to be "y"
> 
> 
> Then rename it to powernv_benh_defconfig :) Everything else seems to cope
> well with "m" - just look above.

No. Most of the above aren't needed as boot devices on powernv systems.

If you look at IPR for example, it's Y not N.

Cheers,
Ben.