Re: [PATCH 3/7] [media] s5p-fimc: Remove to support fimc for S5PC100
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
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
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
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