Re: [PATCH V3 0/8] Clockevents: Migrate to ->set_dev_mode()

On 25 May 2014 15:08, Preeti U Murthy preeti@linux.vnet.ibm.com wrote:

So I would suggest the below for this problem:

Add an additional case CLOCK_EVT_MODE_ONESHOT_STOPPED, which returns an error code. Let the 'default' case stay as it is and return success just like it is implemented today.

Its not a good idea to bug all drivers for every new mode that comes in. Let drivers only take care of modes they are concerned with and for rest they should return -ENOSYS.

Returning success from 'default' look like a poor design to me.