linux-kernel - Re: [PATCH v2 1/2] Move ralink-gdma to its own directory

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CACmrr9hiU2rwncd9sTrh0WVznybJaTP_JMXCaNMzOKDUTJwGEQ@mail.gmail.com>
Date:   Thu, 10 Jan 2019 19:13:29 -0700
From:   George Hilliard <thirtythreeforty@...il.com>
To:     NeilBrown <neil@...wn.name>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-kernel@...r.kernel.org, devel@...verdev.osuosl.org
Subject: Re: [PATCH v2 1/2] Move ralink-gdma to its own directory

On Thu, Jan 10, 2019, 6:21 PM NeilBrown <neil@...wn.name wrote:
>
> On Thu, Jan 10 2019, thirtythreeforty@...il.com wrote:
>
> > From: George Hilliard <thirtythreeforty@...il.com>
> >
> > This is in preparation to allow it and the mt7621-dma drivers to be
> > built separately.  They are completely independent pieces of software,
> > and the Kconfig specifies very different requirements.
> >
> > Cc: linux-kernel@...r.kernel.org
> > Cc: devel@...verdev.osuosl.org
> > Cc: Neil Brown <neil@...wn.name>
> > Signed-off-by: George Hilliard <thirtythreeforty@...il.com>
>
> Hi,
>  thanks for taking an interest in these drivers.
>  I original submitted this code because I though I needed it for my
>  mt7621 hardware, but I've subsequently realized that neither of these
>  dma drivers are used in this hardware.
>  Consequently I cannot test any changes you make.
>  But maybe you can - which would be excellent!
>
>  So this is just letting you and Greg know that despite my stated
>  interest, I cannot actually review or test this.
>
> Thanks,
> NeilBrown
>
>

Thanks for the heads up. Honestly I am not sure to what extent I can
test code changes either, at least with the DMA driver. I'm working
with the MT7688, and official docs for it and its cousin the MT7628
are pretty sparse, so I'm currently not even certain that the ralink
gdma driver works for my SoC.

Onion lists these drivers in their OpenWRT device trees' compatible
strings, so they're related. The SPI driver works out of the box at
least. The MMC driver wants to work but needs debugging. All the
drivers need to be better documented. etc. I hope I can improve them,
and I'll make sure I test any actual kernel code I change!

George