drm_kms_helper cycle detected build error in next

Hi Tomi,

Looks like today's next build fails if omapdrm is enabled as modules with:

depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper
depmod: ERROR: Found 2 modules in dependency cycles!
make: *** [Makefile:1261: _modinst_post] Error 1

Quoted 6 lines

Looks like today's next build fails if omapdrm is enabled as modules with:

depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper
depmod: ERROR: Found 2 modules in dependency cycles!
make: *** [Makefile:1261: _modinst_post] Error 1

Making CONFIG_DRM=y and CONFIG_DRM_KMS_HELPER=y instead of lodable modules makes it build.

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

Quoted 7 lines · Tony Lindgren

Quoted 6 lines

Looks like today's next build fails if omapdrm is enabled as modules with:

depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper
depmod: ERROR: Found 2 modules in dependency cycles!
make: *** [Makefile:1261: _modinst_post] Error 1

Quoted 2 lines

Making CONFIG_DRM=y and CONFIG_DRM_KMS_HELPER=y instead of lodable modules makes it build.

The same happens even after disabling omapdrm, so it's somewhere in the common code.

Interestingly, my depmod crashes when encountering this.

Quoted 15 lines · Tomi Valkeinen

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

Quoted 9 lines · Tony Lindgren

Quoted 6 lines

Looks like today's next build fails if omapdrm is enabled as modules with:

depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper
depmod: ERROR: Found 2 modules in dependency cycles!
make: *** [Makefile:1261: _modinst_post] Error 1

Making CONFIG_DRM=y and CONFIG_DRM_KMS_HELPER=y instead of lodable modules makes it build.

Quoted 4 lines

The same happens even after disabling omapdrm, so it's somewhere in the common code.

Interestingly, my depmod crashes when encountering this.

There is a fix for this in drm-misc, I should probably pull it over.

If I haven't been sent a pull tomorrow, I'll grab it.

Quoted 19 lines · Dave Airlie

Quoted 18 lines · Tomi Valkeinen

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

Quoted 9 lines · Tony Lindgren

Quoted 6 lines

Looks like today's next build fails if omapdrm is enabled as modules with:

depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper
depmod: ERROR: Found 2 modules in dependency cycles!
make: *** [Makefile:1261: _modinst_post] Error 1

Making CONFIG_DRM=y and CONFIG_DRM_KMS_HELPER=y instead of lodable modules makes it build.

The same happens even after disabling omapdrm, so it's somewhere in the common code.

Quoted 5 lines

Quoted 1 line

Interestingly, my depmod crashes when encountering this.

There is a fix for this in drm-misc, I should probably pull it over.

If I haven't been sent a pull tomorrow, I'll grab it.

You're too much ahead for a pull request on your Monday, but you'll get it asap :-)

Quoted 26 lines · Daniel Vetter

Quoted 24 lines · Dave Airlie

Quoted 12 lines · Tomi Valkeinen

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

Quoted 7 lines · Tony Lindgren

Quoted 6 lines

Looks like today's next build fails if omapdrm is enabled as modules with:

depmod: ERROR: Cycle detected: drm_kms_helper -> drm -> drm_kms_helper
depmod: ERROR: Found 2 modules in dependency cycles!
make: *** [Makefile:1261: _modinst_post] Error 1

Quoted 7 lines

Quoted 2 lines

Making CONFIG_DRM=y and CONFIG_DRM_KMS_HELPER=y instead of lodable modules makes it build.

The same happens even after disabling omapdrm, so it's somewhere in the common code.

Interestingly, my depmod crashes when encountering this.

There is a fix for this in drm-misc, I should probably pull it over.

If I haven't been sent a pull tomorrow, I'll grab it.

Quoted 2 lines

You're too much ahead for a pull request on your Monday, but you'll get it asap :-)

Meanwhile, can you guys provide few more clues which patch to pick for the rest of us?

Something like patch subject line, commit id in drm-misc or something along those lines would be nice..

Quoted 6 lines · Tony Lindgren

Meanwhile, can you guys provide few more clues which patch to pick for the rest of us?

Something like patch subject line, commit id in drm-misc or something along those lines would be nice..

I guess linux-next still not yet back to the daily update schedule, that's why you need this? It's been in trees since last week, and now also in drm-next ... Usually no one ever cares about the commit if we fix it right away.

Anyway

commit 512721a14a2a94b48ca0863d48f9829922a816b8

Author: Maarten Lankhorst <[email protected]>
Date:   Fri Oct 13 16:08:53 2017 +0200

drm/drm_of: Move drm_of_panel_bridge_remove_function into header.