All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: Ahmad Fatoum <a.fatoum@pengutronix.de>,
	Patrice CHOTARD <patrice.chotard@st.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>,
	Patrick DELAUNAY <patrick.delaunay@st.com>,
	"linux-stm32@st-md-mailman.stormreply.com"
	<linux-stm32@st-md-mailman.stormreply.com>
Subject: Re: [Linux-stm32] [PATCH 07/22] ARM: dts: stm32: Add alternate pinmux for SDMMC2 pins 4-7
Date: Mon, 30 Mar 2020 13:22:11 +0200	[thread overview]
Message-ID: <fcf49298-a36c-e80e-e62b-1fb9c07f0d6e@denx.de> (raw)
In-Reply-To: <82dcf412-119b-0de2-0c50-f6877a82a812@pengutronix.de>

On 3/30/20 1:17 PM, Ahmad Fatoum wrote:
> Hello Patrice,

Hi,

> On 3/30/20 1:11 PM, Patrice CHOTARD wrote:
>> For your information, another submitted patch uses the same pinctrl sdmmc2_d47_pins_b node with different muxing (SDMMC2_D5)
>>
>> see https://lore.kernel.org/patchwork/patch/1216452/
>>
>> I haven't checked other muxing if there are other conflict.
> 
> (author of linked patch here)
> 
> I don't like the central stm32mp15-pinctrl.dtsi. I'd have preferred if each
> file defined the pinctrl groups it is using.

I'm not a big fan of that either, because this is gonna be a
combinatorial explosion of various pinmux options. But if you have each
board define it's pinmux, it's also gonna become a massive amount of
duplication (like iMX). So I cannot tell which one is better ...

So, just apply one patch or the other, let me know what got applied and
I'll rebase on top of that and resubmit if needed.

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

  reply	other threads:[~2020-03-30 11:22 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-28 17:11 [PATCH 00/22] ARM: dts: stm32: Repair AV96 board Marek Vasut
2020-03-28 17:11 ` [PATCH 01/22] ARM: dts: stm32: Add alternate pinmux for ethernet RGMII Marek Vasut
2020-03-28 17:11 ` [PATCH 02/22] ARM: dts: stm32: Repair ethernet operation on AV96 Marek Vasut
2020-03-28 17:11 ` [PATCH 03/22] ARM: dts: stm32: Add missing ethernet PHY reset " Marek Vasut
2020-03-28 17:11 ` [PATCH 04/22] ARM: dts: stm32: Add missing ethernet PHY skews " Marek Vasut
2020-03-28 17:11 ` [PATCH 05/22] ARM: dts: stm32: Add alternate pinmux for SDMMC1 direction pins Marek Vasut
2020-03-28 17:11 ` [PATCH 06/22] ARM: dts: stm32: Repair SDMMC1 operation on AV96 Marek Vasut
2020-03-28 17:11 ` [PATCH 07/22] ARM: dts: stm32: Add alternate pinmux for SDMMC2 pins 4-7 Marek Vasut
2020-03-30 11:11   ` Patrice CHOTARD
2020-03-30 11:17     ` [Linux-stm32] " Ahmad Fatoum
2020-03-30 11:22       ` Marek Vasut [this message]
2020-03-30 11:37         ` Ahmad Fatoum
2020-03-30 11:45           ` Marek Vasut
2020-03-31  8:58             ` Alexandre Torgue
2020-03-31 13:38               ` Marek Vasut
2020-03-31 16:39                 ` Alexandre Torgue
2020-03-31 16:44                   ` Marek Vasut
2020-04-01  9:49                     ` Alexandre Torgue
2020-03-28 17:11 ` [PATCH 08/22] ARM: dts: stm32: Add eMMC attached to SDMMC2 on AV96 Marek Vasut
2020-03-28 17:11 ` [PATCH 09/22] ARM: dts: stm32: Repair PMIC configuration " Marek Vasut
2020-03-28 17:11 ` [PATCH 10/22] ARM: dts: stm32: Repair PMIC interrupt " Marek Vasut
2020-03-28 17:11 ` [PATCH 11/22] ARM: dts: stm32: Add QSPI NOR " Marek Vasut
2020-03-28 17:11 ` [PATCH 12/22] ARM: dts: stm32: Add configuration EEPROM " Marek Vasut
2020-03-28 17:11 ` [PATCH 13/22] ARM: dts: stm32: Enable GPU " Marek Vasut
2020-03-28 17:11 ` [PATCH 14/22] ARM: dts: stm32: Add alternate pinmux for SDMMC3 pins Marek Vasut
2020-03-28 17:11 ` [PATCH 15/22] ARM: dts: stm32: Enable WiFi on AV96 Marek Vasut
2020-03-28 17:11 ` [PATCH 16/22] ARM: dts: stm32: Add alternate pinmux for USART2 pins Marek Vasut
2020-03-28 17:11 ` [PATCH 17/22] ARM: dts: stm32: Enable Bluetooth on AV96 Marek Vasut
2020-03-28 17:11 ` [PATCH 18/22] ARM: dts: stm32: Add alternate pinmux for LTDC pins Marek Vasut
2020-03-28 17:11 ` [PATCH 19/22] ARM: dts: stm32: Add bindings for HDMI video on AV96 Marek Vasut
2020-03-28 17:11 ` [PATCH 20/22] ARM: dts: stm32: Add bindings for audio " Marek Vasut
2020-03-28 17:11 ` [PATCH 21/22] ARM: dts: stm32: Add bindings for USB " Marek Vasut
2020-03-28 17:11 ` [PATCH 22/22] ARM: dts: stm32: Rename LEDs to match silkscreen " Marek Vasut

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=fcf49298-a36c-e80e-e62b-1fb9c07f0d6e@denx.de \
    --to=marex@denx.de \
    --cc=a.fatoum@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-stm32@st-md-mailman.stormreply.com \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=patrice.chotard@st.com \
    --cc=patrick.delaunay@st.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.