lkml.org 
[lkml]   [2017]   [Jun]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH linux-next v4 2/4] spi: imx: add selection for iMX53 and iMX6 controller
From
Date
Hello Rob

On 06/13/2017 12:53 AM, Rob Herring wrote:
> On Thu, Jun 08, 2017 at 02:16:01PM +0900, Jiada Wang wrote:
>> ECSPI contorller for iMX53 and iMX6 has few hardware issues
>> comparing to iMX51.
>> The change add possibility to detect which controller is used
>> to apply possible workaround and limitations.
>>
>> Signed-off-by: Jiada Wang <jiada_wang@mentor.com>
>> ---
>> .../devicetree/bindings/spi/fsl-imx-cspi.txt | 1 +
>> drivers/spi/spi-imx.c | 26 ++++++++++++++++++++--
>> 2 files changed, 25 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/spi/fsl-imx-cspi.txt b/Documentation/devicetree/bindings/spi/fsl-imx-cspi.txt
>> index 31b5b21..5bf1396 100644
>> --- a/Documentation/devicetree/bindings/spi/fsl-imx-cspi.txt
>> +++ b/Documentation/devicetree/bindings/spi/fsl-imx-cspi.txt
>> @@ -9,6 +9,7 @@ Required properties:
>> - "fsl,imx31-cspi" for SPI compatible with the one integrated on i.MX31
>> - "fsl,imx35-cspi" for SPI compatible with the one integrated on i.MX35
>> - "fsl,imx51-ecspi" for SPI compatible with the one integrated on i.MX51
>> + - "fsl,imx53-ecspi" for SPI compatible with the one integrated on i.MX53 and later Soc
>> - reg : Offset and length of the register set for the device
>> - interrupts : Should contain CSPI/eCSPI interrupt
>> - cs-gpios : Specifies the gpio pins to be used for chipselects.
>> diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c
>> index 4469121..8e6f339 100644
>> --- a/drivers/spi/spi-imx.c
>> +++ b/drivers/spi/spi-imx.c
>> @@ -63,7 +63,8 @@ enum spi_imx_devtype {
>> IMX27_CSPI,
>> IMX31_CSPI,
>> IMX35_CSPI, /* CSPI on all i.mx except above */
>> - IMX51_ECSPI, /* ECSPI on i.mx51 and later */
>> + IMX51_ECSPI, /* ECSPI on i.mx51 */
>> + IMX53_ECSPI, /* ECSPI on i.mx53 and later */
>
> Looks like i.MX51 and i.MX53 are the same. While the DT should have
> different compatibles (with fallbacks), the driver should map them to
> the same type until there's some difference found.
>
the difference between i.MX51 and i.MX53 is introduced in the
4th patch "spi: imx: Add support for SPI Slave mode" in this patch set

do you think, I need to merge the two patches into one?

Thanks,
Jiada

> Rob
>

\
 
 \ /
  Last update: 2017-06-13 07:28    [W:2.418 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site