All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <michal.simek@amd.com>
To: Alexander Dahl <post@lespocky.de>
Cc: <u-boot@lists.denx.de>, Alexander Dahl <ada@thorsis.com>,
	Simon Glass <sjg@chromium.org>, Stefan Roese <sr@denx.de>
Subject: Re: [PATCH v2 1/2] fpga: Convert SYS_FPGA_CHECK_CTRLC to Kconfig
Date: Wed, 13 Jul 2022 15:53:18 +0200	[thread overview]
Message-ID: <421072cd-d754-282d-476f-56033514fa90@amd.com> (raw)
In-Reply-To: <20220713132044.GB1941@falbala.internal.home.lespocky.de>



On 7/13/22 15:20, Alexander Dahl wrote:
> Hello Michal,
> 
> On Wed, Jul 13, 2022 at 02:56:08PM +0200, Michal Simek wrote:
>>
>>
>> On 7/13/22 14:32, Alexander Dahl wrote:
>>> From: Alexander Dahl <ada@thorsis.com>
>>>
>>> After commit 8cca60a2cbf2 ("Kconfig: Remove some symbols from the
>>> whitelist") downstream builds failed for boards setting this in
>>> include/configs/…
>>>
>>> Two FPGA drivers consider this definition.
>>
>> 2?
>> board/astro/mcf5373l/fpga.c
>> drivers/fpga/ACEX1K.c
>> drivers/fpga/virtex2.c
> 
> These are one board and two drivers.  What is your question?

The first one looks like spartan3 fpga driver in board folder (incorrect 
location). It means I consider this as 3 fpga drivers not 2.

But at the end of day just remove that sentence and that's it. Everybody sees 
what you have changed.

Thanks,
Michal

  reply	other threads:[~2022-07-13 13:53 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-13 12:32 [PATCH v2 0/2] fpga: Convert some options to Kconfig Alexander Dahl
2022-07-13 12:32 ` [PATCH v2 1/2] fpga: Convert SYS_FPGA_CHECK_CTRLC " Alexander Dahl
2022-07-13 12:56   ` Michal Simek
2022-07-13 13:20     ` Alexander Dahl
2022-07-13 13:53       ` Michal Simek [this message]
2022-07-13 14:07       ` Alexander Dahl
2022-07-13 14:18         ` Michal Simek
2022-07-13 14:19       ` Tom Rini
2022-07-13 12:33 ` [PATCH v2 2/2] fpga: Convert SYS_FPGA_PROG_FEEDBACK " Alexander Dahl
2022-07-13 12:50   ` Michal Simek
2022-07-13 13:11     ` Alexander Dahl
2022-07-13 14:19       ` Tom Rini

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=421072cd-d754-282d-476f-56033514fa90@amd.com \
    --to=michal.simek@amd.com \
    --cc=ada@thorsis.com \
    --cc=post@lespocky.de \
    --cc=sjg@chromium.org \
    --cc=sr@denx.de \
    --cc=u-boot@lists.denx.de \
    /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.