[Bug 1823] Add new framebuffer ioctl to switch video modes

[bugzilla-daemon at rtems.org]

https://www.rtems.org/bugzilla/show_bug.cgi?id=1823

--- Comment #13 from Ralf Corsepius <ralf.corsepius at rtems.org> 2011-07-04 02:08:31 CDT ---
(In reply to comment #11)
> (In reply to comment #4)
> > (In reply to comment #2)
> > > > a) CPU/BSP specific ioctls should not be part of cpukit.
> > > 
> > > They already are.
> 
> Yeap.

Again - which?

There should not be any CPU/BSP specific ioctls in CPU/BSP independent parts of
the source tree, only general/universal ioctls.

> > 
> > Which? Except as part of score/cpu, such ioctls would qualify as bugs.
> > 
> > The appropriate place to put such ioctls is somewhere outside of cpukit.
> > 
> 
> I do not follow the logic here. I see an API being defined for use in a generic
> manner that can be mapped to a CPU/BSP specific manner as needed by that
> hardware. I thought this was the whole point in the driver interface.
The point is not to pollute CPU-/BSP independent files with CPU/BSP specific
things.

Things like BSP specific frame-buffer ioctls definitely are not CPU/BSP
independent and therefore do not belong into CPU/BSP-independent files.

Where they actually belong to depends upon various details. Depending on their
level of generality, they may belong into score/cpu, libcpu or libbsp.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.