On Mon, Dec 10, 2012 at 06:13:59PM +0100, Henrik Rydberg wrote: > > > > +config MOUSE_PS2_CYPRESS > > > > + bool "Cypress PS/2 mouse protocol extension" if EXPERT > > > > > > Why EXPERT here? > > > > > > > + default y > > > > > > Should it really be default y here? > > > > > > This config entry (with phrases "if EXPERT" and "default y") was simply > > cloned from similar devices (e.g. SYNAPTICS, ALPS, LIFEBOOK). > > > > If your preference is that CYPRESS should be disabled by default, I'll > > omit those phrases. Please advise. > > Dmitry, which default do you prefer here?
Given that psmouse is monolithic and can't be [easily] split into separate drivers (because they each protocol has to be actively probed), I believe that allprotocols should be enabled by default. People who really know what they are doing can reduce the size by compiling parts of it out. So copying what other protocols did was right ;) Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
