All of lore.kernel.org
 help / color / mirror / Atom feed
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: kumar.gala@freescale.com, david-b@pacbell.net,
	linuxppc-dev@ozlabs.org, linux-mtd@lists.infradead.org,
	spi-devel-general@lists.sourceforge.net,
	Mingkai Hu <Mingkai.hu@freescale.com>,
	tie-fei.zang@freescale.com
Subject: Re: [PATCH v4 5/5] mtd: m25p80: add support to parse the partitions by OF node
Date: Sun, 17 Oct 2010 10:44:50 +0300	[thread overview]
Message-ID: <1287301490.1951.0.camel@localhost> (raw)
In-Reply-To: <AANLkTimr03KAKnz5sfwm19=KzB-ik95WM_qQO0YP9qVs@mail.gmail.com>

On Sat, 2010-10-16 at 19:05 -0600, Grant Likely wrote:
> On Sat, Oct 16, 2010 at 1:17 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
> > On Tue, 2010-10-12 at 18:18 +0800, Mingkai Hu wrote:
> >> Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
> >> Acked-by: Grant Likely <grant.likely@secretlab.ca>
> >> ---
> >> v4:
> >>  - Updated to latest kernel base(Linux 2.6.36-rc7).
> >>  - Made changes according to Grant's comments.
> >
> > Looks good to me, pushed to l2-mtd-2.6.git, thanks.
> 
> Wait!  It will break whenever CONFIG_OF is not set.  I broke
> linux-next with this patch.  It can be solved by wrapping the block
> with #ifdef CONFIG_OF, but I'd like to find a better solution.

OK, removed.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

WARNING: multiple messages have this Message-ID (diff)
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: kumar.gala@freescale.com, david-b@pacbell.net,
	linuxppc-dev@ozlabs.org, linux-mtd@lists.infradead.org,
	spi-devel-general@lists.sourceforge.net,
	Mingkai Hu <Mingkai.hu@freescale.com>
Subject: Re: [PATCH v4 5/5] mtd: m25p80: add support to parse the partitions by OF node
Date: Sun, 17 Oct 2010 10:44:50 +0300	[thread overview]
Message-ID: <1287301490.1951.0.camel@localhost> (raw)
In-Reply-To: <AANLkTimr03KAKnz5sfwm19=KzB-ik95WM_qQO0YP9qVs@mail.gmail.com>

On Sat, 2010-10-16 at 19:05 -0600, Grant Likely wrote:
> On Sat, Oct 16, 2010 at 1:17 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
> > On Tue, 2010-10-12 at 18:18 +0800, Mingkai Hu wrote:
> >> Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
> >> Acked-by: Grant Likely <grant.likely@secretlab.ca>
> >> ---
> >> v4:
> >>  - Updated to latest kernel base(Linux 2.6.36-rc7).
> >>  - Made changes according to Grant's comments.
> >
> > Looks good to me, pushed to l2-mtd-2.6.git, thanks.
> 
> Wait!  It will break whenever CONFIG_OF is not set.  I broke
> linux-next with this patch.  It can be solved by wrapping the block
> with #ifdef CONFIG_OF, but I'd like to find a better solution.

OK, removed.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

WARNING: multiple messages have this Message-ID (diff)
From: Artem Bityutskiy <dedekind1@gmail.com>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: kumar.gala@freescale.com, david-b@pacbell.net,
	linuxppc-dev@ozlabs.org, linux-mtd@lists.infradead.org,
	spi-devel-general@lists.sourceforge.net,
	Mingkai Hu <Mingkai.hu@freescale.com>
Subject: Re: [PATCH v4 5/5] mtd: m25p80: add support to parse the partitions by OF node
Date: Sun, 17 Oct 2010 10:44:50 +0300	[thread overview]
Message-ID: <1287301490.1951.0.camel@localhost> (raw)
In-Reply-To: <AANLkTimr03KAKnz5sfwm19=KzB-ik95WM_qQO0YP9qVs@mail.gmail.com>

