From: "Stefan Riedmüller" <S.Riedmueller@phytec.de>
To: "sha@pengutronix.de" <sha@pengutronix.de>
Cc: "andrej.picej@norik.com" <andrej.picej@norik.com>,
"barebox@lists.infradead.org" <barebox@lists.infradead.org>,
Yunus Bas <Y.Bas@phytec.de>
Subject: Re: [PATCH 2/5] ARM: configs: phytec-som-imx6.config
Date: Wed, 17 Nov 2021 09:59:41 +0000 [thread overview]
Message-ID: <171acb6acc23a8c07a74e2d8fdc2d6e1219d43fe.camel@phytec.de> (raw)
In-Reply-To: <20211117070313.GG6556@pengutronix.de>
Hi Sascha,
On Wed, 2021-11-17 at 08:03 +0100, Sascha Hauer wrote:
> On Mon, Nov 15, 2021 at 02:02:05PM +0100, Andrej Picej wrote:
> > Create PHYTEC specific i.MX6 SoM config fragment, which is intended to
> > be merged with the default imx_v7_defconfig. This way, one can clearly
> > outline the differences between standard defconfig and changes PHYTEC
> > makes for i.MX6 SoMs in their BSP. Nonetheless, at the same time, this
> > minimizes efforts required to port patches to defconfig with each new
> > PHYTEC BSP release.
> >
> > For now:
> > - disable BLSPEC,
> > - disable IPU driver,
>
> Why are these disabled? These options shouldn't hurt anyone.
You're right, there should be no issue with the BLSPEC config.
Regarding the IPU driver, I only remember that we had an issue some time ago
with the IPU driver and since then disable it because we don't make use of it
in our BSP. But I can't remember what the actual issue was.
@Andrej, maybe you can take another look at it and see if it can be enabled
without issues on our boards nowadays. In that case we could drop disabling
the IPU driver as well.
>
> > - enable of_fixup_status command,
> > - enable of_overlay command,
> > - enable of_display_timings command and
> > - enable OCOTP fuse writing ability.
>
> We can enable these in imx_v7_defconfig, no need to overwrite them.
Sure let's move them to the defconfig then.
Regards,
Stefan
>
> Sascha
>
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2021-11-17 10:01 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-15 13:02 [PATCH 0/5] PHYTEC patches to be mainlined Andrej Picej
2021-11-15 13:02 ` [PATCH 1/5] flash-header-phytec-pcl063: Set SOC voltage to 1.25 V during boot Andrej Picej
2021-11-15 13:02 ` [PATCH 2/5] ARM: configs: phytec-som-imx6.config Andrej Picej
2021-11-17 7:03 ` Sascha Hauer
2021-11-17 9:59 ` Stefan Riedmüller [this message]
2021-11-17 10:08 ` Andrej Picej
2021-11-15 13:02 ` [PATCH 3/5] regulator: allow use of dummy regulator Andrej Picej
2021-11-17 7:21 ` Sascha Hauer
2021-11-17 8:11 ` Andrej Picej
2021-11-17 9:29 ` Andrej Picej
2021-11-15 13:02 ` [PATCH 4/5] ARM: dts: imx6qdl: pfla02: allow use of dummy regulators Andrej Picej
2021-11-15 13:02 ` [PATCH 5/5] documentation: regulator: add dummy-supply Andrej Picej
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=171acb6acc23a8c07a74e2d8fdc2d6e1219d43fe.camel@phytec.de \
--to=s.riedmueller@phytec.de \
--cc=Y.Bas@phytec.de \
--cc=andrej.picej@norik.com \
--cc=barebox@lists.infradead.org \
--cc=sha@pengutronix.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox