Re: [GIT PULL] Samsung devel for v3.3

2012-01-17 Thread Mark Brown
On Mon, Jan 16, 2012 at 10:40:17PM -0800, Olof Johansson wrote:

 The above two don't seem to have been in the branch that was sent by
 Kukjin, it had:

   ARM: EXYNOS: Invert VCLK polarity for framebuffer on ORIGEN
   ARM: S3C64XX: Fix interrupt configuration for PCA935x on Cragganmore
   ARM: S3C64XX: Fix the memory mapped GPIOs on Cragganmore
   ARM: S3C64XX: Remove hsmmc1 from Cragganmore
   ARM: S3C64XX: Remove unconditional power domain disables
   ARM: SAMSUNG: Declare struct platform_device in plat/s3c64xx-spi.h
   ARM: SAMSUNG: dma-ops.h needs mach/dma.h
   ARM: SAMSUNG: Guard against multiple inclusion of plat/dma.h

 I've pulled the branch from him.

I applied all the fixes I was sitting on that were in -next only at the
time I looked; stuff like this is part of the reason why I tend to avoid
topic branches unless they're likely to get merged in multiple places,
too much bookkeeping.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-16 Thread Olof Johansson
Hi,

On Mon, Jan 9, 2012 at 12:26 AM, Mark Brown
broo...@opensource.wolfsonmicro.com wrote:
 On Sun, Jan 08, 2012 at 05:21:02PM -0800, Olof Johansson wrote:

 Mark, Kukjin, can either of you split out the fixes in a separate branch?

 Here's some of them, there were a few others I'm aware of that depended
 on feature updates.  Ideally we'd actually get all the changes that have
 been in -next in.

 The following changes since commit 5f0a6e2d503896062f641639dacfe5055c2f593b:

  Linux 3.2-rc7 (2011-12-23 21:51:06 -0800)

 are available in the git repository at:
  git://git.kernel.org/pub/scm/linux/kernel/git/broonie/misc.git fixes/samsung

 Mark Brown (5):
      ARM: SAMSUNG: Fix GPIO space reservation for S3C64xx platforms
      ARM: S3C64XX: Correct reservation of GPIOs for CPU module on Cragganmore

The above two don't seem to have been in the branch that was sent by
Kukjin, it had:

  ARM: EXYNOS: Invert VCLK polarity for framebuffer on ORIGEN
  ARM: S3C64XX: Fix interrupt configuration for PCA935x on Cragganmore
  ARM: S3C64XX: Fix the memory mapped GPIOs on Cragganmore
  ARM: S3C64XX: Remove hsmmc1 from Cragganmore
  ARM: S3C64XX: Remove unconditional power domain disables
  ARM: SAMSUNG: Declare struct platform_device in plat/s3c64xx-spi.h
  ARM: SAMSUNG: dma-ops.h needs mach/dma.h
  ARM: SAMSUNG: Guard against multiple inclusion of plat/dma.h

I've pulled the branch from him.

Kukjin, please consider picking up the missing two patches above and
send another pull request with those and any other needed fixes.


-Olof
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [GIT PULL] Samsung devel for v3.3

2012-01-16 Thread Kukjin Kim
Olof Johansson wrote:
 
 Hi,
 
Hi Olof,

 On Mon, Jan 9, 2012 at 12:26 AM, Mark Brown
 broo...@opensource.wolfsonmicro.com wrote:
  On Sun, Jan 08, 2012 at 05:21:02PM -0800, Olof Johansson wrote:
 
  Mark, Kukjin, can either of you split out the fixes in a separate
 branch?
 
  Here's some of them, there were a few others I'm aware of that depended
  on feature updates.  Ideally we'd actually get all the changes that have
  been in -next in.
 
  The following changes since commit
 5f0a6e2d503896062f641639dacfe5055c2f593b:
 
   Linux 3.2-rc7 (2011-12-23 21:51:06 -0800)
 
  are available in the git repository at:
   git://git.kernel.org/pub/scm/linux/kernel/git/broonie/misc.git
 fixes/samsung
 
  Mark Brown (5):
       ARM: SAMSUNG: Fix GPIO space reservation for S3C64xx platforms
       ARM: S3C64XX: Correct reservation of GPIOs for CPU module on
 Cragganmore
 
 The above two don't seem to have been in the branch that was sent by
 Kukjin, it had:
 
   ARM: EXYNOS: Invert VCLK polarity for framebuffer on ORIGEN
   ARM: S3C64XX: Fix interrupt configuration for PCA935x on Cragganmore
   ARM: S3C64XX: Fix the memory mapped GPIOs on Cragganmore
   ARM: S3C64XX: Remove hsmmc1 from Cragganmore
   ARM: S3C64XX: Remove unconditional power domain disables
   ARM: SAMSUNG: Declare struct platform_device in plat/s3c64xx-spi.h
   ARM: SAMSUNG: dma-ops.h needs mach/dma.h
   ARM: SAMSUNG: Guard against multiple inclusion of plat/dma.h
 
 I've pulled the branch from him.
 
