Hi Ramesh,

On 03/11/2016 08:14 AM, Ramesh Shanmugasundaram wrote:

As we are fixing this issue in CAN dev.c, I'll remove this check in ndo_open and 
set CAN_CTRLMODE_FD flag in ctrlmode & remove the flag in ctrlmode_supported in 
the next v3 version of the patch.

I posted a V2 version of that fix some minutes ago.
It also adds some documentation and a new variable ctrlmode_static which makes it clear how to specify static/fixed control modes.

Please use the new can_set_static_ctrlmode() helper as suggested in the changes for m_can.c.

A feedback is welcome whether this new fix fits your expectations.

Are there any further comments on v2 patch please?

Besides the stuff I wrote above: No :-)

Best regards,
Oliver

Reply via email to