All of lore.kernel.org
 help / color / mirror / Atom feed
From: Huang Shijie <b32955@freescale.com>
To: <y@karo-electronics.de>
Cc: "Fabio Estevam" <fabio.estevam@freescale.com>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Russell King" <linux@arm.linux.org.uk>,
	"Pawel Moll" <pawel.moll@arm.com>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"Ian Campbell" <ijc+devicetree@hellion.org.uk>,
	"Artem Bityutskiy" <artem.bityutskiy@linux.intel.com>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	devicetree@vger.kernel.org, "Rob Herring" <robh+dt@kernel.org>,
	linux-mtd@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	"Rob Landley" <rob@landley.net>,
	"Kumar Gala" <galak@codeaurora.org>,
	"Brian Norris" <computersforpeace@gmail.com>,
	"David Woodhouse" <dwmw2@infradead.org>,
	"Sascha Hauer" <kernel@pengutronix.de>,
	"Shawn Guo" <shawn.guo@freescale.com>,
	"Lothar Waßmann" <LW@karo-electronics.de>
Subject: Re: [PATCH] mtd: gpmi: make blockmark swapping optional
Date: Fri, 21 Mar 2014 13:36:12 +0800	[thread overview]
Message-ID: <532BCFCC.4080205@freescale.com> (raw)
In-Reply-To: <1395235406-13449-1-git-send-email-y>

于 2014年03月19日 21:23, y@karo-electronics.de 写道:
> +	/* Set up swap_block_mark, must be set before the gpmi_set_geometry() */
> +	if (!of_property_read_bool(this->dev->of_node, "fsl,no-blockmark-swap"))
> +		this->swap_block_mark = !GPMI_IS_MX23(this);


Our ROM guy had confirmed that the Rom will disable the swapping if we 
set the DISBBM bit.
But please do not change any logic for imx23/imx28.

I really do not know what's the benefit we can get from this patch.
Please send the new version about this patch if you want this feature.


thanks
Huang Shijie

WARNING: multiple messages have this Message-ID (diff)
From: b32955@freescale.com (Huang Shijie)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] mtd: gpmi: make blockmark swapping optional
Date: Fri, 21 Mar 2014 13:36:12 +0800	[thread overview]
Message-ID: <532BCFCC.4080205@freescale.com> (raw)
In-Reply-To: <1395235406-13449-1-git-send-email-y>

? 2014?03?19? 21:23, y at karo-electronics.de ??:
> +	/* Set up swap_block_mark, must be set before the gpmi_set_geometry() */
> +	if (!of_property_read_bool(this->dev->of_node, "fsl,no-blockmark-swap"))
> +		this->swap_block_mark = !GPMI_IS_MX23(this);


Our ROM guy had confirmed that the Rom will disable the swapping if we 
set the DISBBM bit.
But please do not change any logic for imx23/imx28.

I really do not know what's the benefit we can get from this patch.
Please send the new version about this patch if you want this feature.


thanks
Huang Shijie

WARNING: multiple messages have this Message-ID (diff)
From: Huang Shijie <b32955@freescale.com>
To: y@karo-electronics.de
Cc: "Fabio Estevam" <fabio.estevam@freescale.com>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Russell King" <linux@arm.linux.org.uk>,
	"Pawel Moll" <pawel.moll@arm.com>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"Ian Campbell" <ijc+devicetree@hellion.org.uk>,
	"Artem Bityutskiy" <artem.bityutskiy@linux.intel.com>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	devicetree@vger.kernel.org, "Rob Herring" <robh+dt@kernel.org>,
	linux-mtd@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	"Rob Landley" <rob@landley.net>,
	"Kumar Gala" <galak@codeaurora.org>,
	"Brian Norris" <computersforpeace@gmail.com>,
	"David Woodhouse" <dwmw2@infradead.org>,
	"Sascha Hauer" <kernel@pengutronix.de>,
	"Shawn Guo" <shawn.guo@freescale.com>,
	"Lothar Waßmann" <LW@karo-electronics.de>
Subject: Re: [PATCH] mtd: gpmi: make blockmark swapping optional
Date: Fri, 21 Mar 2014 13:36:12 +0800	[thread overview]
Message-ID: <532BCFCC.4080205@freescale.com> (raw)
In-Reply-To: <1395235406-13449-1-git-send-email-y>

于 2014年03月19日 21:23, y@karo-electronics.de 写道:
> +	/* Set up swap_block_mark, must be set before the gpmi_set_geometry() */
> +	if (!of_property_read_bool(this->dev->of_node, "fsl,no-blockmark-swap"))
> +		this->swap_block_mark = !GPMI_IS_MX23(this);


Our ROM guy had confirmed that the Rom will disable the swapping if we 
set the DISBBM bit.
But please do not change any logic for imx23/imx28.

I really do not know what's the benefit we can get from this patch.
Please send the new version about this patch if you want this feature.