No, please drop it because above you said are in mainline now.

- commit 4ff13995(ARM: SAMSUNG: Fix GPIO space reservation for S3C64xx
platforms) 
- commit aaed44e1(ARM: S3C64XX: Correct reservation of GPIOs for CPU module
on Cragganmore)

Seems to be merged via arm-soc/next/boards and there was some
mis-communications.
(Refer to the commit 8df97495 (Merge branch 'samsung/board' into
next/boards))

Sorry for bothering.

 Kukjin, please consider picking up the missing two patches above and
 send another pull request with those and any other needed fixes.
 
As I said, please ignore next-samsung-devel-samsung and Mark's
fixes/samsung.
I think, you can go ahead with my previous pull request, samsung-fixes.

If any problems, please let me know.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim kgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-16 Thread Olof Johansson
Hi,

On Mon, Jan 16, 2012 at 11:23 PM, Kukjin Kim kgene@samsung.com wrote:
 Olof Johansson wrote:

 Hi,

 Hi Olof,

 On Mon, Jan 9, 2012 at 12:26 AM, Mark Brown
 broo...@opensource.wolfsonmicro.com wrote:
  On Sun, Jan 08, 2012 at 05:21:02PM -0800, Olof Johansson wrote:
 
  Mark, Kukjin, can either of you split out the fixes in a separate
 branch?
 
  Here's some of them, there were a few others I'm aware of that depended
  on feature updates.  Ideally we'd actually get all the changes that have
  been in -next in.
 
  The following changes since commit
 5f0a6e2d503896062f641639dacfe5055c2f593b:
 
   Linux 3.2-rc7 (2011-12-23 21:51:06 -0800)
 
  are available in the git repository at:
   git://git.kernel.org/pub/scm/linux/kernel/git/broonie/misc.git
 fixes/samsung
 
  Mark Brown (5):
       ARM: SAMSUNG: Fix GPIO space reservation for S3C64xx platforms
       ARM: S3C64XX: Correct reservation of GPIOs for CPU module on
 Cragganmore

 The above two don't seem to have been in the branch that was sent by
 Kukjin, it had:

       ARM: EXYNOS: Invert VCLK polarity for framebuffer on ORIGEN
       ARM: S3C64XX: Fix interrupt configuration for PCA935x on Cragganmore
       ARM: S3C64XX: Fix the memory mapped GPIOs on Cragganmore
       ARM: S3C64XX: Remove hsmmc1 from Cragganmore
       ARM: S3C64XX: Remove unconditional power domain disables
       ARM: SAMSUNG: Declare struct platform_device in plat/s3c64xx-spi.h
       ARM: SAMSUNG: dma-ops.h needs mach/dma.h
       ARM: SAMSUNG: Guard against multiple inclusion of plat/dma.h

 I've pulled the branch from him.

 No, please drop it because above you said are in mainline now.

 - commit 4ff13995(ARM: SAMSUNG: Fix GPIO space reservation for S3C64xx
 platforms)
 - commit aaed44e1(ARM: S3C64XX: Correct reservation of GPIOs for CPU module
 on Cragganmore)

 Seems to be merged via arm-soc/next/boards and there was some
 mis-communications.
 (Refer to the commit 8df97495 (Merge branch 'samsung/board' into
 next/boards))

 Sorry for bothering.

Ok, no worries, thanks for checking.

 Kukjin, please consider picking up the missing two patches above and
 send another pull request with those and any other needed fixes.

 As I said, please ignore next-samsung-devel-samsung and Mark's
 fixes/samsung.
 I think, you can go ahead with my previous pull request, samsung-fixes.

