Re: [PATCH v3 0/3] provide sdhci driver for mmp2 and pxa910

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Jun 4, 2011 at 12:20 AM, Arnd Bergmann <arnd@xxxxxxxx> 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.
>
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux