lkml.org 
[lkml]   [2014]   [Apr]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH v2 1/2] SPI: Add driver for Cadence SPI controller
    From
    Hi Mark

    On Fri, Apr 4, 2014 at 3:13 AM, Mark Brown <broonie@kernel.org> wrote:
    > On Thu, Apr 03, 2014 at 04:40:30PM +0530, Harini Katakam wrote:
    >> Add driver for Cadence SPI controller. This is used in Xilinx Zynq.
    >
    > I just reviewed a driver for "Zynq Quad SPI controller" from Punnaiah
    > Choudary Kalluri (CCed) which seems *very* similar to this one. Are
    > there opportunities for code sharing here (I'm not entirely sure the
    > hardware blocks are different, though I didn't check in detail).
    >

    Thanks for the review.

    QSPI is a Xilinx IP built on top of cadence SPI with
    considerable functional changes.
    As explained in the QSPI patch, there are three configurations
    QSPI supports :

    - A single flash device connected with 1 CS and 4 IO lines
    - Two flash devices connected over two separate sets of 4 IO lines
    and two CS lines which are driven together.
    - Two flash devices connected with two separate CS line and one
    common set of 4 IO lines.

    This first set of QSPI patches is only for the single flash configuration.
    As the next two configurations follow, QSPI driver will differ from SPI
    even more. That's why it might be better to have two separate drivers.
    It will avoid a lot of "if spi/ if qspi" checks.

    I will send an RFC with proposed changes for all QSPI configurations.

    Also, I've replied to your comments on the QSPI driver.
    (The QSPI driver already addresses the comments for SPI v1)
    Except in two places where the comment was only applicable to QSPI driver,
    the replies hold good for both SPI and QSPI drivers.
    If you would like to continue the discussion on that thread, I'm ok with it.
    FYI, I'll be sending the next versions for both drivers after further
    discussion concludes.

    Regards,
    Harini


    \
     
     \ /
      Last update: 2014-04-04 06:21    [W:4.082 / U:0.208 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site