Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Wed, Mar 12, 2014 at 03:02:02PM +0100, Linus Walleij wrote: > On Fri, Mar 7, 2014 at 2:46 PM, Jason Cooper wrote: > > On Fri, Mar 07, 2014 at 11:56:11AM +0800, Linus Walleij wrote: > > >> For a single patch? > > > > Well, just the doc change and WARN change. I've already taken the dts > > change into mvebu/dt (for arm-soc). > > I've applied this patch [3/3] to the pinctrl tree. Ok, thanks. > > There should be no need for these two to be in mvebu's for-next, so if > > it's easier for you to pick them, please do so. > > > > I'll drop them from mvebu/pinctrl and update mvebu's for-next right now. > > No, I mean, please apply patch 1/3 and 2/3 to the mvebu tree. > There is no need to sync DTS and pinctrl driver changes. This > patch to the pinctrl driver I want to carry however. Sorry I wasn't clear. patch 2/3 depends on 356ca6ce0aa4 devicetree: bindings: update MVEBU pinctrl binding documentation Which is in the mvebu/pinctrl branch you have merged into your tree. I usually keep the binding documentation changes in the same branch with the code changes so that during a future bisect, the documentation has a better chance of being consistent with the code at the point of failure. It's a detail thing, I know, but I couldn't find a downside to doing it. Well, until now. :-) thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 7, 2014 at 2:46 PM, Jason Cooper wrote: > On Fri, Mar 07, 2014 at 11:56:11AM +0800, Linus Walleij wrote: >> For a single patch? > > Well, just the doc change and WARN change. I've already taken the dts > change into mvebu/dt (for arm-soc). I've applied this patch [3/3] to the pinctrl tree. > There should be no need for these two to be in mvebu's for-next, so if > it's easier for you to pick them, please do so. > > I'll drop them from mvebu/pinctrl and update mvebu's for-next right now. No, I mean, please apply patch 1/3 and 2/3 to the mvebu tree. There is no need to sync DTS and pinctrl driver changes. This patch to the pinctrl driver I want to carry however. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Wed, Mar 12, 2014 at 03:02:02PM +0100, Linus Walleij wrote: On Fri, Mar 7, 2014 at 2:46 PM, Jason Cooper ja...@lakedaemon.net wrote: On Fri, Mar 07, 2014 at 11:56:11AM +0800, Linus Walleij wrote: For a single patch? Well, just the doc change and WARN change. I've already taken the dts change into mvebu/dt (for arm-soc). I've applied this patch [3/3] to the pinctrl tree. Ok, thanks. There should be no need for these two to be in mvebu's for-next, so if it's easier for you to pick them, please do so. I'll drop them from mvebu/pinctrl and update mvebu's for-next right now. No, I mean, please apply patch 1/3 and 2/3 to the mvebu tree. There is no need to sync DTS and pinctrl driver changes. This patch to the pinctrl driver I want to carry however. Sorry I wasn't clear. patch 2/3 depends on 356ca6ce0aa4 devicetree: bindings: update MVEBU pinctrl binding documentation Which is in the mvebu/pinctrl branch you have merged into your tree. I usually keep the binding documentation changes in the same branch with the code changes so that during a future bisect, the documentation has a better chance of being consistent with the code at the point of failure. It's a detail thing, I know, but I couldn't find a downside to doing it. Well, until now. :-) thx, Jason. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 7, 2014 at 2:46 PM, Jason Cooper ja...@lakedaemon.net wrote: On Fri, Mar 07, 2014 at 11:56:11AM +0800, Linus Walleij wrote: For a single patch? Well, just the doc change and WARN change. I've already taken the dts change into mvebu/dt (for arm-soc). I've applied this patch [3/3] to the pinctrl tree. There should be no need for these two to be in mvebu's for-next, so if it's easier for you to pick them, please do so. I'll drop them from mvebu/pinctrl and update mvebu's for-next right now. No, I mean, please apply patch 1/3 and 2/3 to the mvebu tree. There is no need to sync DTS and pinctrl driver changes. This patch to the pinctrl driver I want to carry however. Yours, Linus Walleij -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 07, 2014 at 11:56:11AM +0800, Linus Walleij wrote: > On Fri, Mar 7, 2014 at 11:50 AM, Jason Cooper wrote: > > On Fri, Mar 07, 2014 at 11:48:25AM +0800, Linus Walleij wrote: > >> On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper wrote: > >> > On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: > >> >> Pinctrl will WARN on missing DT resources, which is a little bit too > >> >> noisy. Use dev_warn with FW_BUG instead. > >> >> > >> >> Signed-off-by: Sebastian Hesselbarth > >> >> --- > >> >> Cc: Russell King > >> >> Cc: Jason Cooper > >> >> Cc: Andrew Lunn > >> >> Cc: Gregory Clement > >> >> Cc: Linus Walleij > >> >> Cc: devicet...@vger.kernel.org > >> >> Cc: linux-arm-ker...@lists.infradead.org > >> >> Cc: linux-kernel@vger.kernel.org > >> >> --- > >> >> drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- > >> >> 1 file changed, 2 insertions(+), 1 deletion(-) > >> > > >> > Applied to mvebu/pinctrl > >> > >> So are you carrying this patch, or shall I apply it to my tree? > >> > >> In the former case: Acked-by: Linus Walleij > > > > I'll send you a pull tomorrow. > > For a single patch? Well, just the doc change and WARN change. I've already taken the dts change into mvebu/dt (for arm-soc). There should be no need for these two to be in mvebu's for-next, so if it's easier for you to pick them, please do so. I'll drop them from mvebu/pinctrl and update mvebu's for-next right now. thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 07, 2014 at 11:56:11AM +0800, Linus Walleij wrote: On Fri, Mar 7, 2014 at 11:50 AM, Jason Cooper ja...@lakedaemon.net wrote: On Fri, Mar 07, 2014 at 11:48:25AM +0800, Linus Walleij wrote: On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper ja...@lakedaemon.net wrote: On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: Pinctrl will WARN on missing DT resources, which is a little bit too noisy. Use dev_warn with FW_BUG instead. Signed-off-by: Sebastian Hesselbarth sebastian.hesselba...@gmail.com --- Cc: Russell King li...@arm.linux.org.uk Cc: Jason Cooper ja...@lakedaemon.net Cc: Andrew Lunn and...@lunn.ch Cc: Gregory Clement gregory.clem...@free-electrons.com Cc: Linus Walleij linus.wall...@linaro.org Cc: devicet...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org --- drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Applied to mvebu/pinctrl So are you carrying this patch, or shall I apply it to my tree? In the former case: Acked-by: Linus Walleij linus.wall...@linaro.org I'll send you a pull tomorrow. For a single patch? Well, just the doc change and WARN change. I've already taken the dts change into mvebu/dt (for arm-soc). There should be no need for these two to be in mvebu's for-next, so if it's easier for you to pick them, please do so. I'll drop them from mvebu/pinctrl and update mvebu's for-next right now. thx, Jason. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 7, 2014 at 11:50 AM, Jason Cooper wrote: > On Fri, Mar 07, 2014 at 11:48:25AM +0800, Linus Walleij wrote: >> On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper wrote: >> > On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: >> >> Pinctrl will WARN on missing DT resources, which is a little bit too >> >> noisy. Use dev_warn with FW_BUG instead. >> >> >> >> Signed-off-by: Sebastian Hesselbarth >> >> --- >> >> Cc: Russell King >> >> Cc: Jason Cooper >> >> Cc: Andrew Lunn >> >> Cc: Gregory Clement >> >> Cc: Linus Walleij >> >> Cc: devicet...@vger.kernel.org >> >> Cc: linux-arm-ker...@lists.infradead.org >> >> Cc: linux-kernel@vger.kernel.org >> >> --- >> >> drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- >> >> 1 file changed, 2 insertions(+), 1 deletion(-) >> > >> > Applied to mvebu/pinctrl >> >> So are you carrying this patch, or shall I apply it to my tree? >> >> In the former case: Acked-by: Linus Walleij > > I'll send you a pull tomorrow. For a single patch? Remember I want the DTS changes to go to ARM SoC on some different pull... Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 07, 2014 at 11:48:25AM +0800, Linus Walleij wrote: > On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper wrote: > > On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: > >> Pinctrl will WARN on missing DT resources, which is a little bit too > >> noisy. Use dev_warn with FW_BUG instead. > >> > >> Signed-off-by: Sebastian Hesselbarth > >> --- > >> Cc: Russell King > >> Cc: Jason Cooper > >> Cc: Andrew Lunn > >> Cc: Gregory Clement > >> Cc: Linus Walleij > >> Cc: devicet...@vger.kernel.org > >> Cc: linux-arm-ker...@lists.infradead.org > >> Cc: linux-kernel@vger.kernel.org > >> --- > >> drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- > >> 1 file changed, 2 insertions(+), 1 deletion(-) > > > > Applied to mvebu/pinctrl > > So are you carrying this patch, or shall I apply it to my tree? > > In the former case: Acked-by: Linus Walleij I'll send you a pull tomorrow. thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper wrote: > On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: >> Pinctrl will WARN on missing DT resources, which is a little bit too >> noisy. Use dev_warn with FW_BUG instead. >> >> Signed-off-by: Sebastian Hesselbarth >> --- >> Cc: Russell King >> Cc: Jason Cooper >> Cc: Andrew Lunn >> Cc: Gregory Clement >> Cc: Linus Walleij >> Cc: devicet...@vger.kernel.org >> Cc: linux-arm-ker...@lists.infradead.org >> Cc: linux-kernel@vger.kernel.org >> --- >> drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) > > Applied to mvebu/pinctrl So are you carrying this patch, or shall I apply it to my tree? In the former case: Acked-by: Linus Walleij Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: > Pinctrl will WARN on missing DT resources, which is a little bit too > noisy. Use dev_warn with FW_BUG instead. > > Signed-off-by: Sebastian Hesselbarth > --- > Cc: Russell King > Cc: Jason Cooper > Cc: Andrew Lunn > Cc: Gregory Clement > Cc: Linus Walleij > Cc: devicet...@vger.kernel.org > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > --- > drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Applied to mvebu/pinctrl thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: Pinctrl will WARN on missing DT resources, which is a little bit too noisy. Use dev_warn with FW_BUG instead. Signed-off-by: Sebastian Hesselbarth sebastian.hesselba...@gmail.com --- Cc: Russell King li...@arm.linux.org.uk Cc: Jason Cooper ja...@lakedaemon.net Cc: Andrew Lunn and...@lunn.ch Cc: Gregory Clement gregory.clem...@free-electrons.com Cc: Linus Walleij linus.wall...@linaro.org Cc: devicet...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org --- drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Applied to mvebu/pinctrl thx, Jason. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper ja...@lakedaemon.net wrote: On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: Pinctrl will WARN on missing DT resources, which is a little bit too noisy. Use dev_warn with FW_BUG instead. Signed-off-by: Sebastian Hesselbarth sebastian.hesselba...@gmail.com --- Cc: Russell King li...@arm.linux.org.uk Cc: Jason Cooper ja...@lakedaemon.net Cc: Andrew Lunn and...@lunn.ch Cc: Gregory Clement gregory.clem...@free-electrons.com Cc: Linus Walleij linus.wall...@linaro.org Cc: devicet...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org --- drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Applied to mvebu/pinctrl So are you carrying this patch, or shall I apply it to my tree? In the former case: Acked-by: Linus Walleij linus.wall...@linaro.org Yours, Linus Walleij -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 07, 2014 at 11:48:25AM +0800, Linus Walleij wrote: On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper ja...@lakedaemon.net wrote: On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: Pinctrl will WARN on missing DT resources, which is a little bit too noisy. Use dev_warn with FW_BUG instead. Signed-off-by: Sebastian Hesselbarth sebastian.hesselba...@gmail.com --- Cc: Russell King li...@arm.linux.org.uk Cc: Jason Cooper ja...@lakedaemon.net Cc: Andrew Lunn and...@lunn.ch Cc: Gregory Clement gregory.clem...@free-electrons.com Cc: Linus Walleij linus.wall...@linaro.org Cc: devicet...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org --- drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Applied to mvebu/pinctrl So are you carrying this patch, or shall I apply it to my tree? In the former case: Acked-by: Linus Walleij linus.wall...@linaro.org I'll send you a pull tomorrow. thx, Jason. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 3/3] pinctrl: mvebu: silence WARN to dev_warn
On Fri, Mar 7, 2014 at 11:50 AM, Jason Cooper ja...@lakedaemon.net wrote: On Fri, Mar 07, 2014 at 11:48:25AM +0800, Linus Walleij wrote: On Fri, Mar 7, 2014 at 4:18 AM, Jason Cooper ja...@lakedaemon.net wrote: On Wed, Mar 05, 2014 at 01:03:10AM +0100, Sebastian Hesselbarth wrote: Pinctrl will WARN on missing DT resources, which is a little bit too noisy. Use dev_warn with FW_BUG instead. Signed-off-by: Sebastian Hesselbarth sebastian.hesselba...@gmail.com --- Cc: Russell King li...@arm.linux.org.uk Cc: Jason Cooper ja...@lakedaemon.net Cc: Andrew Lunn and...@lunn.ch Cc: Gregory Clement gregory.clem...@free-electrons.com Cc: Linus Walleij linus.wall...@linaro.org Cc: devicet...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Cc: linux-kernel@vger.kernel.org --- drivers/pinctrl/mvebu/pinctrl-dove.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Applied to mvebu/pinctrl So are you carrying this patch, or shall I apply it to my tree? In the former case: Acked-by: Linus Walleij linus.wall...@linaro.org I'll send you a pull tomorrow. For a single patch? Remember I want the DTS changes to go to ARM SoC on some different pull... Yours, Linus Walleij -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
