Hi Benjamin, Just spotted a couple of small bits which, imho, should be fine with squashing with follow up patch(es). On 28 July 2016 at 09:07, Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx> wrote: > +EXPORT_SYMBOL(drm_atomic_helper_normalize_zpos); The symbols is internal only thus the EXPORT_SYMBOL notation isn't needed. Rule of thumb - interfaces used by other modules are declared in include/drm and the symbol(s) should be annotated with EXPORT_SYMBOL* > + > +int drm_plane_atomic_set_zpos_property(struct drm_plane *plane, > + struct drm_plane_state *state, > + struct drm_property *property, > + uint64_t val); > + > +int drm_plane_atomic_get_zpos_property(struct drm_plane *plane, > + const struct drm_plane_state *state, > + struct drm_property *property, > + uint64_t *val); > + Dead code/declarations ? Cheers, Emil -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |