Message ID | 20201104140030.6853-1-mika.westerberg@linux.intel.com |
---|---|
Headers | show |
Series | thunderbolt: Add DMA traffic test driver | expand |
On Wed, Nov 4, 2020 at 4:00 PM Mika Westerberg <mika.westerberg@linux.intel.com> wrote: > > +#define DMA_TEST_DATA_PATTERN 0x0123456789abcdefLL Have you considered making it configurable? For mem test, for example, there is a reason to try different patterns. Not sure if it's relevant here.
On Wed, Nov 04, 2020 at 08:38:10PM +0200, Yehezkel Bernat wrote: > On Wed, Nov 4, 2020 at 4:00 PM Mika Westerberg > <mika.westerberg@linux.intel.com> wrote: > > > > +#define DMA_TEST_DATA_PATTERN 0x0123456789abcdefLL > > Have you considered making it configurable? For mem test, for example, there is > a reason to try different patterns. Not sure if it's relevant here. Yes, we did but we went for the simple hard-coded for now. This can always be enhanched if needed :)
On Wed, Nov 04, 2020 at 08:39:01PM +0200, Yehezkel Bernat wrote: > On Wed, Nov 4, 2020 at 4:00 PM Mika Westerberg > <mika.westerberg@linux.intel.com> wrote: > > > > Hi all, > > > > This series adds a new Thunderbolt service driver that can be used on > > manufacturing floor to test that each Thunderbolt/USB4 port is functional. > > It can be done either using a special loopback dongle that has RX and TX > > lanes crossed, or by connecting a cable back to the host (for those who > > don't have these dongles). > > > > This takes advantage of the existing XDomain protocol and creates XDomain > > devices for the loops back to the host where the DMA traffic test driver > > can bind to. > > > > The DMA traffic test driver creates a tunnel through the fabric and then > > sends and receives data frames over the tunnel checking for different > > errors. > > For the whole series, > > Acked-by: Yehezkel Bernat <YehezkelShB@gmail.com> Thanks!