Done, it was in the batch I just sent a request to Linus for.


-Olof
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Russell King - ARM Linux
On Mon, Jan 09, 2012 at 08:11:54AM -0800, Mark Brown wrote:
 On Mon, Jan 09, 2012 at 03:56:34PM +, Arnd Bergmann wrote:
 
  Of course we will take all bug fixes. Just rebase the rest on top of 3.3-rc1
  and we can take it right away for 3.4.
 
 I've sent a pull request earlier in the thread for the main ones I'm
 aware of (which are mostly fixes for things introduced in this cycle) -
 the others I'm aware of are all specific to Cragganmore and given that
 we can't get any of the features for that merged the fixes aren't going
 to have any practical effect until we can manage to get the feature
 stuff merged so it's not really worth the effort.
 
 This really is very disappointing.

No it isn't.

What is really disappointing is the lack of responsive maintainers for the
Samsung stuff.  It took _two_ bloody months to get the Samsung platforms
sorted for the restart changes in spite of reminding, and a last minute
rush over the course of a couple of days (one _in_ the merge window) to
get it properly merged into my tree.

The only reason something happened was because I stuck a #error into
the Samsung code in linux-next and people started reporting that Samsung
had broken.

This is not the worst of it - Nicolas took _three_ months to get a response
from the shmobile maintainers.  I never got a response from the shmobile
maintainers for the restart changes - so congratulations to the shmobile
maintainers, shmobile is now broken.

The alternative was basically Samsung ending up like shmobile is today.
Maybe that's what should have happened to save folk like Arnd such a
horrible job now.

So, I support Arnd's view: the Samsung stuff is just too late.  Even the
restart updates (which is what has caused this) were too late.  Anything
which causes new merge conflicts in the Samsung code is not acceptable at
this point, even if it's a 'fix' patch.  We've wasted far too much time
trying to get Samsung stuff sorted far too late in the cycle.

Let this be an object lesson in what happens if you leave stuff until the
last minute.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Mark Brown
On Tue, Jan 10, 2012 at 09:06:35AM +, Russell King - ARM Linux wrote:
 On Mon, Jan 09, 2012 at 08:11:54AM -0800, Mark Brown wrote:

  This really is very disappointing.

 No it isn't.

I think we're talking at cross purposes here - I'm saying that this
whole situation is disappointing, not a specific decision.

 What is really disappointing is the lack of responsive maintainers for the
 Samsung stuff.  It took _two_ bloody months to get the Samsung platforms

This is pretty much what I'm saying is disappointing - in this case the
whole fact that we're not managing to get stuff actually merged.  It's
very frustrating that we're ending up in a situation where getting
things applied to the maintainer's tree and into -next (which is usually
the end of what you need to do as a patch submitter) isn't enough to
actually get the changes pushed upstream.

 The only reason something happened was because I stuck a #error into
 the Samsung code in linux-next and people started reporting that Samsung
 had broken.

Yeah, me included.  Like I said I'd probably have sent a fix if I'd been
able to figure out what the changes the error referred to were.

 The alternative was basically Samsung ending up like shmobile is today.
 Maybe that's what should have happened to save folk like Arnd such a
 horrible job now.

So, is there anything that people like me who are contributing to rather
than maintaining things can do to help here beyond chasing maintainers?

Generally my process is roughly to monitor what goes into -next and
chase people if things don't make it in there but that's not working
well here as things are appearing in -next.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Nicolas Pitre
On Tue, 10 Jan 2012, Mark Brown wrote:

 So, is there anything that people like me who are contributing to rather
 than maintaining things can do to help here beyond chasing maintainers?
 
 Generally my process is roughly to monitor what goes into -next and
 chase people if things don't make it in there but that's not working
 well here as things are appearing in -next.

Maybe the Samsung maintainer(s) should target early merge into the 
arm-soc tree instead of going straight to linux-next only.  The former 
ends up in the later anyway.


Nicolas
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Mark Brown
On Tue, Jan 10, 2012 at 01:44:54PM -0500, Nicolas Pitre wrote:
 On Tue, 10 Jan 2012, Mark Brown wrote:

  So, is there anything that people like me who are contributing to rather
  than maintaining things can do to help here beyond chasing maintainers?

 Maybe the Samsung maintainer(s) should target early merge into the 
 arm-soc tree instead of going straight to linux-next only.  The former 
 ends up in the later anyway.

