All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: "Loureiro, Joao" <Joao.Loureiro@philips.com>
Cc: "u-boot@lists.denx.de" <u-boot@lists.denx.de>
Subject: Re: [PATCH] spi: Introduce initial eeprom driver mode support
Date: Tue, 4 Mar 2025 11:47:40 -0600	[thread overview]
Message-ID: <20250304174740.GF2640854@bill-the-cat> (raw)
In-Reply-To: <AM9P122MB02813D5E7C97871951B4EEC0F2C02@AM9P122MB0281.EURP122.PROD.OUTLOOK.COM>

[-- Attachment #1: Type: text/plain, Size: 903 bytes --]

On Mon, Feb 24, 2025 at 11:43:38AM +0000, Loureiro, Joao wrote:

> This patch introduces the initial SPI EEPROM driver mode support
> analogous to the I2C EEPROM driver mode support. The SPI EEPROM
> driver mode support is enabled by default in the sandbox_defconfig.
> 
> Signed-off-by: João Loureiro <joao.loureiro@philips.com>
> ---
>  configs/sandbox_defconfig |   1 +
>  drivers/misc/Kconfig      |   5 +
>  drivers/misc/Makefile     |   1 +
>  drivers/misc/spi_eeprom.c | 254 ++++++++++++++++++++++++++++++++++++++
>  include/dm/uclass-id.h    |   1 +
>  include/spi_eeprom.h      |  88 +++++++++++++
>  6 files changed, 350 insertions(+)
>  create mode 100644 drivers/misc/spi_eeprom.c
>  create mode 100644 include/spi_eeprom.h

In theory, this seems fine. This patch however produces a large number
of checkpatch.pl warnings and errors to resolve first. Thanks!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  reply	other threads:[~2025-03-04 17:47 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-04 11:38 [PATCH] SPI: Introduce initial EEPROM driver mode support Loureiro, Joao
2025-02-24 11:43 ` [PATCH] spi: Introduce initial eeprom " Loureiro, Joao
2025-03-04 17:47   ` Tom Rini [this message]
2025-03-05 12:26 ` [PATCH v2] spi: Introduce initial EEPROM " Loureiro, Joao
2026-06-10 22:11   ` [PATCH v3 0/2] " João Loureiro
2026-06-10 22:11     ` [PATCH v3 1/2] " João Loureiro
2026-06-10 22:12     ` [PATCH v3 2/2] sandbox: spi: Add SPI EEPROM emulator and DM test João Loureiro
2026-06-14 12:18       ` Simon Glass
2026-06-15 17:51     ` [PATCH v4 0/2] spi: Introduce initial EEPROM driver mode support João Loureiro
2026-06-15 17:51       ` [PATCH v4 1/2] " João Loureiro
2026-07-13 13:03         ` Simon Glass
2026-09-05 10:14           ` João Loureiro
2026-06-15 17:51       ` [PATCH v4 2/2] sandbox: spi: Add SPI EEPROM emulator and DM test João Loureiro
2026-09-05 10:24       ` [PATCH v5 0/3] spi: Introduce driver-model support for SPI EEPROMs João Loureiro
2026-09-10 14:30         ` Simon Glass
2026-09-05 10:24       ` [PATCH v5 1/3] test: dm: sf: Get the emulator attached to the flash slave João Loureiro
2026-09-05 10:24       ` [PATCH v5 2/3] spi: Introduce initial driver-model support for SPI EEPROMs João Loureiro
2026-09-05 10:24       ` [PATCH v5 3/3] sandbox: spi: Add SPI EEPROM emulator and DM test João Loureiro

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=20250304174740.GF2640854@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=Joao.Loureiro@philips.com \
    --cc=u-boot@lists.denx.de \
    /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.