[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-can
Subject:    Re: am335x: d_can: rx issue
From:       Alexander Stein <alexander.stein () systec-electronic ! com>
Date:       2014-04-30 6:08:59
Message-ID: 2457559.6qzT8616qY () ws-stein
[Download message RAW]

On Tuesday 29 April 2014 19:50:00, Thomas Gleixner wrote:
> On Tue, 29 Apr 2014, Oliver Hartkopp wrote:
> > On 29.04.2014 19:13, Thomas Gleixner wrote:
> > > On Tue, 29 Apr 2014, Oliver Hartkopp wrote:
> > 
> > >> Btw. I'm currently running Daves net-tree here and was forced to define 
> > >> CONFIG_CAN_C_CAN_STRICT_FRAME_ORDERING during 'make oldconfig' ...
> > >>
> > >> Can this Kconfig option probably be removed in the 3.15-rc phase when
> > >> it's not really needed anymore?
> > >>
> > >> See: http://marc.info/?l=linux-can&m=139746476821294&w=2
> > > 
> > > If it's not necessary for any of the C/DCAN incarnations we can
> > > probably drop it and rip the code out.
> > 
> > Hm. I don't know. AFAIK only Alexander and you detected and discussed the
> > frame re-ordering issue of the C_CAN in length.
> > 
> > I put Alexander in CC. When you both don't have any objections it's up to your
> > opinion to remove this (hopefully) obsolete option.
> 
> It's possible in theory and you can "enforce" it by injecting
> artificial delays, but I doubt it's a real world issue.
> 
> Fine with me to remove it along with the accompanying code cruft.

I dont have any objections too removing it.

Regards,
Alexander
-- 
Dipl.-Inf. Alexander Stein

SYS TEC electronic GmbH
Am Windrad 2
08468 Heinsdorfergrund
Tel.: 03765 38600-1156
Fax: 03765 38600-4100
Email: alexander.stein@systec-electronic.com
Website: www.systec-electronic.com
 
Managing Director: Dipl.-Phys. Siegmar Schmidt
Commercial registry: Amtsgericht Chemnitz, HRB 28082

--
To unsubscribe from this list: send the line "unsubscribe linux-can" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]