* 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