Re: [PATCHv3 0/6] spi: omap2-mcspi: driver updates

From: Shubhrajyoti
Date: Sat Mar 31 2012 - 07:12:43 EST


On Saturday 31 March 2012 11:14 AM, Grant Likely wrote:
> On Fri, 30 Mar 2012 15:50:16 +0530, Shubhrajyoti D <shubhrajyoti@xxxxxx> wrote:
>> The patch series does the following cleanups
>> - Converts the spi to module_platform_driver
>> - Use the devm functions so that the freeing need not
>> be done in the driver.
>> - Makes the driver use autosuspend
>> - Folds Benoit's bus_num removal patch in the series
>>
>> Changes from v1
>> - Makes the driver use autosuspend
>> - Folds Benoit's bus_num removal patch in the series
>>
>> Changes from v2
>> - The tmp variable is used to write this can be optimised
>> as it is not needed if the value is directly written.
>> Acknowledge Tarun for the suggestion.
>>
>> This is also available through
>> git : git://gitorious.org/linus-tree/linus-tree.git
>> branch : spi
>>
>> This is targeted for v3.5.
> Okay, now is a good time to talk about git pull request workflow. 10
> days ago you published a git tree and I replied that I had pulled it
> into mine. Now you've got a new branch with the patches from the old
> branch rebased onto a new head. Compairing your branch with mine now
> looks like this:
>
> $ git fetch git://gitorious.org/linus-tree/linus-tree.git spi
> remote: Counting objects: 34, done.
> remote: Compressing objects: 100% (28/28), done.
> remote: Total 30 (delta 24), reused 6 (delta 2)
> Unpacking objects: 100% (30/30), done.
> From git://gitorious.org/linus-tree/linus-tree
> * branch spi -> FETCH_HEAD
> $ git show-branch --topic origin spi/next FETCH_HEAD
> ! [origin] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/sparc
> ! [spi/next] Merge branch 'spi' of git://gitorious.org/linus-tree/linus-tree into spi/next
> ! [FETCH_HEAD] spi: omap2-mcspi: Trivial optimisation
> ---
> + [FETCH_HEAD] spi: omap2-mcspi: Trivial optimisation
> + [FETCH_HEAD^] spi: omap2-mcspi: add support for pm_runtime autosuspend
> + [FETCH_HEAD~2] spi: omap2-mcspi: use devm_* functions
> + [FETCH_HEAD~3] spi: omap2-mcspi: convert to module_platform_driver
> + [FETCH_HEAD~4] spi: omap2-mcspi: make it behave as a module
> + [FETCH_HEAD~5] spi/omap: Remove bus_num usage for instance index
> - [spi/next] Merge branch 'spi' of git://gitorious.org/linus-tree/linus-tree into spi/next
> + [spi/next^2] OMAP : SPI : use devm_* functions
> + [spi/next^2^] spi: omap2-mcspi: convert to module_platform_driver
> + [spi/next^2~2] spi: omap2-mcspi: make it behave as a module
> +++ [origin~189] Linux 3.3
>
> I cannot merge this branch. If I did it would result in 2 commits for
> each of the commits from the original branch. You'll need go back and
> rebase the new commits on top of the old base or on top of my current
> spi/next branch[1]
>
> [1] git://git.secretlab.ca/git/linux-2.6 spi/next
>
> I've already published that branch, so I will not rebase it either to
> remove the original commits. The new commits must go on top.
>
> Next time, *don't* rebase a branch that has been pulled. It should be
> left alone and new commits added on top of it.
Apologies for this time.
Yes will take care henceforth.
> If your in the
> situation where mainline has commits that you need to bring into the
> branch, then *merge* mainline into your branch (preferably at a tagged
> release point), or ask me to merge in mainline to give you a new
> baseline to work from.
>
> One option I do have is to apply only the patches I'm missing to my
> tree, but I'm not going to because I don't actually know if patches
> 1, 2, or 3 have changed since the version that I've posted. It's
> safer for me to get you to rebase back onto the proper base and post
> only the new patches. Your cover letter should state specifically
> which branch the patches apply on top of.
Will send another pull request.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/