From mboxrd@z Thu Jan  1 00:00:00 1970
From: Boris Brezillon <boris.brezillon@bootlin.com>
Date: Mon, 12 Nov 2018 07:45:26 +0100
Subject: [U-Boot] [PATCH 1/2] dfu: nand: Add missing dependency on
 CMD_MTDPARTS
In-Reply-To: <CAMty3ZCs-J=pSj_kwJt6ePhHuzWbm1CYPgde214t7fkC8H8ZRw@mail.gmail.com>
References: <20181110112209.28872-1-boris.brezillon@bootlin.com>
 <CAMty3ZCs-J=pSj_kwJt6ePhHuzWbm1CYPgde214t7fkC8H8ZRw@mail.gmail.com>
Message-ID: <20181112074526.19c6b2f8@bbrezillon>
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

Hi Jagan,

On Mon, 12 Nov 2018 10:13:40 +0530
Jagan Teki <jagan@amarulasolutions.com> wrote:

> On Sat, Nov 10, 2018 at 4:52 PM Boris Brezillon
> <boris.brezillon@bootlin.com> wrote:
> >
> > dfu_fill_entity_nand() uses find_dev_and_part() and mtdparts_init()
> > which are provided by cmd/mtdparts.c.
> >
> > Add the dependency to avoid build failures when CMD_MTDPARTS is not
> > selected.
> >
> > Reported-by: Jagan Teki <jagan@amarulasolutions.com>
> > Fixes: 6828e602b722d ("dfu: Migrate to Kconfig")
> > Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
> > ---
> >  drivers/dfu/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)  
> 
> Squashed both patches into "cmd: ubi: Remove useless call to
> mtdparts_init()" patch.

Sorry to complain again, but I don't think this was the right thing to
do. Those 2 patches are unrelated to "cmd: ubi: Remove useless call to
->mtdparts_init()", it's just that this commit uncovers problems in the
dependency definition of the DFU_NAND and MTD{PARTS,IDS}_DEFAULT
options.

If you want to keep things bisectable, it would be preferable to
move those 2 commits before "cmd: ubi: Remove useless call to
->mtdparts_init()" (and rework the commit messages accordingly).

Regards,

Boris