All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Lucien.Jheng" <lucienzx159@gmail.com>
To: Simon Glass <sjg@chromium.org>
Cc: u-boot@lists.denx.de
Subject: Re: [v3,0/4] phy: add common PHY polarity properties support
Date: Sat, 25 Apr 2026 10:18:23 +0800	[thread overview]
Message-ID: <9b3f9140-8549-4693-9ff2-0b4db1ea320a@gmail.com> (raw)
In-Reply-To: <CAFLszTgMvRHfrEa8=-m+AszoKvxg-TshcH2qx4z--yN6ZwoQGg@mail.gmail.com>

Hi Simon

Simon Glass 於 2026/4/11 下午 10:37 寫道:
> Hi Lucien,
>
> On 2026-04-05T14:49:40, Lucien.Jheng<lucienzx159@gmail.com> wrote:
>
>> Patches 3-4 add sandbox DM unit tests for the new library
> Patches 3 and 4 have a bisection problem. Patch 3 adds test code
> referencing DT nodes like /phy-common-props-missing, but those nodes
> are only created in patch 4. Please can you squash patches 3 and 4
> together, or reorder so the DTS changes come first?
I will squash patch 3 and 4 together in the next patch.
>
>> Add obj-$(CONFIG_PHY_COMMON_PROPS) build entry in test/dm/Makefile
> I don't see an update to sandbox_defconfig to enable
> CONFIG_PHY_COMMON_PROPS for testing. Please can you add a patch to
> enable PHY_COMMON_PROPS in sandbox_defconfig, or imply it from
> SANDBOX?

I will update sandbox_defconfig in the next patch.

Thanks

>
> Regards,
> Simon

      reply	other threads:[~2026-04-25 18:07 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-05 14:49 [PATCH v3 0/4] phy: add common PHY polarity properties support Lucien.Jheng
2026-04-05 14:49 ` [PATCH v3 1/4] phy: add common PHY " Lucien.Jheng
2026-04-11 14:37   ` Simon Glass
2026-04-24 16:46     ` Lucien.Jheng
2026-04-05 14:49 ` [PATCH v3 2/4] net: phy: air_en8811h: use standard rx-polarity/tx-polarity properties Lucien.Jheng
2026-04-11 14:37   ` Simon Glass
2026-04-24 16:48     ` Lucien.Jheng
2026-04-05 14:49 ` [PATCH v3 3/4] test: dm: add PHY common props unit tests Lucien.Jheng
2026-04-11 14:38   ` Simon Glass
2026-04-05 14:49 ` [PATCH v3 4/4] sandbox: dts: add PHY common props test nodes Lucien.Jheng
2026-04-11 14:38   ` Simon Glass
2026-04-24 17:01     ` Lucien.Jheng
2026-04-11 14:37 ` [v3,0/4] phy: add common PHY polarity properties support Simon Glass
2026-04-25  2:18   ` Lucien.Jheng [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=9b3f9140-8549-4693-9ff2-0b4db1ea320a@gmail.com \
    --to=lucienzx159@gmail.com \
    --cc=sjg@chromium.org \
    --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.