Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Mark Brown
On Thu, Mar 21, 2013 at 01:40:22PM -0700, Joe Perches wrote: > On Thu, 2013-03-21 at 21:37 +0100, Mark Brown wrote: > > you routinely send very trivial > > patches which don't apply cleanly > That's demonstrably false. It's sadly not, at least for the things I get to see - as far as I have been

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Joe Perches
On Thu, 2013-03-21 at 21:37 +0100, Mark Brown wrote: > you routinely send very trivial > patches which don't apply cleanly That's demonstrably false. > people > who submit many patches ought to be examples of how to work smoothly. I'm done with this conversation. -- To unsubscribe from this li

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Mark Brown
On Thu, Mar 21, 2013 at 12:16:45PM -0700, Joe Perches wrote: > The commit isn't in -next so I have no idea what > you actually applied. (nor do I really care btw) The fact that you don't care is kind of the problem here, aside from the extra effort involved the active resistance to change isn't g

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Joe Perches
On Thu, 2013-03-21 at 19:44 +0100, Mark Brown wrote: > On Thu, Mar 21, 2013 at 10:53:13AM -0700, Joe Perches wrote: > > On Thu, 2013-03-21 at 18:44 +0100, Mark Brown wrote: > > > > That one needs a bit more taste and thought to work out what's > > > appropraite than can be guaranteed easily with a

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Mark Brown
On Thu, Mar 21, 2013 at 10:53:13AM -0700, Joe Perches wrote: > On Thu, 2013-03-21 at 18:44 +0100, Mark Brown wrote: > > That one needs a bit more taste and thought to work out what's > > appropraite than can be guaranteed easily with a script, things like > > working out drive level prefixes for e

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Joe Perches
On Thu, 2013-03-21 at 18:44 +0100, Mark Brown wrote: > On Thu, Mar 21, 2013 at 10:16:15AM -0700, Joe Perches wrote: > > On Thu, 2013-03-21 at 17:25 +0100, Mark Brown wrote: > > > > I don't think you're quite understanding the issues with automation > > > here. Or indeed the desired end result. >

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Mark Brown
On Thu, Mar 21, 2013 at 10:16:15AM -0700, Joe Perches wrote: > On Thu, 2013-03-21 at 17:25 +0100, Mark Brown wrote: > > I don't think you're quite understanding the issues with automation > > here. Or indeed the desired end result. > What issue is that? That one needs a bit more taste and thoug

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-21 Thread Joe Perches
On Thu, 2013-03-21 at 17:25 +0100, Mark Brown wrote: > > I suggest you use a git pre-commit hook to your > > tree and use sed/perl to add a specific prefix > > if it doesn't exist. > > http://codeinthehole.com/writing/tips-for-using-a-git-pre-commit-hook/ > > I don't think you're quite understandi

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-20 Thread Joe Perches
On Wed, 2013-03-20 at 17:36 +0100, Mark Brown wrote: > On Wed, Mar 20, 2013 at 08:22:16AM -0700, Joe Perches wrote: > > On Wed, 2013-03-20 at 10:57 +0100, Mark Brown wrote: > > > > I've applied this but I'm fed up to the back teeth of having to hand > > > edit trivial patches from you because you

Re: [PATCH] sound: max98090: Remove executable bit

2013-03-20 Thread Joe Perches
On Wed, 2013-03-20 at 10:57 +0100, Mark Brown wrote: > On Tue, Mar 19, 2013 at 02:58:43PM -0700, Joe Perches wrote: > > Source files shouldn't have the executable bit set. > > I've applied this but I'm fed up to the back teeth of having to hand > edit trivial patches from you because you ignore th

[PATCH] sound: max98090: Remove executable bit

2013-03-19 Thread Joe Perches
Source files shouldn't have the executable bit set. Signed-off-by: Joe Perches --- include/sound/max98090.h| 0 sound/soc/codecs/max98090.c | 0 sound/soc/codecs/max98090.h | 0 3 files changed, 0 insertions(+), 0 deletions(-) diff --git a/include/sound/max98090.h b/include/sound/max98090.h