Bhupesh SHARMA wrote:

[...]

>>> +enum bosch_ccan_operating_mode {
>>> +       CCAN_NORMAL_MODE = 0,
>>> +       CCAN_BASIC_MODE,
>>> +       CCAN_LOOPBACK_MODE,
>>> +       CCAN_LOOPBACK_WITH_SILENT_MODE,
>>> +       CCAN_SILENT_MODE
>>> +};
>> I do not see a need for another enumeration o the type. We already have
>> the CAN_CTRLMODE_*.
> 
> Yes. But I don't see CAN_CTRLMODE_* capturing all the operating mode types 
> supported by Bosch CCAN.

which are missing?

Marc
-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Socketcan-core mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/socketcan-core

Reply via email to