[PATCH 04/14] pinctrl: rzg2l: Drop unnecessary scope

Marek Vasut marek.vasut at mailbox.org
Sun Nov 10 19:33:33 CET 2024


On 10/28/24 9:27 AM, Paul Barker wrote:
> On 27/10/2024 16:16, Marek Vasut wrote:
>> On 10/24/24 5:24 PM, Paul Barker wrote:
>>> In rzg2l_pinconf_set(), there are no new variables defined in the case
>>> statement for PIN_CONFIG_INPUT_ENABLE so no additional scope is needed.
>>>
>>> Signed-off-by: Paul Barker <paul.barker.ct at bp.renesas.com>
>> Reviewed-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
>>
>> btw. it might make sense to split the series per subsystem so it can go
>> in piece by piece.
> 
> I could split this into clk (patch 1), pinctrl (patch 2-4), Ethernet phy
> (patch 5-8) and ravb prep patches (9-10), sent as independent series.

Yes please.

> The support for RZ/G2L Ethernet (patches 11 & 12) would have to wait
> until all those series are merged. The devicetree changes (patches 13 &
> 14) could be sent once the pinctrl patches are merged.

Or just include a patchwork links to the dependencies.

> I think that this would delay things and prevent RZ/G2L Ethernet support
> from landing in v2024.01, so I sent this as one series. I can break it
> up if needed though. Let me know what you would prefer.

Please break it up, I'll try to pick it into 2025.01 still.
It is easier to pick it in more manageable chunks.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list