thanks
Huang Shijie


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Huang Shijie <b32955@freescale.com>
To: <y@karo-electronics.de>
Cc: "Arnd Bergmann" <arnd@arndb.de>,
	"Artem Bityutskiy" <artem.bityutskiy@linux.intel.com>,
	"Brian Norris" <computersforpeace@gmail.com>,
	"David Woodhouse" <dwmw2@infradead.org>,
	"Fabio Estevam" <fabio.estevam@freescale.com>,
	"Ian Campbell" <ijc+devicetree@hellion.org.uk>,
	"Kumar Gala" <galak@codeaurora.org>,
	"Lothar Waßmann" <LW@karo-electronics.de>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Pawel Moll" <pawel.moll@arm.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Rob Landley" <rob@landley.net>,
	"Russell King" <linux@arm.linux.org.uk>,
	"Sascha Hauer" <kernel@pengutronix.de>,
	"Shawn Guo" <shawn.guo@freescale.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-mtd@lists.infradead.org
Subject: Re: [PATCH] mtd: gpmi: make blockmark swapping optional
Date: Fri, 21 Mar 2014 13:36:12 +0800	[thread overview]
Message-ID: <532BCFCC.4080205@freescale.com> (raw)
In-Reply-To: <1395235406-13449-1-git-send-email-y>

于 2014年03月19日 21:23, y@karo-electronics.de 写道:
> +	/* Set up swap_block_mark, must be set before the gpmi_set_geometry() */
> +	if (!of_property_read_bool(this->dev->of_node, "fsl,no-blockmark-swap"))
> +		this->swap_block_mark = !GPMI_IS_MX23(this);


Our ROM guy had confirmed that the Rom will disable the swapping if we 
set the DISBBM bit.
But please do not change any logic for imx23/imx28.

I really do not know what's the benefit we can get from this patch.
Please send the new version about this patch if you want this feature.


thanks
Huang Shijie


  parent reply	other threads:[~2014-03-21  5:36 UTC|newest]

