Re: [GIT PULL FOR 3.7] Samsung media drivers fixes
Em Thu, 25 Oct 2012 20:59:09 +0200 Sylwester Nawrocki sylvester.nawro...@gmail.com escreveu: On 10/25/2012 06:46 PM, Mauro Carvalho Chehab wrote: Em Thu, 25 Oct 2012 18:01:17 +0200 Sylwester Nawrockis.nawro...@samsung.com escreveu: Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #includelinux/sizes.h (2012-10-10 08:17:16 -0300) are available in the git repository at: git://git.infradead.org/users/kmpark/linux-samsung v4l_fixes_for_v3.7 for you to fetch changes up to df79eb9e19331685e509d62112972b3c35569f0b: s5p-fimc: Fix potential NULL pointer dereference (2012-10-25 16:08:12 +0200) Jesper Juhl (1): s5p-tv: don't include linux/version.h in mixer_video.c Sachin Kamat (5): s5p-mfc: Fix compilation warning exynos-gsc: Fix compilation warning s5p-mfc: Make 'clk_ref' static in s5p_mfc_pm.c s5p-fimc: Make 'fimc_pipeline_s_stream' function static s5p-fimc: Fix potential NULL pointer dereference Shaik Ameer Basha (3): exynos-gsc: change driver compatible string exynos-gsc: fix variable type in gsc_m2m_device_run() s5p-fimc: fix variable type in fimc_device_run() Sylwester Nawrocki (4): s5p-fimc: Don't ignore return value of vb2_queue_init() s5p-csis: Select S5P_SETUP_MIPIPHY s5p-fimc: Add missing new line character s5p-fimc: Fix platform entities registration Only a few of the above seems to be material for -rc: s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago) s5p-fimc: Fix platform entities registration (60 seconds ago) s5p-csis: Select S5P_SETUP_MIPIPHY (60 seconds ago) s5p-fimc: Don't ignore return value of vb2_queue_init() (61 seconds ago) The other ones are warnings/sparse warnings and cleanups. I'll be applying only the 4 above patches for 3.7, adding the other ones for 3.8. Sorry for mixing them up. Except those 4, exynos-gsc: change driver compatible string really needs to go in 3.7. Bootloaders will be supplying an FDT node for this device with compatible string samsung,exynos5-gsc, not samsung,exynos5250-gsc. In case this patch is applied only starting from 3.8, kernels 3.7, where the GScaler driver was first added, will have broken support for this device. Hence this patch should be considered as a real bug fix. For those embedded systems it might not be a big deal, since it rarely happens pure mainline kernel is used for production. But in principle it's better to apply that patch, to avoid mess where different kernels require different compatible string. This would mean an ABI breakage. Ok, added it at my 3.7 sandbox. Regards, Mauro -- To unsubscribe from this list: send the line unsubscribe linux-media 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 FOR 3.7] Samsung media drivers fixes
Em Fri, 26 Oct 2012 12:15:41 +0200 Sylwester Nawrocki s.nawro...@samsung.com escreveu: On 10/25/2012 08:59 PM, Sylwester Nawrocki wrote: On 10/25/2012 06:46 PM, Mauro Carvalho Chehab wrote: Em Thu, 25 Oct 2012 18:01:17 +0200 Sylwester Nawrockis.nawro...@samsung.com escreveu: Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #includelinux/sizes.h (2012-10-10 08:17:16 -0300) are available in the git repository at: git://git.infradead.org/users/kmpark/linux-samsung v4l_fixes_for_v3.7 for you to fetch changes up to df79eb9e19331685e509d62112972b3c35569f0b: s5p-fimc: Fix potential NULL pointer dereference (2012-10-25 16:08:12 +0200) Jesper Juhl (1): s5p-tv: don't include linux/version.h in mixer_video.c Sachin Kamat (5): s5p-mfc: Fix compilation warning exynos-gsc: Fix compilation warning s5p-mfc: Make 'clk_ref' static in s5p_mfc_pm.c s5p-fimc: Make 'fimc_pipeline_s_stream' function static s5p-fimc: Fix potential NULL pointer dereference Shaik Ameer Basha (3): exynos-gsc: change driver compatible string exynos-gsc: fix variable type in gsc_m2m_device_run() s5p-fimc: fix variable type in fimc_device_run() Sylwester Nawrocki (4): s5p-fimc: Don't ignore return value of vb2_queue_init() s5p-csis: Select S5P_SETUP_MIPIPHY s5p-fimc: Add missing new line character s5p-fimc: Fix platform entities registration Only a few of the above seems to be material for -rc: s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago) s5p-fimc: Fix platform entities registration (60 seconds ago) s5p-csis: Select S5P_SETUP_MIPIPHY (60 seconds ago) s5p-fimc: Don't ignore return value of vb2_queue_init() (61 seconds ago) The other ones are warnings/sparse warnings and cleanups. I'll be applying only the 4 above patches for 3.7, adding the other ones for 3.8. I've noticed s5p-fimc: Fix platform entities registration will cause a race condition. Can you please drop entirely these two patches from the 3.7 queue: s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago) s5p-fimc: Fix platform entities registration (60 seconds ago) ? Alternatively I can send a subsequent follow up patch. As I already applied pinchartl patches on my 3.7 sandbox, I prefer if you could send it as a follow up patch. Regards, Mauro -- To unsubscribe from this list: send the line unsubscribe linux-media 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 FOR 3.7] Samsung media drivers fixes
On 10/25/2012 08:59 PM, Sylwester Nawrocki wrote: On 10/25/2012 06:46 PM, Mauro Carvalho Chehab wrote: Em Thu, 25 Oct 2012 18:01:17 +0200 Sylwester Nawrockis.nawro...@samsung.com escreveu: Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #includelinux/sizes.h (2012-10-10 08:17:16 -0300) are available in the git repository at: git://git.infradead.org/users/kmpark/linux-samsung v4l_fixes_for_v3.7 for you to fetch changes up to df79eb9e19331685e509d62112972b3c35569f0b: s5p-fimc: Fix potential NULL pointer dereference (2012-10-25 16:08:12 +0200) Jesper Juhl (1): s5p-tv: don't include linux/version.h in mixer_video.c Sachin Kamat (5): s5p-mfc: Fix compilation warning exynos-gsc: Fix compilation warning s5p-mfc: Make 'clk_ref' static in s5p_mfc_pm.c s5p-fimc: Make 'fimc_pipeline_s_stream' function static s5p-fimc: Fix potential NULL pointer dereference Shaik Ameer Basha (3): exynos-gsc: change driver compatible string exynos-gsc: fix variable type in gsc_m2m_device_run() s5p-fimc: fix variable type in fimc_device_run() Sylwester Nawrocki (4): s5p-fimc: Don't ignore return value of vb2_queue_init() s5p-csis: Select S5P_SETUP_MIPIPHY s5p-fimc: Add missing new line character s5p-fimc: Fix platform entities registration Only a few of the above seems to be material for -rc: s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago) s5p-fimc: Fix platform entities registration (60 seconds ago) s5p-csis: Select S5P_SETUP_MIPIPHY (60 seconds ago) s5p-fimc: Don't ignore return value of vb2_queue_init() (61 seconds ago) The other ones are warnings/sparse warnings and cleanups. I'll be applying only the 4 above patches for 3.7, adding the other ones for 3.8. I've noticed s5p-fimc: Fix platform entities registration will cause a race condition. Can you please drop entirely these two patches from the 3.7 queue: s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago) s5p-fimc: Fix platform entities registration (60 seconds ago) ? Alternatively I can send a subsequent follow up patch. Sorry for mixing them up. Except those 4, exynos-gsc: change driver compatible string really needs to go in 3.7. Bootloaders will be supplying an FDT node for this device with compatible string samsung,exynos5-gsc, not samsung,exynos5250-gsc. In case this patch is applied only starting from 3.8, kernels 3.7, where the GScaler driver was first added, will have broken support for this device. Hence this patch should be considered as a real bug fix. For those embedded systems it might not be a big deal, since it rarely happens pure mainline kernel is used for production. But in principle it's better to apply that patch, to avoid mess where different kernels require different compatible string. This would mean an ABI breakage. -- Regards, Sylwester -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[GIT PULL FOR 3.7] Samsung media drivers fixes
Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #include linux/sizes.h (2012-10-10 08:17:16 -0300) are available in the git repository at: git://git.infradead.org/users/kmpark/linux-samsung v4l_fixes_for_v3.7 for you to fetch changes up to df79eb9e19331685e509d62112972b3c35569f0b: s5p-fimc: Fix potential NULL pointer dereference (2012-10-25 16:08:12 +0200) Jesper Juhl (1): s5p-tv: don't include linux/version.h in mixer_video.c Sachin Kamat (5): s5p-mfc: Fix compilation warning exynos-gsc: Fix compilation warning s5p-mfc: Make 'clk_ref' static in s5p_mfc_pm.c s5p-fimc: Make 'fimc_pipeline_s_stream' function static s5p-fimc: Fix potential NULL pointer dereference Shaik Ameer Basha (3): exynos-gsc: change driver compatible string exynos-gsc: fix variable type in gsc_m2m_device_run() s5p-fimc: fix variable type in fimc_device_run() Sylwester Nawrocki (4): s5p-fimc: Don't ignore return value of vb2_queue_init() s5p-csis: Select S5P_SETUP_MIPIPHY s5p-fimc: Add missing new line character s5p-fimc: Fix platform entities registration drivers/media/platform/exynos-gsc/gsc-core.c |8 +++-- drivers/media/platform/exynos-gsc/gsc-m2m.c|2 +- drivers/media/platform/s5p-fimc/Kconfig|1 + drivers/media/platform/s5p-fimc/fimc-capture.c |4 ++- drivers/media/platform/s5p-fimc/fimc-lite.c|4 ++- drivers/media/platform/s5p-fimc/fimc-m2m.c |2 +- drivers/media/platform/s5p-fimc/fimc-mdevice.c | 45 drivers/media/platform/s5p-mfc/s5p_mfc_enc.c |2 +- drivers/media/platform/s5p-mfc/s5p_mfc_pm.c|2 +- drivers/media/platform/s5p-tv/mixer_video.c|1 - 10 files changed, 38 insertions(+), 33 deletions(-) Thanks, Sylwester -- To unsubscribe from this list: send the line unsubscribe linux-media 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 FOR 3.7] Samsung media drivers fixes
Em Thu, 25 Oct 2012 18:01:17 +0200 Sylwester Nawrocki s.nawro...@samsung.com escreveu: Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #include linux/sizes.h (2012-10-10 08:17:16 -0300) are available in the git repository at: git://git.infradead.org/users/kmpark/linux-samsung v4l_fixes_for_v3.7 for you to fetch changes up to df79eb9e19331685e509d62112972b3c35569f0b: s5p-fimc: Fix potential NULL pointer dereference (2012-10-25 16:08:12 +0200) Jesper Juhl (1): s5p-tv: don't include linux/version.h in mixer_video.c Sachin Kamat (5): s5p-mfc: Fix compilation warning exynos-gsc: Fix compilation warning s5p-mfc: Make 'clk_ref' static in s5p_mfc_pm.c s5p-fimc: Make 'fimc_pipeline_s_stream' function static s5p-fimc: Fix potential NULL pointer dereference Shaik Ameer Basha (3): exynos-gsc: change driver compatible string exynos-gsc: fix variable type in gsc_m2m_device_run() s5p-fimc: fix variable type in fimc_device_run() Sylwester Nawrocki (4): s5p-fimc: Don't ignore return value of vb2_queue_init() s5p-csis: Select S5P_SETUP_MIPIPHY s5p-fimc: Add missing new line character s5p-fimc: Fix platform entities registration Only a few of the above seems to be material for -rc: s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago) s5p-fimc: Fix platform entities registration (60 seconds ago) s5p-csis: Select S5P_SETUP_MIPIPHY (60 seconds ago) s5p-fimc: Don't ignore return value of vb2_queue_init() (61 seconds ago) The other ones are warnings/sparse warnings and cleanups. I'll be applying only the 4 above patches for 3.7, adding the other ones for 3.8. Regards, Mauro -- To unsubscribe from this list: send the line unsubscribe linux-media 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 FOR 3.7] Samsung media drivers fixes
On 10/25/2012 06:46 PM, Mauro Carvalho Chehab wrote: Em Thu, 25 Oct 2012 18:01:17 +0200 Sylwester Nawrockis.nawro...@samsung.com escreveu: Hi Mauro, please pull following fixes for v3.7-rc. The following changes since commit 1fdead8ad31d3aa833bc37739273fcde89ace93c: [media] m5mols: Add missing #includelinux/sizes.h (2012-10-10 08:17:16 -0300) are available in the git repository at: git://git.infradead.org/users/kmpark/linux-samsung v4l_fixes_for_v3.7 for you to fetch changes up to df79eb9e19331685e509d62112972b3c35569f0b: s5p-fimc: Fix potential NULL pointer dereference (2012-10-25 16:08:12 +0200) Jesper Juhl (1): s5p-tv: don't include linux/version.h in mixer_video.c Sachin Kamat (5): s5p-mfc: Fix compilation warning exynos-gsc: Fix compilation warning s5p-mfc: Make 'clk_ref' static in s5p_mfc_pm.c s5p-fimc: Make 'fimc_pipeline_s_stream' function static s5p-fimc: Fix potential NULL pointer dereference Shaik Ameer Basha (3): exynos-gsc: change driver compatible string exynos-gsc: fix variable type in gsc_m2m_device_run() s5p-fimc: fix variable type in fimc_device_run() Sylwester Nawrocki (4): s5p-fimc: Don't ignore return value of vb2_queue_init() s5p-csis: Select S5P_SETUP_MIPIPHY s5p-fimc: Add missing new line character s5p-fimc: Fix platform entities registration Only a few of the above seems to be material for -rc: s5p-fimc: Fix potential NULL pointer dereference (59 seconds ago) s5p-fimc: Fix platform entities registration (60 seconds ago) s5p-csis: Select S5P_SETUP_MIPIPHY (60 seconds ago) s5p-fimc: Don't ignore return value of vb2_queue_init() (61 seconds ago) The other ones are warnings/sparse warnings and cleanups. I'll be applying only the 4 above patches for 3.7, adding the other ones for 3.8. Sorry for mixing them up. Except those 4, exynos-gsc: change driver compatible string really needs to go in 3.7. Bootloaders will be supplying an FDT node for this device with compatible string samsung,exynos5-gsc, not samsung,exynos5250-gsc. In case this patch is applied only starting from 3.8, kernels 3.7, where the GScaler driver was first added, will have broken support for this device. Hence this patch should be considered as a real bug fix. For those embedded systems it might not be a big deal, since it rarely happens pure mainline kernel is used for production. But in principle it's better to apply that patch, to avoid mess where different kernels require different compatible string. This would mean an ABI breakage. -- Thanks, Sylwester -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html