From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id D288337C91F; Thu, 4 Jun 2026 10:55:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780570524; cv=none; b=FrbZh66ulhW9oTkvzVnjcFUP++XdWtAe4A7Y0m36LoxLvaQUAVoSkDtBbxWsFPJCp/zEb6TF3ef73311FS6Xxx4Nqv6GcNlxPULDx8qqzxGv5mD1XHio8/aXEST3dkFXyq8r8m+p2Uz9ebWBaWcPGbTEuzDADNd45Pm+pMGDIm0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780570524; c=relaxed/simple; bh=d4bP0wMj8a3/uR7atdVyxMeW3EzT3BnpkPvm3mnSwRY=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=aJ2YijX8nPjkBAmYZS1NLA6UntDzzmRBqVWojnumqkVPeRumh8GJ/8YvqX/jSyA5xuLfIc2wntQUtRAKxnh9beKtaYbmXBGxEY53Ahu5ZTkJZVhIPuyvu9Sn1lQIP6+r9IgoSTKSKP8gP9hHtsuVSdjx+9YiiwqmmKBIqlwrJhg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=MVw3ZEV1; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="MVw3ZEV1" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 176943297; Thu, 4 Jun 2026 03:55:17 -0700 (PDT) Received: from ryzen.lan (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 8E95C3F7D8; Thu, 4 Jun 2026 03:55:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1780570522; bh=d4bP0wMj8a3/uR7atdVyxMeW3EzT3BnpkPvm3mnSwRY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=MVw3ZEV1sbJtFTRPnYGc6Ua9Xwm6pUOD/dpSkK+1o3NqmWhUsVg60ic6L9HLUZDKr mQ2UmNCUJChc6voaH5jw+TjQM1pURI+UZVq4qrfGxLH4c0EEF+1l7e6wQikUXn0oC+ 3ku3KrnusebqKMOYJCkTuhQChXfiOlhe23KIL7WQ= Date: Thu, 4 Jun 2026 12:54:14 +0200 From: Andre Przywara To: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland Cc: linux-gpio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, Linus Walleij Subject: Re: [PATCH v2 0/3] pinctrl: sunxi: a523: fix GPIO IRQ operation Message-ID: <20260604125414.0e361fcf@ryzen.lan> In-Reply-To: <20260327113006.3135663-1-andre.przywara@arm.com> References: <20260327113006.3135663-1-andre.przywara@arm.com> Organization: Arm Ltd. X-Mailer: Claws Mail 4.4.0 (GTK 3.24.31; x86_64-slackware-linux-gnu) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Fri, 27 Mar 2026 11:30:03 +0000 Andre Przywara wrote: Hi, > this is the minimal fix version for the GPIO IRQ operation on the > Allwinner A523/A527/T527 SoCs. SD card detection is broken as a result, > which is a major annoyance. Those patches here fix that problem, and > should go into v7.0 still, if possible. what is the status of this series, please? We missed v7.0 back then, but please can we take this now, for v7.1? There are review tags for every patch, and this fixes SD card operation on all A523/A527/T572 boards. The series still applies cleanly on current master, if I should resend anyway, please let me know. Cheers, Andre > I dropped the more involved fixes from v1, the risk for regressions is > now very low: > - The quirk flag is just dropped from the A523, not the other SoCs. I > confirmed this again with an experiment, for both the primary and > secondary pincontroller. This avoids fixing the workaround code for > now, which is more involved, but for now unneeded. > - The DT patch just adds the missing interrupt. The IRQ association was > always wrong and never worked, so this can't make it possibly worse. > Together those two patches (plus the required binding change) fix the > problem, I would appreciate if this could be taken ASAP, into v7.0 still. > The generic pinctrl code is now untouched, which makes this also much > easier to backport, and drops the dependencies on other v7.0-rc fixes. > > Bases on v7.0-rc1, but applies on later revisions as well. > > Please have a look and test, especially on A523/A527/T527 boards! > > Changelog v1 .. v2: > - drop generic pinctrl fixes (for now) > - drop quirk removal from other SoCs (for now) > - add Chen-Yu's tag > > Cheers, > Andre > > Andre Przywara (3): > pinctrl: sunxi: a523: Remove unneeded IRQ remuxing flag > dt-bindings: pinctrl: sun55i-a523: increase IRQ banks number > arm64: dts: allwinner: a523: Add missing GPIO interrupt > > .../bindings/pinctrl/allwinner,sun55i-a523-pinctrl.yaml | 8 ++++---- > arch/arm64/boot/dts/allwinner/sun55i-a523.dtsi | 3 ++- > drivers/pinctrl/sunxi/pinctrl-sun55i-a523-r.c | 1 - > drivers/pinctrl/sunxi/pinctrl-sun55i-a523.c | 1 - > 4 files changed, 6 insertions(+), 7 deletions(-) > > base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f