All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Andre Przywara <andre.przywara@arm.com>
Cc: Arnd Bergmann <arnd@arndb.de>, Maxime Ripard <mripard@kernel.org>,
	Chen-Yu Tsai <wens@csie.org>,
	 Jernej Skrabec <jernej.skrabec@gmail.com>,
	Samuel Holland <samuel@sholland.org>,
	 Rob Herring <robh+dt@kernel.org>,
	Mesih Kilinc <mesihkilinc@gmail.com>,
	 Icenowy Zheng <icenowy@aosc.io>,
	Jesse Taube <mr.bossman075@gmail.com>,
	 Giulio Benetti <giulio.benetti@benettiengineering.com>,
	 George Hilliard <thirtythreeforty@gmail.com>,
	DTML <devicetree@vger.kernel.org>,
	 Linux ARM <linux-arm-kernel@lists.infradead.org>,
	linux-sunxi@lists.linux.dev,
	 Russell King <linux@armlinux.org.uk>,
	Olof Johansson <olof@lixom.net>, SoC Team <soc@kernel.org>
Subject: Re: [PATCH 13/14] ARM: configs: sync multi_v5_defconfig from savedefconfig
Date: Tue, 8 Mar 2022 14:40:39 +0100	[thread overview]
Message-ID: <CAK8P3a3g_zPW5xEFRku3TsZTdovevwyxyWVaN_=eD+FN2G8YLg@mail.gmail.com> (raw)
In-Reply-To: <20220308120750.0bbe0122@donnerap.cambridge.arm.com>

On Tue, Mar 8, 2022 at 1:07 PM Andre Przywara <andre.przywara@arm.com> wrote:
> If you find that useful, I can try to find those dependency chains for the
> other options, but I definitely lack the knowledge about the history of
> those old platforms, so I can't reason about them. But I could present you
> the findings and you can then say what to do?

Back to this point: the most interesting ones are those that broke because of
some added dependency, or an option that got renamed. This is what I would
use to find the root cause in a scripted way (after identifying the last "good"
version):

$ git bisect run bash -c "make -s O=obj-arm ARCH=arm
multi_v5_defconfig && grep CONFIG_DRM_PANEL_SIMPLE  obj-arm/.config"

You can group the ones that went away because of a deleted platform
or driver, or those that are just default-enabled now, but for cases that
no longer use a driver that may be needed by some platform, just
having the information about what patch broke them usually tells us
what happened. We still need to decide on an appropriate action then,
which could be to include the new dependency explicitly, or to remove
the entire platform that used the driver because we find that the maintainers
never noticed it breaking.

         Arnd

WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: Andre Przywara <andre.przywara@arm.com>
Cc: Arnd Bergmann <arnd@arndb.de>, Maxime Ripard <mripard@kernel.org>,
	Chen-Yu Tsai <wens@csie.org>,
	 Jernej Skrabec <jernej.skrabec@gmail.com>,
	Samuel Holland <samuel@sholland.org>,
	Rob Herring <robh+dt@kernel.org>,
	Mesih Kilinc <mesihkilinc@gmail.com>,
	 Icenowy Zheng <icenowy@aosc.io>,
	Jesse Taube <mr.bossman075@gmail.com>,
	 Giulio Benetti <giulio.benetti@benettiengineering.com>,
	 George Hilliard <thirtythreeforty@gmail.com>,
	DTML <devicetree@vger.kernel.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	linux-sunxi@lists.linux.dev,
	 Russell King <linux@armlinux.org.uk>,
	Olof Johansson <olof@lixom.net>, SoC Team <soc@kernel.org>
Subject: Re: [PATCH 13/14] ARM: configs: sync multi_v5_defconfig from savedefconfig
Date: Tue, 8 Mar 2022 14:40:39 +0100	[thread overview]
Message-ID: <CAK8P3a3g_zPW5xEFRku3TsZTdovevwyxyWVaN_=eD+FN2G8YLg@mail.gmail.com> (raw)
In-Reply-To: <20220308120750.0bbe0122@donnerap.cambridge.arm.com>

On Tue, Mar 8, 2022 at 1:07 PM Andre Przywara <andre.przywara@arm.com> wrote:
> If you find that useful, I can try to find those dependency chains for the
> other options, but I definitely lack the knowledge about the history of
> those old platforms, so I can't reason about them. But I could present you
> the findings and you can then say what to do?

Back to this point: the most interesting ones are those that broke because of
some added dependency, or an option that got renamed. This is what I would
use to find the root cause in a scripted way (after identifying the last "good"
version):

$ git bisect run bash -c "make -s O=obj-arm ARCH=arm
multi_v5_defconfig && grep CONFIG_DRM_PANEL_SIMPLE  obj-arm/.config"

You can group the ones that went away because of a deleted platform
or driver, or those that are just default-enabled now, but for cases that
no longer use a driver that may be needed by some platform, just
having the information about what patch broke them usually tells us
what happened. We still need to decide on an appropriate action then,
which could be to include the new dependency explicitly, or to remove
the entire platform that used the driver because we find that the maintainers
never noticed it breaking.

         Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2022-03-08 13:46 UTC|newest]

