lkml.org 
[lkml]   [2011]   [Oct]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [PATCH 02/15] ATA : vortex86 : fix vortex86dx/sx hardware CRC bug.
    From
    Hi,

    When I developed this patch for the Vortex86dx. I recall trying many
    combinations of udma and mwdma. At the time I developed it, I came
    to the conclusion that both udma and mwdma both failed with CRC
    errors being reported. I am not readily set-up to redo the testing to
    see if mwdma is indeed also affected by the errata. It will be a couple
    of weeks before I can re-test this patch on the hardware with mwdma
    turned on.

    Thanks,
    Paul

    On Thu, Oct 27, 2011 at 2:43 AM, Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:
    > On Tue, 25 Oct 2011 23:16:04 -0500
    > Paul Schilling <paul.s.schilling@gmail.com> wrote:
    >
    >> This fixes a DMA issue related to a CRC bug on
    >> the RDC pata peripherial found on the vortex86sx and vortex86dx.
    >
    > What is the RDC erratum reference for this ? Also it seems to make no
    > sense
    >
    >
    >>       static const struct ata_port_info info_rdc_11 = {
    >>               .flags = ATA_FLAG_SLAVE_POSS,
    >>               .pio_mask = ATA_PIO4,
    >> -             .mwdma_mask = ATA_MWDMA2,
    >> +             .mwdma_mask = 0,
    >> +             .udma_mask = 0,
    >
    > MWDMA is not part of the CRC hardware anyway
    >
    > Are you sure these newer variants are simply not using a different set of
    > timing parameters thus showing bugs on your system ?
    >
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2011-10-31 20:09    [W:0.031 / U:118.024 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site