public inbox for linux-mmc@vger.kernel.org
 help / color / mirror / Atom feed
* Build breakage in sdhci-s3c
@ 2011-12-02 11:58 Mark Brown
  2011-12-02 12:59 ` Sangwook Lee
  0 siblings, 1 reply; 5+ messages in thread
From: Mark Brown @ 2011-12-02 11:58 UTC (permalink / raw)
  To: Sangwook Lee, Chris Ball; +Cc: Kukjin Kim, linaro-dev, linux-mmc

The mmc and therefore -next currently fail to build on Samsung platforms
due to commit 666b6a (mmc: sdhci-s3c: Add pm_caps into SD/MMC host)
which added use of a new field pm_caps in the platform data without
actually adding that field to the struct.  I guess there's a separate
patch floating about somewhere which adds the field but it's not in
-next and both commits should be going in via the same tree to avoid
bisection breakage - ideally both the field and its use should be added
as part of a single commit.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Build breakage in sdhci-s3c
  2011-12-02 11:58 Build breakage in sdhci-s3c Mark Brown
@ 2011-12-02 12:59 ` Sangwook Lee
  2011-12-02 13:04   ` Mark Brown
  0 siblings, 1 reply; 5+ messages in thread
From: Sangwook Lee @ 2011-12-02 12:59 UTC (permalink / raw)
  To: Kukjin Kim; +Cc: Chris Ball, linaro-dev, linux-mmc, Mark Brown

Hi Kukjin

I checked out your git to look into Mark's email.

git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git for-next

and I found out that the following patch wasn't applied This  patch
was acked by you and Thomas.

http://comments.gmane.org/gmane.linux.kernel.samsung-soc/7832
This patch has the title : [PATCH v2 1/2] ARM: SAMSUNG: Add pm_caps
into platform data

Could you help me to solve the below problem ?

Thanks
Sangwook


On 2 December 2011 11:58, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> The mmc and therefore -next currently fail to build on Samsung platforms
> due to commit 666b6a (mmc: sdhci-s3c: Add pm_caps into SD/MMC host)
> which added use of a new field pm_caps in the platform data without
> actually adding that field to the struct.  I guess there's a separate
> patch floating about somewhere which adds the field but it's not in
> -next and both commits should be going in via the same tree to avoid
> bisection breakage - ideally both the field and its use should be added
> as part of a single commit.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Build breakage in sdhci-s3c
  2011-12-02 12:59 ` Sangwook Lee
@ 2011-12-02 13:04   ` Mark Brown
  2011-12-02 13:11     ` Sangwook Lee
  2011-12-03  9:25     ` Kukjin Kim
  0 siblings, 2 replies; 5+ messages in thread
From: Mark Brown @ 2011-12-02 13:04 UTC (permalink / raw)
  To: Sangwook Lee; +Cc: Kukjin Kim, Chris Ball, linux-mmc

On Fri, Dec 02, 2011 at 12:59:42PM +0000, Sangwook Lee wrote:

> and I found out that the following patch wasn't applied This  patch
> was acked by you and Thomas.

> http://comments.gmane.org/gmane.linux.kernel.samsung-soc/7832
> This patch has the title : [PATCH v2 1/2] ARM: SAMSUNG: Add pm_caps
> into platform data

> Could you help me to solve the below problem ?

Probably the simplest/best solution here is for Chris to apply this
patch with Kukjin and Thomas' acks to the mmc tree.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Build breakage in sdhci-s3c
  2011-12-02 13:04   ` Mark Brown
@ 2011-12-02 13:11     ` Sangwook Lee
  2011-12-03  9:25     ` Kukjin Kim
  1 sibling, 0 replies; 5+ messages in thread
From: Sangwook Lee @ 2011-12-02 13:11 UTC (permalink / raw)
  To: Mark Brown, Thomas Abraham; +Cc: Kukjin Kim, Chris Ball, linux-mmc

+Thomas


On 2 December 2011 13:04, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> On Fri, Dec 02, 2011 at 12:59:42PM +0000, Sangwook Lee wrote:
>
>> and I found out that the following patch wasn't applied This  patch
>> was acked by you and Thomas.
>
>> http://comments.gmane.org/gmane.linux.kernel.samsung-soc/7832
>> This patch has the title : [PATCH v2 1/2] ARM: SAMSUNG: Add pm_caps
>> into platform data
>
>> Could you help me to solve the below problem ?
>
> Probably the simplest/best solution here is for Chris to apply this
> patch with Kukjin and Thomas' acks to the mmc tree.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 5+ messages in thread

* RE: Build breakage in sdhci-s3c
  2011-12-02 13:04   ` Mark Brown
  2011-12-02 13:11     ` Sangwook Lee
@ 2011-12-03  9:25     ` Kukjin Kim
  1 sibling, 0 replies; 5+ messages in thread
From: Kukjin Kim @ 2011-12-03  9:25 UTC (permalink / raw)
  To: 'Mark Brown', 'Sangwook Lee'
  Cc: 'Chris Ball', linux-mmc

Mark Brown wrote:
> 
> On Fri, Dec 02, 2011 at 12:59:42PM +0000, Sangwook Lee wrote:
> 
> > and I found out that the following patch wasn't applied This  patch
> > was acked by you and Thomas.
> 
> > http://comments.gmane.org/gmane.linux.kernel.samsung-soc/7832
> > This patch has the title : [PATCH v2 1/2] ARM: SAMSUNG: Add pm_caps
> > into platform data
> 
> > Could you help me to solve the below problem ?
> 
> Probably the simplest/best solution here is for Chris to apply this
> patch with Kukjin and Thomas' acks to the mmc tree.

Uhm...it is due to late pushing from my local, I need to sort out my tree
now :(

Anyway, Mark's suggestion makes sense, in this case. Because it can be
happened during merge window even though I solve/push it now.

Hi Chris,
Could you please pick up Sangwook Lee's '[PATCH v2 1/2] ARM: SAMSUNG: Add
pm_caps into platform data' with my ack?

If any problems, please let me know.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2011-12-03  9:25 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-02 11:58 Build breakage in sdhci-s3c Mark Brown
2011-12-02 12:59 ` Sangwook Lee
2011-12-02 13:04   ` Mark Brown
2011-12-02 13:11     ` Sangwook Lee
2011-12-03  9:25     ` Kukjin Kim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox