linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: George Hilliard <thirtythreeforty@gmail.com>
To: NeilBrown <neil@brown.name>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	linux-kernel@vger.kernel.org, devel@driverdev.osuosl.org
Subject: Re: [PATCH v2 1/2] Move ralink-gdma to its own directory
Date: Thu, 10 Jan 2019 19:13:29 -0700	[thread overview]
Message-ID: <CACmrr9hiU2rwncd9sTrh0WVznybJaTP_JMXCaNMzOKDUTJwGEQ@mail.gmail.com> (raw)
In-Reply-To: <874laggf9y.fsf@notabene.neil.brown.name>

On Thu, Jan 10, 2019, 6:21 PM NeilBrown <neil@brown.name wrote:
>
> On Thu, Jan 10 2019, thirtythreeforty@gmail.com wrote:
>
> > From: George Hilliard <thirtythreeforty@gmail.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@vger.kernel.org
> > Cc: devel@driverdev.osuosl.org
> > Cc: Neil Brown <neil@brown.name>
> > Signed-off-by: George Hilliard <thirtythreeforty@gmail.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

      reply	other threads:[~2019-01-11  2:13 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CACmrr9gHucWaWtz07kQms6AsquLKAHbK+_Q0kF=92RGo6HmAFA@mail.gmail.com>
2019-01-11  0:43 ` [PATCH v2 1/2] Move ralink-gdma to its own directory thirtythreeforty
2019-01-11  0:43   ` [PATCH v2 2/2] Mediatek staging: Use individual config flags in Makefile thirtythreeforty
2019-01-15 15:30     ` [PATCH v2 2/2] staging: Mediatek: " Greg Kroah-Hartman
2019-01-15 17:02       ` [PATCH v3] " George Hilliard
2019-01-15 17:20         ` Greg Kroah-Hartman
2019-01-15 17:29           ` [PATCH v4] " George Hilliard
2019-01-15 17:43             ` Greg Kroah-Hartman
2019-01-16  6:48               ` [PATCH v5] " George Hilliard
2019-01-16  4:43             ` [PATCH v4] " kbuild test robot
2019-01-15 17:32           ` [PATCH v3] " George Hilliard
2019-01-11  1:21   ` [PATCH v2 1/2] Move ralink-gdma to its own directory NeilBrown
2019-01-11  2:13     ` George Hilliard [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CACmrr9hiU2rwncd9sTrh0WVznybJaTP_JMXCaNMzOKDUTJwGEQ@mail.gmail.com \
    --to=thirtythreeforty@gmail.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=neil@brown.name \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).