U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Christoph Niedermaier <cniedermaier@dh-electronics.com>
To: Michael Nazzareno Trimarchi <michael@amarulasolutions.com>
Cc: U-Boot-Denx <u-boot@lists.denx.de>,
	linux-amarula <linux-amarula@amarulasolutions.com>,
	Peng Fan <peng.fan@nxp.com>, Adam Ford <aford173@gmail.com>,
	Lukasz Majewski <lukma@denx.de>,
	Sean Anderson <seanga2@gmail.com>, Simon Glass <sjg@chromium.org>,
	Stefano Babic <sbabic@nabladev.com>,
	Tom Rini <trini@konsulko.com>
Subject: RE: [PATCH v3 1/4] clk: imx: add i.MX6UL clk driver
Date: Fri, 30 May 2025 19:23:30 +0000	[thread overview]
Message-ID: <0c86ed2a45be47cf9ea591d8872019b3@dh-electronics.com> (raw)
In-Reply-To: <CAOf5uwmpau9YUQevx0+gKyLn4_Q0GhaWpXqBzBZ-eDm8zjweBw@mail.gmail.com>

Hello,

> From: Michael Nazzareno Trimarchi <michael@amarulasolutions.com>
> Sent: Friday, May 30, 2025 5:12 PM
> Subject: Re: [PATCH v3 1/4] clk: imx: add i.MX6UL clk driver
> 
> [...]
> 
>>> I have only an imx6ull for testing. At the moment it looks good, although
>>> the Ethernet on my device is not yet working with this clock driver. I need
>>> to find out why it's not working and will probably do a follow up patch.
>>>
>>> With that changes above
>>> Reviewed-by: Christoph Niedermaier <cniedermaier@dh-electronics.com>
>>>
>> 
>> I will add you review and tested-by: during respin. For the ethernet I
>> can take a look
>> but I have not in my module
> 
> Not necessary, with the imx6ulz this makes no sense. I'll have a look
> at it. I will use your patch v4 as the base for my adjustments.
> 
> I have another upstream on going so I should able to test. Anyway I hope that this is the last respin.

I can't say when Ethernet will work on my device. But from my point
of view, the current status of your patch can already be applied.
 
> Do you test the nand?

No, unfortunately I have eMMC on my imx6ull.


Regards
Christoph

      reply	other threads:[~2025-05-30 19:23 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-29  5:10 [PATCH v3 1/4] clk: imx: add i.MX6UL clk driver Michael Trimarchi
2025-05-29  5:10 ` [PATCH v3 2/4] mtd: mxs_nand_dt: Move from clk_get/clk_enable to clk_bulk api Michael Trimarchi
2025-05-29  5:10 ` [PATCH v3 3/4] mtd: nand: Add support for EDO mode 1-5 to IMX6ULL platform Michael Trimarchi
2025-05-29  5:10 ` [PATCH v3 4/4] configs/imx6ulz_smm_m2_defconfig: Enable clock framework Michael Trimarchi
2025-05-30 14:45 ` [PATCH v3 1/4] clk: imx: add i.MX6UL clk driver Christoph Niedermaier
2025-05-30 15:12   ` Michael Nazzareno Trimarchi
2025-05-30 18:13     ` Christoph Niedermaier
2025-05-30 18:20       ` Michael Nazzareno Trimarchi
2025-05-30 19:23         ` Christoph Niedermaier [this message]

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=0c86ed2a45be47cf9ea591d8872019b3@dh-electronics.com \
    --to=cniedermaier@dh-electronics.com \
    --cc=aford173@gmail.com \
    --cc=linux-amarula@amarulasolutions.com \
    --cc=lukma@denx.de \
    --cc=michael@amarulasolutions.com \
    --cc=peng.fan@nxp.com \
    --cc=sbabic@nabladev.com \
    --cc=seanga2@gmail.com \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox