[PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-05 Thread Lee Jones
If a user specifies the use of RC as a capability, they should really be enabling RC Core code. If they do not we WARN() them of this and disable the capability for them. Once we know RC Core code has not been enabled, we can update the user's capabilities and use them as a term of reference for

[PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-05 Thread Lee Jones
If a user specifies the use of RC as a capability, they should really be enabling RC Core code. If they do not we WARN() them of this and disable the capability for them. Once we know RC Core code has not been enabled, we can update the user's capabilities and use them as a term of reference for

Re: [PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-05 Thread Lee Jones
On Tue, 04 Apr 2017, Russell King - ARM Linux wrote: > On Tue, Apr 04, 2017 at 05:10:05PM +0100, Lee Jones wrote: > > @@ -237,7 +241,6 @@ struct cec_adapter *cec_allocate_adapter(const struct > > cec_adap_ops *ops, > > if (!(caps & CEC_CAP_RC)) > > return adap; > > > > -#if

Re: [PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-05 Thread Lee Jones
On Tue, 04 Apr 2017, Russell King - ARM Linux wrote: > On Tue, Apr 04, 2017 at 05:10:05PM +0100, Lee Jones wrote: > > @@ -237,7 +241,6 @@ struct cec_adapter *cec_allocate_adapter(const struct > > cec_adap_ops *ops, > > if (!(caps & CEC_CAP_RC)) > > return adap; > > > > -#if

Re: [PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-04 Thread Russell King - ARM Linux
On Tue, Apr 04, 2017 at 05:10:05PM +0100, Lee Jones wrote: > @@ -237,7 +241,6 @@ struct cec_adapter *cec_allocate_adapter(const struct > cec_adap_ops *ops, > if (!(caps & CEC_CAP_RC)) > return adap; > > -#if IS_REACHABLE(CONFIG_RC_CORE) > /* Prepare the RC input device

Re: [PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-04 Thread Russell King - ARM Linux
On Tue, Apr 04, 2017 at 05:10:05PM +0100, Lee Jones wrote: > @@ -237,7 +241,6 @@ struct cec_adapter *cec_allocate_adapter(const struct > cec_adap_ops *ops, > if (!(caps & CEC_CAP_RC)) > return adap; > > -#if IS_REACHABLE(CONFIG_RC_CORE) > /* Prepare the RC input device

[PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-04 Thread Lee Jones
If a user specifies the use of RC as a capability, they should really be enabling RC Core code. If they do not we WARN() them of this and disable the capability for them. Once we know RC Core code has not been enabled, we can update the user's capabilities and use them as a term of reference for

[PATCH 2/2] [media] cec: Handle RC capability more elegantly

2017-04-04 Thread Lee Jones
If a user specifies the use of RC as a capability, they should really be enabling RC Core code. If they do not we WARN() them of this and disable the capability for them. Once we know RC Core code has not been enabled, we can update the user's capabilities and use them as a term of reference for