[PATCH v2] drm/radeon: evergreen_hpd_init()/_fini(): fix HPD IRQ bitset

2016-03-22 Thread Nicolai Stange
Alex Deucher writes: > On Mon, Mar 21, 2016 at 8:10 AM, Nicolai Stange > wrote: >> The values of all but the RADEON_HPD_NONE members of the radeon_hpd_id >> enum transform 1:1 into bit positions within the 'enabled' bitset as >> assembled by evergreen_hpd_init(): >> >> enabled |= 1 << radeon_

[PATCH v2] drm/radeon: evergreen_hpd_init()/_fini(): fix HPD IRQ bitset

2016-03-22 Thread Alex Deucher
On Tue, Mar 22, 2016 at 8:29 AM, Nicolai Stange wrote: > Alex Deucher writes: > >> On Mon, Mar 21, 2016 at 8:10 AM, Nicolai Stange >> wrote: >>> The values of all but the RADEON_HPD_NONE members of the radeon_hpd_id >>> enum transform 1:1 into bit positions within the 'enabled' bitset as >>> as

[PATCH v2] drm/radeon: evergreen_hpd_init()/_fini(): fix HPD IRQ bitset

2016-03-21 Thread Nicolai Stange
The values of all but the RADEON_HPD_NONE members of the radeon_hpd_id enum transform 1:1 into bit positions within the 'enabled' bitset as assembled by evergreen_hpd_init(): enabled |= 1 << radeon_connector->hpd.hpd; However, if ->hpd.hpd happens to equal RADEON_HPD_NONE == 0xff, UBSAN reports

[PATCH v2] drm/radeon: evergreen_hpd_init()/_fini(): fix HPD IRQ bitset

2016-03-21 Thread Alex Deucher
On Mon, Mar 21, 2016 at 8:10 AM, Nicolai Stange wrote: > The values of all but the RADEON_HPD_NONE members of the radeon_hpd_id > enum transform 1:1 into bit positions within the 'enabled' bitset as > assembled by evergreen_hpd_init(): > > enabled |= 1 << radeon_connector->hpd.hpd; > > However,