[RFC PATCH 1/1] drm/exynos: Move platform drivers registration to module init

[Vivek Gautam]

Hi Javier,


On Mon, Nov 24, 2014 at 3:35 PM, Javier Martinez Canillas
<javier.martinez at collabora.co.uk> wrote:
> Hello Ajay,
>
> On 11/21/2014 09:57 PM, Javier Martinez Canillas wrote:
>> On 11/21/2014 06:32 PM, Ajay kumar wrote:
>>> Hi,
>>>
>>> I have rebased my bridge series on top of linux-next.
>>>
>>> This is my git log:
>>> 4b38a6f Revert "Revert "ARM: exynos_defconfig: Enable options for
>>> display panel support""
>>> 6fb39a7 ARM: dts: peach-pit: represent the connection between bridge
>>> and panel using videoport and endpoints
>>> aee649c ARM: dts: snow: represent the connection between bridge and
>>> panel using videoport and endpoints
>>> 5b76d8d drm/bridge: Add i2c based driver for ps8622/ps8625 bridge
>>> 581257f Documentation: bridge: Add documentation for ps8622 DT properties
>>> 178e8b9 Documentation: devicetree: Add vendor prefix for parade
>>> 0ceea75 Documentation: drm: bridge: move to video/bridge
>>> f143e2e drm/bridge: ptn3460: use gpiod interface
>>> 2d5cb9d drm/bridge: ptn3460: probe connector at the end of bridge attach
>>> 32ac563 drm/bridge: ptn3460: support drm_panel
>>> 91c6c30 drm/exynos: dp: support drm_bridge
>>> 7eea7eb drm/bridge: ptn3460: Convert to i2c driver model
>>> 602f343 drm/bridge: make bridge registration independent of drm flow
>>> 14c7143 drm/bridge: do not pass drm_bridge_funcs to drm_bridge_init
>>> 2c01ac4 drm/bridge: ptn3460: Few trivial cleanups
>>> 7415f6c arm: dts: Exynos5: Use pmu_system_controller phandle for dp phy
>>> 28655d1 drm/exynos: Move platform drivers registration to module init
>>> ed6778a Add linux-next specific files for 20141121
>>>
>>> I have attached the rebased patches as well.
>>> I tested it on snow, peach_pit and peach_pi without *clk_ignore_unused*.
>>> Display is totally fine with exynos_defconfig (booting is fine even
>>> with CONFIG_SND_SOC_SNOW=y)
>>>
>>
>> Thanks for forward porting your patches to linux-next. Unfortunately I
>> won't have time to test them until Monday but I wonder why you didn't
>> have the boot issues that we have with next-20141121.
>>
>
> I tested your ToT patches on top of next-20141121, this is my git log:
>
> 93fe3d7 Revert "Revert "ARM: exynos_defconfig: Enable options for display panel support""
> 552f74e ARM: dts: peach-pit: represent the connection between bridge and panel using videoport and endpoints
> dbbc293 ARM: dts: snow: represent the connection between bridge and panel using videoport and endpoints
> d8687f8 drm/bridge: Add i2c based driver for ps8622/ps8625 bridge
> f29a649 Documentation: bridge: Add documentation for ps8622 DT properties
> 6ade887 Documentation: devicetree: Add vendor prefix for parade
> d81c42d Documentation: drm: bridge: move to video/bridge
> 50b9828 drm/bridge: ptn3460: use gpiod interface
> 1274c56 drm/bridge: ptn3460: probe connector at the end of bridge attach
> f3cf063 drm/bridge: ptn3460: support drm_panel
> cab682b drm/exynos: dp: support drm_bridge
> 6e78916 drm/bridge: ptn3460: Convert to i2c driver model
> 93f4b5f drm/bridge: make bridge registration independent of drm flow
> 81a038f drm/bridge: do not pass drm_bridge_funcs to drm_bridge_init
> eb6996e drm/bridge: ptn3460: Few trivial cleanups
> c41fa5d arm: dts: Exynos5: Use pmu_system_controller phandle for dp phy
> 51b2c75 drm/exynos: Move platform drivers registration to module init
> ed6778a Add linux-next specific files for 20141121
>
>> I found that the commit ae43b32 ("ARM: 8202/1: dmaengine: pl330: Add
>> runtime Power Management support v12") had to be reverted in order to
>> boot linux-next.
>>
>
> Display works but I needed to revert the mentioned commit, otherwise
> the boot hangs as reported before. I'm using exynos_defconfig and this
> kernel command line:
>
> console=ttySAC3,115200N8 debug earlyprintk root=/dev/mmcblk1p2 rootwait rw

My git log --oneline gives following git hash
vivek at vivek-linuxpc:~/MAINLINE_KERNEL/linux-next$ git log --oneline

9af3770 arm: dts: Exynos5: Use pmu_system_controller phandle for dp phy
1558298 drm/exynos: Move platform drivers registration to module init
4df404f Revert "Revert "ARM: exynos_defconfig: Enable options for
display panel support""
ed6778a Add linux-next specific files for 20141121
eb052c1 Merge branch 'akpm/master'
9c46812 mm: add strictlimit knob

Also attached my config (which is just made out of exynos_defconfig) :
config_next20141124

The boot arguments used are:
[    0.000000] Kernel command line: root=/dev/mmcblk1p1 rootwait ro
console=ttySAC3,115200 debug earlyprintk init=/linuxrc

Also attached the complete log: kernel-log_next20141124.

I am still not sure how SND_SOC creates a problem for you. I did not
revert any patch (apart from the display config revert)
as seen in my git hash.

So with this setup, my peach-pi boots up fine, display comes up;
CONFIG_SND_SOC is still enabled.


-- 
Best Regards
Vivek Gautam
Samsung R&D Institute, Bangalore
India
-------------- next part --------------
A non-text attachment was scrubbed...
Name: config_next20141124
Type: application/octet-stream
Size: 81025 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20141124/6c761fa1/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kernel-log_next20141124
Type: application/octet-stream
Size: 35935 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20141124/6c761fa1/attachment-0003.obj>