That sounds like it'd be helpful overall but it's something that has to
be sorted out at the maintainer level.  I'm guessing there's not really
much that contributors can do here?
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Nicolas Pitre
On Tue, 10 Jan 2012, Mark Brown wrote:

 On Tue, Jan 10, 2012 at 01:44:54PM -0500, Nicolas Pitre wrote:
  On Tue, 10 Jan 2012, Mark Brown wrote:
 
   So, is there anything that people like me who are contributing to rather
   than maintaining things can do to help here beyond chasing maintainers?
 
  Maybe the Samsung maintainer(s) should target early merge into the 
  arm-soc tree instead of going straight to linux-next only.  The former 
  ends up in the later anyway.
 
 That sounds like it'd be helpful overall but it's something that has to
 be sorted out at the maintainer level.  I'm guessing there's not really
 much that contributors can do here?

Maybe if enough contributor pressure builds up something will change...


Nicolas
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Mark Brown
On Tue, Jan 10, 2012 at 08:13:54PM +, Arnd Bergmann wrote:
 On Tuesday 10 January 2012, Mark Brown wrote:

  That sounds like it'd be helpful overall but it's something that has to
  be sorted out at the maintainer level.  I'm guessing there's not really
  much that contributors can do here?

 I think you did everything as good as you could, we just need to routinely
 call for everyone to submit stuff in time. A number of maintainers sent stuff
 after Christmas (which I expected to start the merge window) and were mostly
 lucky because Linus gave us an extra 10 days to sort things out.

I guess it would also be useful to have some way to compare what's in
-next with what's in the arm-soc tree and chase people if that diff gets
big.  I do also wonder if it's worth letting people push stuff to you
more aggressively - right now you seem to be asking people to batch
things up and I wonder if that's making it a it easier for things to end
up dropping on the floor if a time based routine isn't working well for
people.

 end, samsung also did for the most part but not entirely and you were
 unfortunate to be the contributor of the patches that missed out.

It's not just me, I'm just vocal and perhaps more to the point spend a
reasonable amount of time chasing stuff into various trees so want to
figure out if I need to change what I'm doing with that.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Olof Johansson
On Tue, Jan 10, 2012 at 2:37 PM, Mark Brown
broo...@opensource.wolfsonmicro.com wrote:
 On Tue, Jan 10, 2012 at 08:13:54PM +, Arnd Bergmann wrote:
 On Tuesday 10 January 2012, Mark Brown wrote:

  That sounds like it'd be helpful overall but it's something that has to
  be sorted out at the maintainer level.  I'm guessing there's not really
  much that contributors can do here?

 I think you did everything as good as you could, we just need to routinely
 call for everyone to submit stuff in time. A number of maintainers sent stuff
 after Christmas (which I expected to start the merge window) and were mostly
 lucky because Linus gave us an extra 10 days to sort things out.

 I guess it would also be useful to have some way to compare what's in
 -next with what's in the arm-soc tree and chase people if that diff gets
 big.  I do also wonder if it's worth letting people push stuff to you
 more aggressively - right now you seem to be asking people to batch
 things up and I wonder if that's making it a it easier for things to end
 up dropping on the floor if a time based routine isn't working well for
 people.

I actually did just this today based on this discussion, and I'll do
it through the next staging cycle to keep a track of the arm backlog
of how much is sitting in maintainer trees vs what has already been
merged into Russell's tree or arm-soc. We can include these stats in
next rounds last call for patches email to help catch forgotten
branches.

I merged rmk's for-next branch with the arm-soc one, merged that on
top of mainline and diffed arch/arm with what is in linux-next.

Right now, plus or minus some sloppy merge conflict resolutions on my
part, the statistics are:

git diff --stat next/master -- arch/arm
[..]
 113 files changed, 670 insertions(+), 810 deletions(-)

