All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takahiro Kuwano <tkuw584924@gmail.com>
To: Tudor Ambarus <tudor.ambarus@linaro.org>, u-boot@lists.denx.de
Cc: jagan@amarulasolutions.com, vigneshr@ti.com, pratyush@kernel.org,
	d-gole@ti.com, Bacem.Daassi@infineon.com,
	Takahiro Kuwano <Takahiro.Kuwano@infineon.com>
Subject: Re: [PATCH 3/4] mtd: spi-nor-core: Rework default_init() to take flash_parameter
Date: Mon, 15 Apr 2024 16:09:31 +0900	[thread overview]
Message-ID: <7149031f-b359-4218-9a52-e4ab0ea16c4c@gmail.com> (raw)
In-Reply-To: <377037b3-1bf2-492b-bb88-de233d8b373c@linaro.org>

Hi Tudor,

On 4/15/2024 3:47 PM, Tudor Ambarus wrote:
> 
> 
> On 4/15/24 05:33, tkuw584924@gmail.com wrote:
>> From: Takahiro Kuwano <Takahiro.Kuwano@infineon.com>
>>
>> default_init() fixup hook should be used to initialize flash parameters
>> when its information is not provided in SFDP. To support that case, it
>> needs to take flash_parameter structure like as other hooks.
>>
>> Signed-off-by: Takahiro Kuwano <Takahiro.Kuwano@infineon.com>
>> ---
> 
> I'd like to get rid of the default_init hook, let's not extend it if
> possible. Can you use the late_init hook instead?
> 
It looks easy to migrate from default_init to late_init so I will do it.
Could you provide the links to related discussion in Linux MTD side so that
I can summarize it in commit message?

Thanks,
Takahiro

  reply	other threads:[~2024-04-15  7:09 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-15  4:33 [PATCH 0/4] mtd: Make sure UBIFS does not do multi-pass page programming on flashes that don't support it tkuw584924
2024-04-15  4:33 ` [PATCH 1/4] mtd: ubi: Do not zero out EC and VID on ECC-ed NOR flashes tkuw584924
2024-04-15  6:42   ` Tudor Ambarus
2024-04-15  4:33 ` [PATCH 2/4] mtd: spi-nor: Allow flashes to specify MTD writesize tkuw584924
2024-04-15  6:53   ` Tudor Ambarus
2024-04-15  4:33 ` [PATCH 3/4] mtd: spi-nor-core: Rework default_init() to take flash_parameter tkuw584924
2024-04-15  6:47   ` Tudor Ambarus
2024-04-15  7:09     ` Takahiro Kuwano [this message]
2024-04-15  7:27       ` Tudor Ambarus
2024-04-15  7:34         ` Takahiro Kuwano
2024-04-15  4:33 ` [PATCH 4/4] mtd: spi-nor: Set ECC unit size to MTD writesize in Infineon SEMPER flashes tkuw584924
2024-04-15  6:53   ` Tudor Ambarus
2024-04-15  6:56 ` [PATCH 0/4] mtd: Make sure UBIFS does not do multi-pass page programming on flashes that don't support it Tudor Ambarus
2024-04-15  7:00   ` Takahiro Kuwano

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=7149031f-b359-4218-9a52-e4ab0ea16c4c@gmail.com \
    --to=tkuw584924@gmail.com \
    --cc=Bacem.Daassi@infineon.com \
    --cc=Takahiro.Kuwano@infineon.com \
    --cc=d-gole@ti.com \
    --cc=jagan@amarulasolutions.com \
    --cc=pratyush@kernel.org \
    --cc=tudor.ambarus@linaro.org \
    --cc=u-boot@lists.denx.de \
    --cc=vigneshr@ti.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.