Hi Mr. Park, Mr. Kim,

I had a suggestion here from Tomasz about dividing this patch-set into
2 portions:

1) DT related (patches 1-4) for samsung-dt branch.
2) Clocks, Arch data related to Exynos5. (patches 5,6) for
exynos-drm-fixes branch.

Rationale behind this is kgene tree with all 6 patches applied will
have broken drm
exynos4 and incomplete drm exynos5.

I want to know your opinion on this.

regards,
Rahul Sharma

On Tue, Oct 16, 2012 at 3:01 PM, Tomasz Figa <t.f...@samsung.com> wrote:
> Hi Rahul,
>
>
>
> On Tuesday 16 of October 2012 05:00:28 Rahul Sharma wrote:
>
>> This patch set adds the DT based support for Samsung's Exynos5250. It
>
>> adds device tree nodes for hdmi, mixer, hdmiphy and hdmiddc. The name
>
>> of these devices are changed to the one matching with drivers.
>
>> Exynos-drm and exynos hdmi-drm-commmon devices are removed from machine
>
>> init code.
>
>>
>
>> Exynos-drm and exynos hdmi-drm-commmon devices are removed from machine
>
>> init code. Patch set which adds this code is posted to dri-devel list at
>
>> http://comments.gmane.org/gmane.comp.video.dri.devel/75121.
>
>>
>
>> This patchset is based on linux v3.6-rc6, branch v3.7-next/dt-samsung at
>
>> git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
>
>>
>
>> v1:
>
>> - dropped patch for hpd gpio initialisation from machine init.
>
>> - dropped patch for platform device registration.
>
>> - removed platform device registration from non-dt platforms.
>
>>
>
>> v2:
>
>> - removed version information from hdmi, mixer dt nodes.
>
>> - added DT binding documentation for hdmi, mixer, hdmiphy and hdmiddc.
>
>>
>
>> v3:
>
>> - corrected indentations.
>
>> - changed dt node names to name@address format.
>
>>
>
>> Rahul Sharma (6):
>
>> dts: exynos: add device tree support for exynos5 hdmi
>
>> dts: exynos: add device tree support for exynos5 mixer
>
>> dts: exynos: add device tree support for exynos5 hdmiphy
>
>> dts: exynos: add device tree support for exynos5 hdmiddc
>
>> arm: exynos: add clocks for exynos5 hdmi
>
>> arm: exynos: removing exynos-drm device registration from non-dt
>
>> platforms
>
>>
>
>> .../devicetree/bindings/drm/exynos/hdmi.txt | 22
>
>> +++++++++++++++ .../devicetree/bindings/drm/exynos/hdmiddc.txt |
>
>> 12 ++++++++ .../devicetree/bindings/drm/exynos/hdmiphy.txt | 12
>
>> ++++++++ .../devicetree/bindings/drm/exynos/mixer.txt | 15
>
>> ++++++++++ arch/arm/boot/dts/exynos5250-smdk5250.dts | 24
>
>> +++++++++++++++- arch/arm/boot/dts/exynos5250.dtsi |
>
>> 20 +++++++++++++ arch/arm/mach-exynos/Makefile |
>
>> 1 -
>
>> arch/arm/mach-exynos/clock-exynos5.c | 14 ++++++++-
>
>> arch/arm/mach-exynos/dev-drm.c | 29
>
>> -------------------- arch/arm/mach-exynos/include/mach/map.h
>
>> | 2 +
>
>> arch/arm/mach-exynos/mach-exynos5-dt.c | 8 +++++
>
>> arch/arm/mach-exynos/mach-nuri.c | 3 --
>
>> arch/arm/mach-exynos/mach-origen.c | 3 --
>
>> arch/arm/mach-exynos/mach-smdk4x12.c | 3 --
>
>> arch/arm/mach-exynos/mach-smdkv310.c | 3 --
>
>> arch/arm/mach-exynos/mach-universal_c210.c | 3 --
>
>> arch/arm/plat-samsung/include/plat/devs.h | 2 -
>
>> 17 files changed, 126 insertions(+), 50 deletions(-)
>
>> create mode 100644
>
>> Documentation/devicetree/bindings/drm/exynos/hdmi.txt create mode
>
>> 100644 Documentation/devicetree/bindings/drm/exynos/hdmiddc.txt create
>
>> mode 100644 Documentation/devicetree/bindings/drm/exynos/hdmiphy.txt
>
>> create mode 100644
>
>> Documentation/devicetree/bindings/drm/exynos/mixer.txt delete mode
>
>> 100644 arch/arm/mach-exynos/dev-drm.c
>
>
>
> The patches look fine, but Kukjin's tree doesn't contain all the
>
> dependencies for them to be usable.
>
>
>
> Shouldn't they be based on exynos-drm-next branch of Kyungmin's tree at
>
> infradead instead:
>
>
>
> http://git.infradead.org/users/kmpark/linux-samsung/shortlog/refs/heads/exynos-drm-next
>
>
>
> Best regards,
>
> --
>
> Tomasz Figa
>
> Samsung Poland R&D 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

Reply via email to