Re: [PATCH 3/7] [media] s5p-fimc: Remove to support fimc for S5PC100

2011-05-17 Thread Sylwester Nawrocki
On 05/16/2011 09:15 AM, Kukjin Kim wrote:
 Sylwester Nawrocki wrote:
 
 Shouldn't there be some deprecation period for whole s5pc100 support
 removal ?
 
 This is that. But I know, there is no time...
 
 It looks a bit rude to me that suddenly whole support for the SoC is
 vanished.
 
 Hmm...what did you think about my previous comments: 'removing some
 mach-s5ps'?

I would expect one kernel release period of so notice for such a change.
But it doesn't have to be true in this specific case.

 
 I know at least one active user of mainline FIMC driver @ s5pc100.
 
 Do you _really_ want to keep it in later mainline?

I don't personally have interest in maintaining this driver for s5pc100.
It just adds unnecessary complexity to my work and in fact I have limited
possibilities now to test the driver on s5pc100.
But if there are users and it really doesn't cost much to keep the support
for s5pc100 why bother to remove it?  Just to get rid of one mach-* directory ? 

 
 I was not sure we should keep continually 6442 and C100 in mainline when I
 decided to removing 'mach-s5p'.

I really don't care about 6442, but what's the problem with s5pc100 ?
Is it being discontinued ? Or there is little users of it ?

I would much more like to see attempts to consolidate the code rather than
simply removing it.


Regards,
-- 
Sylwester Nawrocki
Samsung Poland RD Center
--
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: [PATCH 0/3] ARM: Removing ARCH_S5P6442

2011-05-17 Thread Pavel Machek
On Thu 2011-05-12 10:08:06, Kukjin Kim wrote:
 This patch remvoes mach-s5p6442 directory for ARCH_S5P6442 in arch/arm.

Why remove it? No users? Broken? ...?

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
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: [PATCH 3/3 v5] v4l: Add v4l2 subdev driver for S5P/EXYNOS4 MIPI-CSI receivers

2011-05-17 Thread Sylwester Nawrocki
Hi Laurent,

On 05/15/2011 11:10 PM, Laurent Pinchart wrote:
 On Sunday 15 May 2011 11:33:14 Sylwester Nawrocki wrote:
 On 05/14/2011 05:29 PM, Laurent Pinchart wrote:
 On Wednesday 11 May 2011 17:17:10 Sylwester Nawrocki wrote:

snip


 I perhaps need to isolate functions out from of s5pcsis_resume/suspend and
 then call that in s_power op and s5pcsis_resume/suspend. Don't really like
 this idea though... It would virtually render pm_runtime_* calls unusable
 in this sub-device driver, those would make sense only in the host driver..
 
 I think using the pm_runtime_* calls make sense, they could replace the subdev
 s_power operation in the long term. We'll need to evaluate that (I'm not sure
 if runtime PM is available on all platforms targeted by V4L2 for instance).

That sounds like a really good direction. It would let us have clear 
standardized
rules for power handling in V4L2. And with new chips of more complicated
topologies the power handling details could be taken care of by the Runtime PM
subsystem. As the runtime PM becomes more granular it appears a good alternative
for simple subdev s_power call.

Unfortunately still only a few architectures support runtime PM at device bus 
level
in the mainline. What I identified is:
 arm/mach-exynos4
 arm/mach-omap1
 arm/mach-omap2
 arm/mach-shmobile

So very few, no x86 here.

As we have I2C, SPI and platform device v4l subdevs ideally those buses should
support Runtime PM.

 I just wanted to put all what is needed to control device's power in the PM
 helpers and then use pm_runtime_* calls where required.

--
Regards,

Sylwester Nawrocki
--
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: [PATCH 3/3 v5] v4l: Add v4l2 subdev driver for S5P/EXYNOS4 MIPI-CSI receivers

2011-05-17 Thread Mark Brown
On Tue, May 17, 2011 at 10:06:23PM +0200, Sylwester Nawrocki wrote:

 As we have I2C, SPI and platform device v4l subdevs ideally those buses should
 support Runtime PM.

They all do so.
--
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