Re: [PATCH v2 09/13] can: pruss CAN driver.

[Subhasish Ghosh]

Hello,

I had a discussion regarding this with Wolfgang:

http://www.mail-archive.com/socketcan-users@xxxxxxxxxxxxxxxx/msg00324.html

The problem here is that we must configure the mailbox ID's and this support 
is not available in the socketCan sub-system.

--------------------------------------------------
From: "Kurt Van Dijck" <kurt.van.dijck@xxxxxx>
Sent: Friday, February 11, 2011 8:36 PM
To: "Subhasish Ghosh" <subhasish@xxxxxxxxxxxxxxxxxxxx>
Cc: <davinci-linux-open-source@xxxxxxxxxxxxxxxxxxxx>; 
<linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>; <m-watkins@xxxxxx>; 
<nsekhar@xxxxxx>; <sachi@xxxxxxxxxxxxxxxxxxxx>; "Wolfgang Grandegger" 
<wg@xxxxxxxxxxxxxx>; "open list:CAN NETWORK DRIVERS" 
<socketcan-core@xxxxxxxxxxxxxxxx>; "open list:CAN NETWORK DRIVERS" 
<netdev@xxxxxxxxxxxxxxx>; "open list" <linux-kernel@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH v2 09/13] can: pruss CAN driver.

> On Fri, Feb 11, 2011 at 08:21:28PM +0530, Subhasish Ghosh wrote:
> > +config DA8XX_PRU_CANID_MBX0
> > + hex "CANID for mailbox 0"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 0
> > + Default value is set to 0x123, change this as required.
> > +
> > +config DA8XX_PRU_CANID_MBX1
> > + hex "CANID for mailbox 1"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 1
> > + Default value is set to 0x123, change this as required.
> > +
> > +config DA8XX_PRU_CANID_MBX2
> > + hex "CANID for mailbox 2"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 2
> > + Default value is set to 0x123, change this as required.
> > +
> > +config DA8XX_PRU_CANID_MBX3
> > + hex "CANID for mailbox 3"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 3
> > + Default value is set to 0x123, change this as required.
> > +
> > +config DA8XX_PRU_CANID_MBX4
> > + hex "CANID for mailbox 4"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 4
> > + Default value is set to 0x123, change this as required.
> > +
> > +config DA8XX_PRU_CANID_MBX5
> > + hex "CANID for mailbox 5"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 5
> > + Default value is set to 0x123, change this as required.
> > +
> > +config DA8XX_PRU_CANID_MBX6
> > + hex "CANID for mailbox 6"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 6
> > + Default value is set to 0x123, change this as required.
> > +
> > +config DA8XX_PRU_CANID_MBX7
> > + hex "CANID for mailbox 7"
> > + depends on CAN_TI_DA8XX_PRU
> > + default "0x123"
> > + ---help---
> > + Enter the CANID for mailbox 7
> > + Default value is set to 0x123, change this as required.
> Why is filling the mailboxes the job of kernel config?
>
> Regards,
> Kurt 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/