Thread overview: 94+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-07 14:34 [PATCH 00/14] ARM: suniv: dts: update Allwinner F1C100 Andre Przywara
2022-03-07 14:34 ` Andre Przywara
2022-03-07 14:34 ` [PATCH 01/14] dt-bindings: watchdog: sunxi: fix F1C100s compatible Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-08 16:08   ` Rob Herring
2022-03-08 16:08     ` Rob Herring
2022-03-09 23:02   ` Guenter Roeck
2022-03-09 23:02     ` Guenter Roeck
2022-03-10  0:46   ` Samuel Holland
2022-03-10  0:46     ` Samuel Holland
2022-03-14 17:39     ` Andre Przywara
2022-03-14 17:39       ` Andre Przywara
2022-03-07 14:34 ` [PATCH 02/14] ARM: dts: suniv: F1C100: fix watchdog compatible Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-09 23:03   ` Guenter Roeck
2022-03-09 23:03     ` Guenter Roeck
2022-03-07 14:34 ` [PATCH 03/14] dt-bindings: arm: sunxi: document LicheePi Nano name Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-08 16:09   ` Rob Herring
2022-03-08 16:09     ` Rob Herring
2022-03-11  1:27   ` Samuel Holland
2022-03-11  1:27     ` Samuel Holland
2022-03-07 14:34 ` [PATCH 04/14] ARM: dts: suniv: F1C100: add clock and reset macros Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-11  1:30   ` Samuel Holland
2022-03-11  1:30     ` Samuel Holland
2022-03-07 14:34 ` [PATCH 05/14] ARM: dts: suniv: F1C100: fix CPU node Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-08  2:44   ` Jesse Taube
2022-03-08  2:44     ` Jesse Taube
2022-03-08  4:23     ` Icenowy Zheng
2022-03-08  4:23       ` Icenowy Zheng
2022-03-08 10:42     ` Andre Przywara
2022-03-08 10:42       ` Andre Przywara
2022-03-07 14:34 ` [PATCH 06/14] ARM: dts: suniv: F1C100: fix timer node Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-11  2:19   ` Samuel Holland
2022-03-11  2:19     ` Samuel Holland
2022-03-07 14:34 ` [PATCH 07/14] dt-bindings: mmc: sunxi: add Allwinner F1c100s compatible Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-08 16:10   ` Rob Herring
2022-03-08 16:10     ` Rob Herring
2022-03-11  2:19   ` Samuel Holland
2022-03-11  2:19     ` Samuel Holland
2022-03-11 15:41   ` Ulf Hansson
2022-03-11 15:41     ` Ulf Hansson
2022-03-07 14:34 ` [PATCH 08/14] ARM: dts: suniv: F1C100: add MMC controllers Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-11  2:19   ` Samuel Holland
2022-03-11  2:19     ` Samuel Holland
2022-03-07 14:34 ` [PATCH 09/14] ARM: dts: suniv: licheepi-nano: add microSD card Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-11  2:19   ` Samuel Holland
2022-03-11  2:19     ` Samuel Holland
2022-03-07 14:34 ` [PATCH 10/14] dt-bindings: spi: sunxi: document F1C100 controllers Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-08 16:10   ` Rob Herring
2022-03-08 16:10     ` Rob Herring
2022-03-11  2:19   ` Samuel Holland
2022-03-11  2:19     ` Samuel Holland
2022-03-07 14:34 ` [PATCH 11/14] ARM: dts: suniv: F1C100: add SPI support Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-11  2:19   ` Samuel Holland
2022-03-11  2:19     ` Samuel Holland
2022-03-11 13:33     ` Andre Przywara
2022-03-11 13:33       ` Andre Przywara
2022-03-07 14:34 ` [PATCH 12/14] ARM: dts: suniv: licheepi-nano: add SPI flash Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-11  2:20   ` Samuel Holland
2022-03-11  2:20     ` Samuel Holland
2022-03-07 14:34 ` [PATCH 13/14] ARM: configs: sync multi_v5_defconfig from savedefconfig Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-08  9:38   ` Arnd Bergmann
2022-03-08  9:38     ` Arnd Bergmann
2022-03-08 12:07     ` Andre Przywara
2022-03-08 12:07       ` Andre Przywara
2022-03-08 13:33       ` Arnd Bergmann
2022-03-08 13:33         ` Arnd Bergmann
2022-03-08 13:40       ` Arnd Bergmann [this message]
2022-03-08 13:40         ` Arnd Bergmann
2022-03-08 14:30       ` Nicolas Ferre
2022-03-08 14:30         ` Nicolas Ferre
2022-03-08 15:17         ` Arnd Bergmann
2022-03-08 15:17           ` Arnd Bergmann
2022-03-10 10:33     ` Andre Przywara
2022-03-10 10:33       ` Andre Przywara
2022-03-07 14:34 ` [PATCH 14/14] ARM: configs: multi_v5: Enable Allwinner F1C100 Andre Przywara
2022-03-07 14:34   ` Andre Przywara
2022-03-07 18:03 ` [PATCH 00/14] ARM: suniv: dts: update " Jesse Taube
2022-03-07 18:03   ` Jesse Taube
2022-03-07 18:22   ` Giulio Benetti
2022-03-07 18:22     ` Giulio Benetti
2022-03-11  1:38 ` Jesse Taube
2022-03-11  1:38   ` Jesse Taube

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='CAK8P3a3g_zPW5xEFRku3TsZTdovevwyxyWVaN_=eD+FN2G8YLg@mail.gmail.com' \
    --to=arnd@arndb.de \
    --cc=andre.przywara@arm.com \
    --cc=devicetree@vger.kernel.org \
    --cc=giulio.benetti@benettiengineering.com \
    --cc=icenowy@aosc.io \
    --cc=jernej.skrabec@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-sunxi@lists.linux.dev \
    --cc=linux@armlinux.org.uk \
    --cc=mesihkilinc@gmail.com \
    --cc=mr.bossman075@gmail.com \
    --cc=mripard@kernel.org \
    --cc=olof@lixom.net \
    --cc=robh+dt@kernel.org \
    --cc=samuel@sholland.org \
    --cc=soc@kernel.org \
    --cc=thirtythreeforty@gmail.com \
    --cc=wens@csie.org \
    /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.