Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Stefan Agner
On 2015-12-09 21:55, Bhuvanchandra DV wrote: > DSPI instances in Vybrid have a different amount of chip selects > and CTARs (Clock and transfer Attributes Register). In case of > DSPI1 we only have 2 CTAR registers and 4 CS. In present driver > implementation CTAR offset is derived from CS

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Stefan Agner
On 2015-12-10 01:06, Alexander Stein wrote: > On Thursday 10 December 2015 14:14:11, Bhuvanchandra DV wrote: >> On 12/10/2015 12:45 PM, Alexander Stein wrote: >> > On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: >> >> DSPI instances in Vybrid have a different amount of chip selects

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Alexander Stein
On Thursday 10 December 2015 14:14:11, Bhuvanchandra DV wrote: > On 12/10/2015 12:45 PM, Alexander Stein wrote: > > On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: > >> DSPI instances in Vybrid have a different amount of chip selects > >> and CTARs (Clock and transfer Attributes

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Bhuvanchandra DV
On 12/10/2015 12:45 PM, Alexander Stein wrote: On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: DSPI instances in Vybrid have a different amount of chip selects and CTARs (Clock and transfer Attributes Register). In case of DSPI1 we only have 2 CTAR registers and 4 CS. In present

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Bhuvanchandra DV
On 12/10/2015 12:45 PM, Alexander Stein wrote: On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: DSPI instances in Vybrid have a different amount of chip selects and CTARs (Clock and transfer Attributes Register). In case of DSPI1 we only have 2 CTAR registers and 4 CS. In present

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Alexander Stein
On Thursday 10 December 2015 14:14:11, Bhuvanchandra DV wrote: > On 12/10/2015 12:45 PM, Alexander Stein wrote: > > On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: > >> DSPI instances in Vybrid have a different amount of chip selects > >> and CTARs (Clock and transfer Attributes

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Stefan Agner
On 2015-12-09 21:55, Bhuvanchandra DV wrote: > DSPI instances in Vybrid have a different amount of chip selects > and CTARs (Clock and transfer Attributes Register). In case of > DSPI1 we only have 2 CTAR registers and 4 CS. In present driver > implementation CTAR offset is derived from CS

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-10 Thread Stefan Agner
On 2015-12-10 01:06, Alexander Stein wrote: > On Thursday 10 December 2015 14:14:11, Bhuvanchandra DV wrote: >> On 12/10/2015 12:45 PM, Alexander Stein wrote: >> > On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: >> >> DSPI instances in Vybrid have a different amount of chip selects

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-09 Thread Alexander Stein
On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: > DSPI instances in Vybrid have a different amount of chip selects > and CTARs (Clock and transfer Attributes Register). In case of > DSPI1 we only have 2 CTAR registers and 4 CS. In present driver > implementation CTAR offset is

[PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-09 Thread Bhuvanchandra DV
DSPI instances in Vybrid have a different amount of chip selects and CTARs (Clock and transfer Attributes Register). In case of DSPI1 we only have 2 CTAR registers and 4 CS. In present driver implementation CTAR offset is derived from CS instance which will lead to out of bound access if chip

[PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-09 Thread Bhuvanchandra DV
DSPI instances in Vybrid have a different amount of chip selects and CTARs (Clock and transfer Attributes Register). In case of DSPI1 we only have 2 CTAR registers and 4 CS. In present driver implementation CTAR offset is derived from CS instance which will lead to out of bound access if chip

Re: [PATCH v2] spi-fsl-dspi: Fix CTAR Register access

2015-12-09 Thread Alexander Stein
On Thursday 10 December 2015 11:25:30, Bhuvanchandra DV wrote: > DSPI instances in Vybrid have a different amount of chip selects > and CTARs (Clock and transfer Attributes Register). In case of > DSPI1 we only have 2 CTAR registers and 4 CS. In present driver > implementation CTAR offset is