On Thu, Feb 18, 2016 at 05:53:13PM +0200, Daniel Baluta wrote:
> From: Adriana Reus <[email protected]>
>
> Add a check in i2c_mux_master_xfer before calling the select callback.
> This is necessary so that NULL callbacks can be safely registered.
>
> Signed-off-by: Adriana Reus <[email protected]>
> Signed-off-by: Daniel Baluta <[email protected]>
Hmm, rather than supporting that in the core, I'd prefer to have the
driver pass an empty function instead. Then, in the driver, we can have
a comment explaining the special situation.
On Tue, Mar 1, 2016 at 10:30 PM, Wolfram Sang <[email protected]> wrote:
> On Thu, Feb 18, 2016 at 05:53:13PM +0200, Daniel Baluta wrote:
>> From: Adriana Reus <[email protected]>
>>
>> Add a check in i2c_mux_master_xfer before calling the select callback.
>> This is necessary so that NULL callbacks can be safely registered.
>>
>> Signed-off-by: Adriana Reus <[email protected]>
>> Signed-off-by: Daniel Baluta <[email protected]>
>
> Hmm, rather than supporting that in the core, I'd prefer to have the
> driver pass an empty function instead. Then, in the driver, we can have
> a comment explaining the special situation.
Agree. This seems a better idea forcing the user to explain the situation :).