(this diffstat does not include late/* branches since they got
included in the arm-soc side of the diff)

Most of these are rightfully still there; some changes are going
through other trees such as some PCI changes, ASoC, PM, a few fixes
that haven't been sent up to arm-soc yet, etc.  A couple of patches
seem to have been queued in for-3.4 branches a bit early (Stephen
doesn't generally want people to start queueing new stuff until the
merge window is over), but that's just a couple of them.

So, it looks like there's no major backlog left for any specific vendor subtree.

 end, samsung also did for the most part but not entirely and you were
 unfortunate to be the contributor of the patches that missed out.

 It's not just me, I'm just vocal and perhaps more to the point spend a
 reasonable amount of time chasing stuff into various trees so want to
 figure out if I need to change what I'm doing with that.

What I would do myself is that if I hadn't seen the patches land in
the topmost staging tree by -rc6 or -rc7, I would ping the owner of
the tree that the patches are sitting in to make sure they go up.
Hopefully this kind of thing will be a rare scenario.


-Olof
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Kukjin Kim
Russell King - ARM Linux wrote:
 
 On Mon, Jan 09, 2012 at 08:11:54AM -0800, Mark Brown wrote:

[...]

  This really is very disappointing.
 
 No it isn't.
 
 What is really disappointing is the lack of responsive maintainers for the
 Samsung stuff.  It took _two_ bloody months to get the Samsung platforms
 sorted for the restart changes in spite of reminding, and a last minute
 rush over the course of a couple of days (one _in_ the merge window) to
 get it properly merged into my tree.
 
Yeah, I had to do earlier and it's true that many conflicts caused from my
late ARM restart working for Samsung stuff.

[...]

 So, I support Arnd's view: the Samsung stuff is just too late.  Even the
 restart updates (which is what has caused this) were too late.  Anything
 which causes new merge conflicts in the Samsung code is not acceptable at
 this point, even if it's a 'fix' patch.  We've wasted far too much time
 trying to get Samsung stuff sorted far too late in the cycle.
 
 Let this be an object lesson in what happens if you leave stuff until the
 last minute.

I will do more carefully next time and of course not too late.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim kgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [GIT PULL] Samsung devel for v3.3

2012-01-10 Thread Kukjin Kim
Olof Johansson wrote:
 

[...]

 What I would do myself is that if I hadn't seen the patches land in
 the topmost staging tree by -rc6 or -rc7, I would ping the owner of
 the tree that the patches are sitting in to make sure they go up.

Would be helpful to us.

 Hopefully this kind of thing will be a rare scenario.
 
Yes, and Samsung's topic branch will be sent to arm-soc as soon as possible
when it's ready from me.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim kgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-09 Thread Mark Brown
On Sun, Jan 08, 2012 at 05:21:02PM -0800, Olof Johansson wrote:

 Mark, Kukjin, can either of you split out the fixes in a separate branch?

Here's some of them, there were a few others I'm aware of that depended
on feature updates.  Ideally we'd actually get all the changes that have
been in -next in.

The following changes since commit 5f0a6e2d503896062f641639dacfe5055c2f593b:

  Linux 3.2-rc7 (2011-12-23 21:51:06 -0800)

are available in the git repository at:
  git://git.kernel.org/pub/scm/linux/kernel/git/broonie/misc.git fixes/samsung

Mark Brown (5):
  ARM: SAMSUNG: Fix GPIO space reservation for S3C64xx platforms
  ARM: S3C64XX: Correct reservation of GPIOs for CPU module on Cragganmore
  ARM: SAMSUNG: Guard against multiple inclusion of plat/dma.h
  ARM: SAMSUNG: dma-ops.h needs mach/dma.h
  ARM: SAMSUNG: Declare struct platform_device in plat/s3c64xx-spi.h

 arch/arm/mach-s3c64xx/Kconfig|6 +++---
 arch/arm/mach-s3c64xx/include/mach/crag6410.h|3 ++-
 arch/arm/mach-s3c64xx/mach-crag6410.c|2 +-
 arch/arm/plat-samsung/Kconfig|8 
 arch/arm/plat-samsung/include/plat/dma-ops.h |1 +
 arch/arm/plat-samsung/include/plat/dma.h |6 +-
 arch/arm/plat-samsung/include/plat/s3c64xx-spi.h |2 ++
 7 files changed, 22 insertions(+), 6 deletions(-)
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-09 Thread Russell King - ARM Linux
On Sun, Jan 08, 2012 at 08:49:53PM +, Mark Brown wrote:
 On Fri, Jan 06, 2012 at 01:58:29PM -0800, Olof Johansson wrote:
 
   Please pull Samsung devel for v3.3 from:
   ?git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
   next-samsung-devel-samsung
 
  Thanks. I've pulled this into late/devel, based on earlier email from
  Arnd we might end up sending this in towards the end of the merge
  window, but there's also a chance it will wait for 3.4.
 
 That'd be extremely disappointing, especially given that there's some
 bug fixes in there for updates going in this merge window without which
 mainline is going to have problems on s3c64xx (mostly the GPIO stuff).

Here we go again with sucky work practices.

This sucky behaviour has been around for a long time, I've long since
given up complaining about it as it's exactly like talking to a bloody
brick wall.  People just continue mixing development and fixes together.

They then wonder why their fixes don't make it into mainline in a timely
fashion.

Maybe delaying the whole lot will make people change their behaviour:
it's a behaviour that needs to change at the submitters end.  There's
nothing which Olof or Arnd can reasonably do to expedite the fixes.

So, please direct your complaints to the submitter for not having a
work practice which ensures that fixes receive a higher submission
priority.

Note: Linus _has_ taken a copy of linux-next (read the 3.2 release email),
and _is_ checking whether development stuff was in linux-next prior to
the merge window opening.  It would be very unwise to send new development
which wasn't already there.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-09 Thread Arnd Bergmann
On Monday 09 January 2012, Kukjin Kim wrote:
 In my opinion, as you know, it would be better to us if you could send this in
 this merge window.
 Actually, it does not have any dependency with others, in addition, for a long
 time this has been included in linux-next for this
 merge window.

Your patches have caused too much problems already so far, with many complex
merges (multiple files combined into new files in the restart branch, but
modified in multiple other conflicting branches), and I would rather not
see *anything* besides fixes from you for late 3.3 patches. If it hadn't
been for the conflicts I mentioned, everything else from arm-soc could
have been sent last Saturday already, but this way I spent a national
holiday and a weekend day, both during my vacation, mostly trying to
understand what you were trying to do and get it into shape for upstream
submission, giving up in the end.

I don't consider it enough to have patches in linux-next before the merge
window, and I thought I had made it clear enough that everything has to be
in arm-soc before the merge window. I realize that sometimes there is stuff
that gets done last minute and really has to get merged, but since you have
had it in your own tree for so long, that certainly doesn't apply here.

 If this can be missed from sending list to Linus, please kindly let me know.

Sorry, but the chances are pretty slim this time.

 Let me send this to Linus at the end of this merge window.

No, if you do this, I will NAK that pull request.

Of course we will take all bug fixes. Just rebase the rest on top of 3.3-rc1
and we can take it right away for 3.4.

Arnd
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-09 Thread Mark Brown
On Mon, Jan 09, 2012 at 03:56:34PM +, Arnd Bergmann wrote:

 Of course we will take all bug fixes. Just rebase the rest on top of 3.3-rc1
 and we can take it right away for 3.4.

I've sent a pull request earlier in the thread for the main ones I'm
aware of (which are mostly fixes for things introduced in this cycle) -
the others I'm aware of are all specific to Cragganmore and given that
we can't get any of the features for that merged the fixes aren't going
to have any practical effect until we can manage to get the feature
stuff merged so it's not really worth the effort.

This really is very disappointing.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-09 Thread Mark Brown
On Mon, Jan 09, 2012 at 09:58:39AM +, Russell King - ARM Linux wrote:
 On Sun, Jan 08, 2012 at 08:49:53PM +, Mark Brown wrote:

  That'd be extremely disappointing, especially given that there's some
  bug fixes in there for updates going in this merge window without which
  mainline is going to have problems on s3c64xx (mostly the GPIO stuff).

 Here we go again with sucky work practices.

 This sucky behaviour has been around for a long time, I've long since
 given up complaining about it as it's exactly like talking to a bloody
 brick wall.  People just continue mixing development and fixes together.

Actually in this case the issue is slightly different - the fixes are
mostly fixes for issues introduced by other development going in during
this merge window, what's gone wrong is that they've been applied to a
different branch to that which had the problem.  Still an issue of
course, just a different one.

That said there's also an issue if pure development gets delayed - it
makes it harder to do further work based on top of the work that got
delayed, especially if any cross tree issues come into play.

 Note: Linus _has_ taken a copy of linux-next (read the 3.2 release email),
 and _is_ checking whether development stuff was in linux-next prior to
 the merge window opening.  It would be very unwise to send new development
 which wasn't already there.

That's not an issue here, all the stuff that's being discussed is in
-next but not sent to the arm-soc tree.
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-08 Thread Mark Brown
On Fri, Jan 06, 2012 at 01:58:29PM -0800, Olof Johansson wrote:

  Please pull Samsung devel for v3.3 from:
  ?git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
  next-samsung-devel-samsung

 Thanks. I've pulled this into late/devel, based on earlier email from
 Arnd we might end up sending this in towards the end of the merge
 window, but there's also a chance it will wait for 3.4.

That'd be extremely disappointing, especially given that there's some
bug fixes in there for updates going in this merge window without which
mainline is going to have problems on s3c64xx (mostly the GPIO stuff).
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-08 Thread Olof Johansson
On Sun, Jan 8, 2012 at 12:49 PM, Mark Brown
broo...@opensource.wolfsonmicro.com wrote:
 On Fri, Jan 06, 2012 at 01:58:29PM -0800, Olof Johansson wrote:

  Please pull Samsung devel for v3.3 from:
  ?git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
  next-samsung-devel-samsung

 Thanks. I've pulled this into late/devel, based on earlier email from
 Arnd we might end up sending this in towards the end of the merge
 window, but there's also a chance it will wait for 3.4.

 That'd be extremely disappointing, especially given that there's some
 bug fixes in there for updates going in this merge window without which
 mainline is going to have problems on s3c64xx (mostly the GPIO stuff).

Of course bug fixes should go in.

Mark, Kukjin, can either of you split out the fixes in a separate branch?


Thanks,

-Olof
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [GIT PULL] Samsung devel for v3.3

2012-01-08 Thread Olof Johansson
On Sun, Jan 8, 2012 at 5:40 PM, Kukjin Kim kgene@samsung.com wrote:
 Olof Johansson wrote:

 On Sun, Jan 8, 2012 at 12:49 PM, Mark Brown
 broo...@opensource.wolfsonmicro.com wrote:
  On Fri, Jan 06, 2012 at 01:58:29PM -0800, Olof Johansson wrote:
 
   Please pull Samsung devel for v3.3 from:
   ?git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
   next-samsung-devel-samsung
 
  Thanks. I've pulled this into late/devel, based on earlier email from
  Arnd we might end up sending this in towards the end of the merge
  window, but there's also a chance it will wait for 3.4.
 
  That'd be extremely disappointing, especially given that there's some
  bug fixes in there for updates going in this merge window without which
  mainline is going to have problems on s3c64xx (mostly the GPIO stuff).

 Of course bug fixes should go in.

 Mark, Kukjin, can either of you split out the fixes in a separate branch?

 Hi Olof,

 In my opinion, as you know, it would be better to us if you could send this 
 in this merge window.
 Actually, it does not have any dependency with others, in addition, for a 
 long time this has been included in linux-next for this
 merge window.
 If this can be missed from sending list to Linus, please kindly let me know. 
 Let me send this to Linus at the end of this merge
 window.

We've been pretty clear on this: Everything that came in after Arnd's
email (that wasn't obvious fixes) would be queued as late/* and, if
there is time, go in towards the end of the merge window. There's
plenty of time left in the window so right now it looks likely to
happen, especially since the branch lacks dependencies and applies
relatively cleanly.


-Olof
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [GIT PULL] Samsung devel for v3.3

2012-01-07 Thread Kukjin Kim
Olof Johansson wrote:
 
 On Tue, Jan 3, 2012 at 3:33 PM, Kukjin Kim kgene@samsung.com wrote:
  Hi Arnd,
 
  Please pull Samsung devel for v3.3 from:
   git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
  next-samsung-devel-samsung
 
  It is including s3c64xx cpuidle and development for Cragganmore, ORIGEN
 and
  some boards.
 
  If any problems, please let me know.
 
 Thanks. I've pulled this into late/devel, based on earlier email from
 Arnd we might end up sending this in towards the end of the merge
 window, but there's also a chance it will wait for 3.4.
 
OK, but I hope we can see them on 3.3-rc1 and actually, they were queuing in
my devel branch for a long time.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim kgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html