Received-SPF: pass (google.com: domain of linux-kernel+bounces-7429-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
From: Aleksandrs Vinarskis <alex.vinarskis@gmail.com>
To: alex.vinarskis@gmail.com
Cc: andriy.shevchenko@linux.intel.com,
	hdegoede@redhat.com,
	lee@kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH v2 0/2] Fix LPSS clock divider for XPS 9530
Date: Wed, 20 Dec 2023 21:56:19 +0100
Message-Id: <20231220205621.8575-1-alex.vinarskis@gmail.com>
In-Reply-To: <20231220073148.19402-1-alex.vinarskis@gmail.com>
References: <20231220073148.19402-1-alex.vinarskis@gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit

Thanks to Andy Shevchenko for the review and suggestions. Fixed
accordingly.

---

Dell XPS 9530 (2023) uses spi-pxa2xx with clock-divider enabled from
intel-lpss with the ratio of 1:32767 (Dell's firmware bug). This caps SPI
controller's speed at very low value of 3051Hz, which makes the interface
practically unusable. Since either spi-pxa2xx or intel-lpss should have
clock divider enabled, not both, and SPI controller can have higher speed
than requested by the device, it is preffered to disable intel-lpss
clock-divider, and let SPI controller handle it.

Fixing this issue directly in Dell firmware by setting lpss divider to 1:1
would've been the ideal solution, but is unlikely to ever happen.

Particular driver already implements customized solution for handling buggy
ACPI tables for select Microsoft devices. This patch series converts it to
a more generic quirk table, and adds a new quirk QUIRK_CLOCK_DIVIDER_UNITY
which forces clock divider to be set to 1:1. In the future, devices with
similar bug (if any) can be easily added to the same pci id lookup table.

Aleksandrs Vinarskis (2):
  mfd: intel-lpss: Switch to generalized quirk table
  mfd: intel-lpss: Introduce QUIRK_CLOCK_DIVIDER_UNITY for XPS 9530

 drivers/mfd/intel-lpss-pci.c | 25 ++++++++++++++++---------
 drivers/mfd/intel-lpss.c     |  9 ++++++++-
 drivers/mfd/intel-lpss.h     | 14 +++++++++++++-
 3 files changed, 37 insertions(+), 11 deletions(-)

-- 
2.40.1