* [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910
@ 2011-06-03 9:48 zhangfei gao
2011-06-03 16:20 ` Arnd Bergmann
2011-06-03 21:08 ` Philip Rakity
0 siblings, 2 replies; 6+ messages in thread
From: zhangfei gao @ 2011-06-03 9:48 UTC (permalink / raw)
To: linux-arm-kernel
v2->v3, modify according to Arnd and Philip suggestion.
Here are patches to provide separate sdhci driver for mmp2 and pxa910
based on sdhci-platfm.
Rename to sdhci-pxav3.c and sdhci-pxav2.c
Remove sdhci-pxa.c, which is used to share among pxa serious, since
under this method, platform difference have to be put under arch/arm,
which is not easy to track.
As a result, mmp2 mmc resource should be updated accordingly.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910
2011-06-03 9:48 [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910 zhangfei gao
@ 2011-06-03 16:20 ` Arnd Bergmann
2011-06-07 5:26 ` zhangfei gao
2011-06-03 21:08 ` Philip Rakity
1 sibling, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2011-06-03 16:20 UTC (permalink / raw)
To: linux-arm-kernel
On Friday 03 June 2011, zhangfei gao wrote:
> Here are patches to provide separate sdhci driver for mmp2 and pxa910
> based on sdhci-platfm.
> Rename to sdhci-pxav3.c and sdhci-pxav2.c
> Remove sdhci-pxa.c, which is used to share among pxa serious, since
> under this method, platform difference have to be put under arch/arm,
> which is not easy to track.
> As a result, mmp2 mmc resource should be updated accordingly.
Looks all good content-wise. I do have a few comments regarding the
style of submission, please follow that the next time:
* Make all patches a reply to the [PATCH 0/x] email, so they show up
as a single thread. This helps finding the emails when your mail
client sorts threads by last reply. The option in git-send-email
is '--thread --no-chain-reply'.
* The changelog has extra spaces in it that shouldn't be there. I assume
they come from using git-show instead of git-format-patch to create
the emails.
* Make sure a series is bisectable. This means that each patch results
in a working kernel without regressions when you apply only part of
the series. There are multiple ways to get there. In this case, you
could fold the patch that updates the resources into the patch that
changes the format. Alternatively, you could first create the new driver
in the new format, then change the resources, and finally remove the
now obsolete driver.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910
2011-06-03 16:20 ` Arnd Bergmann
@ 2011-06-07 5:26 ` zhangfei gao
0 siblings, 0 replies; 6+ messages in thread
From: zhangfei gao @ 2011-06-07 5:26 UTC (permalink / raw)
To: linux-arm-kernel
On Sat, Jun 4, 2011 at 12:20 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Friday 03 June 2011, zhangfei gao wrote:
>> Here are patches to provide separate sdhci driver for mmp2 and pxa910
>> based on sdhci-platfm.
>> Rename to sdhci-pxav3.c and sdhci-pxav2.c
>> Remove sdhci-pxa.c, which is used to share among pxa serious, since
>> under this method, platform difference have to be put under arch/arm,
>> which is not easy to track.
>> As a result, mmp2 mmc resource should be updated accordingly.
>
> Looks all good content-wise. I do have a few comments regarding the
> style of submission, please follow that the next time:
>
> * Make all patches a reply to the ?[PATCH 0/x] email, so they show up
> ?as a single thread. This helps finding the emails when your mail
> ?client sorts threads by last reply. The option in git-send-email
> ?is '--thread --no-chain-reply'.
>
> * The changelog has extra spaces in it that shouldn't be there. I assume
> ?they come from using git-show instead of git-format-patch to create
> ?the emails.
>
> * Make sure a series is bisectable. This means that each patch results
> ?in a working kernel without regressions when you apply only part of
> ?the series. There are multiple ways to get there. In this case, you
> ?could fold the patch that updates the resources into the patch that
> ?changes the format. Alternatively, you could first create the new driver
> ?in the new format, then change the resources, and finally remove the
> ?now obsolete driver.
>
> ? ? ? ?Arnd
Thanks Arnd for your patient explanation, will take more care next time.
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910
2011-06-03 9:48 [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910 zhangfei gao
2011-06-03 16:20 ` Arnd Bergmann
@ 2011-06-03 21:08 ` Philip Rakity
2011-06-07 5:44 ` zhangfei gao
1 sibling, 1 reply; 6+ messages in thread
From: Philip Rakity @ 2011-06-03 21:08 UTC (permalink / raw)
To: linux-arm-kernel
Please update mmp2 and pxa defconfigs.
regards,
Philip
On Jun 3, 2011, at 2:48 AM, zhangfei gao wrote:
> v2->v3, modify according to Arnd and Philip suggestion.
>
> Here are patches to provide separate sdhci driver for mmp2 and pxa910
> based on sdhci-platfm.
> Rename to sdhci-pxav3.c and sdhci-pxav2.c
> Remove sdhci-pxa.c, which is used to share among pxa serious, since
> under this method, platform difference have to be put under arch/arm,
> which is not easy to track.
> As a result, mmp2 mmc resource should be updated accordingly.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910
2011-06-03 21:08 ` Philip Rakity
@ 2011-06-07 5:44 ` zhangfei gao
2011-06-07 7:00 ` Arnd Bergmann
0 siblings, 1 reply; 6+ messages in thread
From: zhangfei gao @ 2011-06-07 5:44 UTC (permalink / raw)
To: linux-arm-kernel
On Sat, Jun 4, 2011 at 5:08 AM, Philip Rakity <prakity@marvell.com> wrote:
>
> Please update mmp2 and pxa defconfigs.
The trend is replacing defconfig with device tree, it may not
acceptable to update defconfig.
>
> regards,
>
> Philip
>
> On Jun 3, 2011, at 2:48 AM, zhangfei gao wrote:
>
>> v2->v3, modify according to Arnd and Philip suggestion.
>>
>> Here are patches to provide separate sdhci driver for mmp2 and pxa910
>> based on sdhci-platfm.
>> Rename to sdhci-pxav3.c and sdhci-pxav2.c
>> Remove sdhci-pxa.c, which is used to share among pxa serious, since
>> under this method, platform difference have to be put under arch/arm,
>> which is not easy to track.
>> As a result, mmp2 mmc resource should be updated accordingly.
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910
2011-06-07 5:44 ` zhangfei gao
@ 2011-06-07 7:00 ` Arnd Bergmann
0 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2011-06-07 7:00 UTC (permalink / raw)
To: linux-arm-kernel
On Tuesday 07 June 2011 07:44:42 zhangfei gao wrote:
> On Sat, Jun 4, 2011 at 5:08 AM, Philip Rakity <prakity@marvell.com> wrote:
> >
> > Please update mmp2 and pxa defconfigs.
>
> The trend is replacing defconfig with device tree, it may not
> acceptable to update defconfig.
Actually, the defconfigs are not going away, at least not directly.
The device tree will replace the need for individual board files,
but not for defconfig files. As a side-effect, the number of
defconfig files can get reduced, but we should still maintain
them.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-06-07 7:00 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-03 9:48 [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910 zhangfei gao
2011-06-03 16:20 ` Arnd Bergmann
2011-06-07 5:26 ` zhangfei gao
2011-06-03 21:08 ` Philip Rakity
2011-06-07 5:44 ` zhangfei gao
2011-06-07 7:00 ` Arnd Bergmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).