On Wed, 2012-07-04 at 11:06 +0300, Shmulik Ladkani wrote:
> Introduce 'ubi->bad_peb_limit', which specifies an upper limit of PEBs> ubi expects to go bad.> Currently, it is initialized to a fixed percentage of total PEBs in the> ubi device (configurable via CONFIG_MTD_UBI_BEB_LIMIT).> > The 'bad_peb_limit' is intended to be used for calculating the amount of> PEBs ubi needs to reserve for bad eraseblock handling.> > Signed-off-by: Shmulik Ladkani <shmulik.ladkani@gmail.com>> ---> arch/arm/configs/sam9_l9260_defconfig | 1 +
I've also amended the Kconfig text a tiny bit and dropped the defconfig
changes - let's have them separately as a single patch at the end of the
series.

On Wed, 18 Jul 2012 13:40:53 +0300 Artem Bityutskiy <dedekind1@gmail.com> wrote:
> I've also amended the Kconfig text a tiny bit and dropped the defconfig> changes - let's have them separately as a single patch at the end of the> series.
Wouldn't having the defconfig change as the last patch break things for
those defconfigs that had explicitly set CONFIG_MTD_UBI_BEB_RESERVE
other than the default?
Meaning, if the one-before-last would be "kill CONFIG_MTD_UBI_BEB_RESERVE",
then those defconfigs that had _explicitly_ set a BEB_RESERVE value,
which do not YET set a BEB_LIMIT value, will have their BEB_LIMIT as
the default - but they actually meant a specific value other than the
default.
This is why I tried to:
- set the CONFIG_MTD_UBI_BEB_LIMIT in defconfigs as part of the commit
which introduces this config (copy same value as their RESERVE config)
- kill all CONFIG_MTD_UBI_BEB_RESERVE references from defconfigs as part
of the commit which kills it
Regards,
Shmulik

On Thu, 2012-07-19 at 09:16 +0300, Shmulik Ladkani wrote:
> On Wed, 18 Jul 2012 13:40:53 +0300 Artem Bityutskiy <dedekind1@gmail.com> wrote:> > I've also amended the Kconfig text a tiny bit and dropped the defconfig> > changes - let's have them separately as a single patch at the end of the> > series.> > Wouldn't having the defconfig change as the last patch break things for> those defconfigs that had explicitly set CONFIG_MTD_UBI_BEB_RESERVE> other than the default?
OK, fair enough, but let's have it as a 2 separate patches anyway. It is
not a big deal to first change the defconfig, and then actually add the
new option.