From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: Jean Delvare <jdelvare@suse.de>,
linux-remoteproc@vger.kernel.org,
Ohad Ben-Cohen <ohad@wizery.com>,
Dmitry Tarnyagin <dmitry.tarnyagin@lockless.no>,
Suman Anna <s-anna@ti.com>
Subject: Re: remoteproc/ste: Hide config option by default
Date: Fri, 2 Dec 2016 13:56:39 -0800 [thread overview]
Message-ID: <20161202215639.GE9322@tuxbot> (raw)
In-Reply-To: <CACRpkdbVYE3ya+GfQ0+4V4XfdQF548ZOfsEcggDCrAdX-VB4RQ@mail.gmail.com>
On Fri 02 Dec 02:08 PST 2016, Linus Walleij wrote:
> On Thu, Dec 1, 2016 at 9:49 AM, Jean Delvare <jdelvare@suse.de> wrote:
>
> > Back in July 2014 I asked around what was the intended target
> > platform for the STE Modem remoteproc driver, so that I could add the
> > proper hardware dependency to its config option. The answer I got was
> > that there was no known publicly available hardware needing it and it
> > was unlikely that there ever would.
> >
> > Still there were objections to deleting the driver, which I do not
> > really understand, but I do respect. But as there is no point in
> > presenting the config option by default, let's hide it unless
> > build-testing.
> >
> > Signed-off-by: Jean Delvare <jdelvare@suse.de>
> > Cc: Ohad Ben-Cohen <ohad@wizery.com>
> > Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> > Cc: Linus Walleij <linus.walleij@linaro.org>
> > Cc: Dmitry Tarnyagin <dmitry.tarnyagin@lockless.no>
> > Cc: Suman Anna <s-anna@ti.com>
> > ---
> > As suggested by Linus Walleij over 2 years ago, sorry for not
> > following up back then:
> > https://lkml.org/lkml/2014/7/25/218
> >
> > The driver could also be moved to staging/.
>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
>
> Also for moving it to staging, if you want. Or deleting it.
>
If there's no known present or future users of this code I would prefer
that we just drop it - it will stay with us in the git history if
someone wants it back.
I do not want to see it moved to staging, I still need to maintain it
when the internal remoteproc APIs changes.
Regards,
Bjorn
prev parent reply other threads:[~2016-12-02 21:56 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-01 8:49 remoteproc/ste: Hide config option by default Jean Delvare
2016-12-02 10:08 ` Linus Walleij
2016-12-02 21:56 ` Bjorn Andersson [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=20161202215639.GE9322@tuxbot \
--to=bjorn.andersson@linaro.org \
--cc=dmitry.tarnyagin@lockless.no \
--cc=jdelvare@suse.de \
--cc=linus.walleij@linaro.org \
--cc=linux-remoteproc@vger.kernel.org \
--cc=ohad@wizery.com \
--cc=s-anna@ti.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.