Thread overview: 125+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-19 13:23 [PATCH] mtd: gpmi: make blockmark swapping optional y
2014-03-19 13:23 ` y at karo-electronics.de
2014-03-20  8:33 ` Juergen Beisert
2014-03-20  8:33   ` Juergen Beisert
2014-03-20  8:33   ` Juergen Beisert
2014-03-20  8:33   ` Juergen Beisert
2014-03-20  9:20   ` Lothar Waßmann
2014-03-20  9:20     ` Lothar Waßmann
2014-03-20  9:20     ` Lothar Waßmann
2014-03-20  9:20     ` Lothar Waßmann
2014-03-20  9:15 ` Huang Shijie
2014-03-20  9:15   ` Huang Shijie
2014-03-20  9:15   ` Huang Shijie
2014-03-20  9:15   ` Huang Shijie
2014-03-20  9:21   ` Lothar Waßmann
2014-03-20  9:21     ` Lothar Waßmann
2014-03-20  9:21     ` Lothar Waßmann
2014-03-20  9:21     ` Lothar Waßmann
2014-03-20  9:47     ` Huang Shijie
2014-03-20  9:47       ` Huang Shijie
2014-03-20  9:47       ` Huang Shijie
2014-03-20  9:47       ` Huang Shijie
2014-03-20 10:09       ` Lothar Waßmann
2014-03-20 10:09         ` Lothar Waßmann
2014-03-20 10:09         ` Lothar Waßmann
2014-03-20 10:09         ` Lothar Waßmann
2014-03-20 10:56         ` Huang Shijie
2014-03-20 10:56           ` Huang Shijie
2014-03-20 10:56           ` Huang Shijie
2014-03-20 10:56           ` Huang Shijie
2014-03-21  5:36 ` Huang Shijie [this message]
2014-03-21  5:36   ` Huang Shijie
2014-03-21  5:36   ` Huang Shijie
2014-03-21  5:36   ` Huang Shijie
2014-03-21 10:50   ` [PATCHv2 1/1] " Lothar Waßmann
2014-03-21 10:50     ` Lothar Waßmann
2014-03-21 10:50     ` Lothar Waßmann
2014-03-24  9:59     ` Huang Shijie
2014-03-24  9:59       ` Huang Shijie
2014-03-24  9:59       ` Huang Shijie
2014-03-24  9:59       ` Huang Shijie
2014-03-26  8:51       ` Lothar Waßmann
2014-03-26  8:51         ` Lothar Waßmann
2014-03-26  8:51         ` Lothar Waßmann
2014-03-26 10:41         ` Huang Shijie
2014-03-26 10:41           ` Huang Shijie
2014-03-26 10:41           ` Huang Shijie
2014-03-26 10:41           ` Huang Shijie
2014-03-26 11:55           ` Lothar Waßmann
2014-03-26 11:55             ` Lothar Waßmann
2014-03-26 11:55             ` Lothar Waßmann
2014-03-26 11:55             ` Lothar Waßmann
2014-03-27  9:59             ` Huang Shijie
2014-03-27  9:59               ` Huang Shijie
2014-03-27  9:59               ` Huang Shijie
2014-03-27  9:59               ` Huang Shijie
2014-03-27 12:21               ` Lothar Waßmann
2014-03-27 12:21                 ` Lothar Waßmann
2014-03-27 12:21                 ` Lothar Waßmann
2014-03-27 12:21                 ` Lothar Waßmann
2014-03-28  2:09                 ` Huang Shijie
2014-03-28  2:09                   ` Huang Shijie
2014-03-28  2:09                   ` Huang Shijie
2014-03-28  2:09                   ` Huang Shijie
2014-03-28  8:16                   ` Lothar Waßmann
2014-03-28  8:16                     ` Lothar Waßmann
2014-03-28  8:16                     ` Lothar Waßmann
2014-03-28  8:39                     ` Huang Shijie
2014-03-28  8:39                       ` Huang Shijie
2014-03-28  8:39                       ` Huang Shijie
2014-03-28  8:39                       ` Huang Shijie
2014-03-28  9:00                       ` Sascha Hauer
2014-03-28  9:00                         ` Sascha Hauer
2014-03-28  9:00                         ` Sascha Hauer
2014-03-28  9:26                         ` Huang Shijie
2014-03-28  9:26                           ` Huang Shijie
2014-03-28  9:26                           ` Huang Shijie
2014-03-28  9:31                           ` Lothar Waßmann
2014-03-28  9:31                             ` Lothar Waßmann
2014-03-28  9:31                             ` Lothar Waßmann
2014-03-28  9:31                             ` Lothar Waßmann
2014-03-28 10:09                             ` Huang Shijie
2014-03-28 10:09                               ` Huang Shijie
2014-03-28 10:09                               ` Huang Shijie
2014-03-28 10:09                               ` Huang Shijie
2014-03-28 10:18                               ` Lothar Waßmann
2014-03-28 10:18                                 ` Lothar Waßmann
2014-03-28 10:18                                 ` Lothar Waßmann
2014-03-28 10:18                                 ` Lothar Waßmann
2014-03-28  9:01                       ` Lothar Waßmann
2014-03-28  9:01                         ` Lothar Waßmann
2014-03-28  9:01                         ` Lothar Waßmann
2014-03-28  9:01                         ` Lothar Waßmann
2014-03-28  9:33                         ` Huang Shijie
2014-03-28  9:33                           ` Huang Shijie
2014-03-28  9:33                           ` Huang Shijie
2014-03-28  9:33                           ` Huang Shijie
2014-03-28  9:38                           ` Huang Shijie
2014-03-28  9:38                             ` Huang Shijie
2014-03-28  9:38                             ` Huang Shijie
2014-03-28  9:38                             ` Huang Shijie
2014-03-28 10:13                             ` Lothar Waßmann
2014-03-28 10:13                               ` Lothar Waßmann
2014-03-28 10:13                               ` Lothar Waßmann
2014-03-28 10:13                               ` Lothar Waßmann
2014-03-28 10:35 ` [PATCHv3 0/2] " Lothar Waßmann
2014-03-28 10:35   ` Lothar Waßmann
2014-03-28 10:35   ` Lothar Waßmann
2014-03-28 10:35   ` [PATCHv3 1/2] " Lothar Waßmann
2014-03-28 10:35     ` Lothar Waßmann
2014-03-28 10:35     ` Lothar Waßmann
2014-03-31  2:06     ` Huang Shijie
2014-03-31  2:06       ` Huang Shijie
2014-03-31  2:06       ` Huang Shijie
2014-03-28 10:35   ` [PATCHv3 2/2] of/mtd/nand: add generic binding and helper for NAND_BBT_NO_OOB_BBM Lothar Waßmann
2014-03-28 10:35     ` Lothar Waßmann
2014-03-28 10:35     ` Lothar Waßmann
  -- strict thread matches above, loose matches on Subject: below --
2014-03-19 13:23 [PATCH] mtd: gpmi: make blockmark swapping optional y
2014-03-19 13:23 y
2014-03-19 13:23 y
2014-03-19 13:23 y
2014-03-19 13:23 y
2014-03-19 13:23 y
2014-03-19 13:23 y
2014-03-19 13:23 y

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=532BCFCC.4080205@freescale.com \
    --to=b32955@freescale.com \
    --cc=LW@karo-electronics.de \
    --cc=arnd@arndb.de \
    --cc=artem.bityutskiy@linux.intel.com \
    --cc=computersforpeace@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dwmw2@infradead.org \
    --cc=fabio.estevam@freescale.com \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=rob@landley.net \
    --cc=robh+dt@kernel.org \
    --cc=shawn.guo@freescale.com \
    --cc=y@karo-electronics.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.