Hi Alexey, On Tue, 29 Mar 2016 13:22:49 +0300 Alexey Brodkin <Alexey.Brodkin@xxxxxxxxxxxx> wrote: > As a pair to already existing drm_connector_unplug_all() > (which we'll rename in this series to drm_connector_unregister_all()) > we're adding generic implementation of what is already done in some drivers > for registering all connectors. > > After implementation of that new helper we're updating 2 drivers > that used to use it's own implementation: > [1] atmel_hlcdc > [2] rcar_du > > Other drivers still use load() callback and so should be first modified so > their load() gets called from their probe() explicitly. > > Build- and run-tested on yet to be upstreamed ARC PGU (part of AXS10x board). > > Changes v4 -> v5: > * Added missing mutex unlock on a fail path in drm_connector_register_all() > Thanks David for his attention and patience! > > Changes v3 -> v4: > * Based on current drm-intel/topic/drm-misc > It's now on commit 6c87e5c3ec6db052f3744804a517b6fb003906e1 > And since thet new base already has > "drm: Rename drm_connector_unplug_all() to drm_connector_unregister_all()" > this series now only includes 3 subsequent patches. > > * In drm_connector_register_all() fail path which calls unregister_all() > is moved outside of loop&locked section (as suggested by Daniel) > > Changes v2 -> v3: > * Added acks for 1, 3 and 4 patches > * Updated kerneldoc descriptins of both register_ and unregister_all() > * Updated commit messages (mostly spellos and grammar issues) > > Changes v1 -> v2: > * Rename drm_connector_unplug_all() to drm_connector_unregister_all() > * Use drm_for_each_connector() instead of list_for_each_entry() > * Updated kerneldoc for drm_dev_register() > > Cc: Daniel Vetter <daniel@xxxxxxxx> > Cc: David Airlie <airlied@xxxxxxxx> > Cc: Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx> Next time, can you please send all the patches of the series, not only the cover letter. Thanks, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com
