Re: [PATCH v9 0/2] spi: cadence-quadpsi: Add support for the Cadence QSPI controller

[Ramuthevar, Vadivel MuruganX]

Hi Vignesh,

On 17/2/2020 8:18 PM, Vignesh Raghavendra wrote:
> Hi Vadivel,
>
> On 17/02/20 5:22 pm, Mark Brown wrote:
> > On Mon, Feb 17, 2020 at 05:28:38PM +0800, Ramuthevar, Vadivel MuruganX wrote:
> > > On 14/2/2020 8:11 PM, Mark Brown wrote:
> > > > Given that this is a new driver I'd be very surprised if it broke other
> > > > users?  I can imagine it might not work for them and it would definitely
> > > > be much better to get their review but it shouldn't be any worse than
> > > > the current lack of support.
> > > *[Vignesh]*  The legacy driver under drivers/mtd/spi-nor will be removed as
> > > we cannot
> > > support both SPI NOR and SPI NAND with single driver if its under
> > > spi-nor. New driver should be functionally equivalent to existing one.
> > > So I suggest you test this driver on legcay SoCFPGA products.
> > You're not actually removing the driver here, you're adding another
> > driver for the same thing.
> I agree with Mark here.
>
> I realized that you are using same CONFIG option as the old one to build
> this driver. This causes new driver to fail to probe as old driver would
> bind to the node instead (both drivers will be built into the kernel and
> both drivers have same compatible).
>
> So, you should remove the old driver. Could you also include patches
> removing old driver? New driver and bindings are anyways backward
> compatible with existing one
Sure , will remove the existing driver and sending single patch, Thanks!

Regards
Vadivel