Re: [PATCH V2 00/11] enable imx6q_sabrelite sgtl5000 audio support
On Tue, May 22, 2012 at 09:39:21AM +0800, Richard Zhao wrote: On Mon, May 21, 2012 at 03:45:44PM +0100, Mark Brown wrote: For the first patch it wasn't clear if the changes in pinctrl were the actual dependencies or if there were more things needed from the ARM tree, I remembered someone already explained the dependencies. It depends on imx pinctrl work and dummy pinctrl enabled in ARM platform code. It is why Shawn suggest it goes to ARM SoC. There was mention of some changes but only a vauge description of them, I've no idea where they are or anything, initially people had mentioned an already cross-merged point in Linus' tree but apparently that wasn't sufficient due to these arch specific bits and I don't know where those are. We should really have got this resolved in -next... Guys, you really need to think about how you're organising what you're doing more. You need to split your work out into focused lines of development rather than just having a single branch. Except the clean up patch, others are all related to enable audio. Do you mean I supposed to split them in advance? Well, a lot of the time there's no actual dependency (eg, you add a driver and add registration of that device so there's no need for them to go in together since both are perfectly fine without the other) then it is best to merge things separately. If there are dependencies then that needs to be handled differently but it needs communication and ideally the dependencies would be on focused things that can just be pulled in. In the case of things like the devm_clk_get() cleanup if there's any complexity at all then it's usually best just to punt for a release. signature.asc Description: Digital signature
Re: [PATCH V2 00/11] enable imx6q_sabrelite sgtl5000 audio support
Hi Mark, ASoC: imx-audmux: add pinctrl support ASoC: fsl_ssi: convert to use devm_clk_get Linus has sent out pull request for pinctrl. So is that ok for you to pick up the above two patch? Thanks Richard -- To unsubscribe from this list: send the line unsubscribe linux-i2c in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH V2 00/11] enable imx6q_sabrelite sgtl5000 audio support
On Mon, May 21, 2012 at 08:59:31PM +0800, Richard Zhao wrote: ASoC: imx-audmux: add pinctrl support ASoC: fsl_ssi: convert to use devm_clk_get Linus has sent out pull request for pinctrl. So is that ok for you to pick up the above two patch? No, apart from anything else the second patch depends on clkdev which is nothing to do with pinctrl! For the first patch it wasn't clear if the changes in pinctrl were the actual dependencies or if there were more things needed from the ARM tree, and in any case until those patches are actually in the ASoC tree we'll break the build if they get applied. Guys, you really need to think about how you're organising what you're doing more. You need to split your work out into focused lines of development rather than just having a single branch. This patch series contains a whole bunch of different changes (the devm_clk_get() change is as far as I can tell completely unrelated to the rest for example) with unclear dependencies on multiple external trees. You should be splitting unrelated changes out, trying to minimise interdependencies, and clearly identifying the dependencies that are there we can get things applied in a timely fashion. Please resend these patches *after* their dependencies are in mainline. signature.asc Description: Digital signature
Re: [PATCH V2 00/11] enable imx6q_sabrelite sgtl5000 audio support
On Mon, May 21, 2012 at 03:45:44PM +0100, Mark Brown wrote: On Mon, May 21, 2012 at 08:59:31PM +0800, Richard Zhao wrote: ASoC: imx-audmux: add pinctrl support ASoC: fsl_ssi: convert to use devm_clk_get Linus has sent out pull request for pinctrl. So is that ok for you to pick up the above two patch? No, apart from anything else the second patch depends on clkdev which is nothing to do with pinctrl! All right, It's my fault to mix a clean up patch. It may hold until the merge window is closed. For the first patch it wasn't clear if the changes in pinctrl were the actual dependencies or if there were more things needed from the ARM tree, I remembered someone already explained the dependencies. It depends on imx pinctrl work and dummy pinctrl enabled in ARM platform code. It is why Shawn suggest it goes to ARM SoC. and in any case until those patches are actually in the ASoC tree we'll break the build if they get applied. All righ, It's ok, if you think, I have to wait for one more cycle to see the patch on mainline. Guys, you really need to think about how you're organising what you're doing more. You need to split your work out into focused lines of development rather than just having a single branch. Except the clean up patch, others are all related to enable audio. Do you mean I supposed to split them in advance? This patch series contains a whole bunch of different changes (the devm_clk_get() change is as far as I can tell completely unrelated to the rest for example) with unclear dependencies on multiple external trees. You should be splitting unrelated changes out, trying to minimise interdependencies, and clearly identifying the dependencies that are there we can get things applied in a timely fashion. hmm.. I should write more about dependencies in commit message. Please resend these patches *after* their dependencies are in mainline. OK. Thanks Richard -- To unsubscribe from this list: send the line unsubscribe linux-i2c in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html