On Sat, 2010-10-16 at 19:05 -0600, Grant Likely wrote:
> On Sat, Oct 16, 2010 at 1:17 PM, Artem Bityutskiy <dedekind1@gmail.com> wrote:
> > On Tue, 2010-10-12 at 18:18 +0800, Mingkai Hu wrote:
> >> Signed-off-by: Mingkai Hu <Mingkai.hu@freescale.com>
> >> Acked-by: Grant Likely <grant.likely@secretlab.ca>
> >> ---
> >> v4:
> >>  - Updated to latest kernel base(Linux 2.6.36-rc7).
> >>  - Made changes according to Grant's comments.
> >
> > Looks good to me, pushed to l2-mtd-2.6.git, thanks.
> 
> Wait!  It will break whenever CONFIG_OF is not set.  I broke
> linux-next with this patch.  It can be solved by wrapping the block
> with #ifdef CONFIG_OF, but I'd like to find a better solution.

OK, removed.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

  reply	other threads:[~2010-10-17  7:47 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-12 10:18 [PATCH v4 0/5] refactor spi_mpc8xxx.c and add eSPI controller support Mingkai Hu
2010-10-12 10:18 ` Mingkai Hu
2010-10-12 10:18 ` Mingkai Hu
2010-10-12 10:18 ` [PATCH v4 1/5] spi/mpc8xxx: rename spi_mpc8xxx.c to spi_fsl_spi.c Mingkai Hu
2010-10-12 10:18   ` Mingkai Hu
2010-10-12 10:18   ` Mingkai Hu
2010-10-12 10:18   ` [PATCH v4 2/5] spi/mpc8xxx: refactor the common code for SPI/eSPI controller Mingkai Hu
2010-10-12 10:18     ` Mingkai Hu
2010-10-12 10:18     ` Mingkai Hu
2010-10-12 10:18     ` [PATCH v4 3/5] eSPI: add eSPI controller support Mingkai Hu
2010-10-12 10:18       ` Mingkai Hu
2010-10-12 10:18       ` Mingkai Hu
2010-10-12 10:18       ` [PATCH v4 4/5] powerpc/of: add eSPI controller dts bindings and DTS modification Mingkai Hu
2010-10-12 10:18         ` Mingkai Hu
2010-10-12 10:18         ` [PATCH v4 5/5] mtd: m25p80: add support to parse the partitions by OF node Mingkai Hu
2010-10-12 10:18           ` Mingkai Hu
2010-10-12 10:18           ` Mingkai Hu
2010-10-13  3:44           ` Grant Likely
2010-10-13  3:44             ` Grant Likely
2010-10-16 19:17           ` Artem Bityutskiy
2010-10-16 19:17             ` Artem Bityutskiy
2010-10-16 19:17             ` Artem Bityutskiy
2010-10-17  1:05             ` Grant Likely
2010-10-17  1:05               ` Grant Likely
2010-10-17  1:05               ` Grant Likely
2010-10-17  7:44               ` Artem Bityutskiy [this message]
2010-10-17  7:44                 ` Artem Bityutskiy
2010-10-17  7:44                 ` Artem Bityutskiy
2010-10-22  6:01                 ` Kumar Gala
2010-10-22  6:01                   ` Kumar Gala
2010-10-22  7:39                   ` Grant Likely
2010-10-13  3:41         ` [PATCH v4 4/5] powerpc/of: add eSPI controller dts bindings and DTS modification Grant Likely
2010-10-13  3:41           ` Grant Likely
2010-10-13  3:41           ` Grant Likely
2010-10-13  3:41       ` [PATCH v4 3/5] eSPI: add eSPI controller support Grant Likely
2010-10-13  3:41         ` Grant Likely
2010-10-13  3:41         ` Grant Likely
2010-10-13  3:41     ` [PATCH v4 2/5] spi/mpc8xxx: refactor the common code for SPI/eSPI controller Grant Likely
2010-10-13  3:41       ` Grant Likely
2010-10-13  3:41       ` Grant Likely
2010-10-13  3:40   ` [PATCH v4 1/5] spi/mpc8xxx: rename spi_mpc8xxx.c to spi_fsl_spi.c Grant Likely
2010-10-13  3:40     ` Grant Likely

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=1287301490.1951.0.camel@localhost \
    --to=dedekind1@gmail.com \
    --cc=Mingkai.hu@freescale.com \
    --cc=david-b@pacbell.net \
    --cc=grant.likely@secretlab.ca \
    --cc=kumar.gala@freescale.com \
    --cc=linux-mtd@lists.infradead.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=spi-devel-general@lists.sourceforge.net \
    --cc=tie-fei.zang@freescale.com \
    /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.