From: NeilBrown <neil@brown.name>
To: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Cc: gregkh@linuxfoundation.org, driverdev-devel@linuxdriverproject.org
Subject: Re: [PATCH v2 0/3] staging: mt7621-pci-phy: simplify driver to don't use child nodes in DT
Date: Fri, 29 Mar 2019 17:09:22 +1100 [thread overview]
Message-ID: <87ef6q2ovh.fsf@notabene.neil.brown.name> (raw)
In-Reply-To: <20190329055243.14673-1-sergio.paracuellos@gmail.com>
[-- Attachment #1.1: Type: text/plain, Size: 1193 bytes --]
On Fri, Mar 29 2019, Sergio Paracuellos wrote:
> This series change both bindings and driver to don't use child nodes in the device tree.
>
> This changes are inspired after Rob's bindings review which can be found here:
>
> http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2019-March/133123.html
>
> Changes in v2:
> - Use #phy_cells=1 into pcie1_phy also.
> - Update bindings documentation with this change.
> - Assume the cost of allocating MAX_PHYS phy's for both ports in driver code.
>
> Hope this helps.
Thanks. Works correctly now.
Reviewed-by: NeilBrown <neil@brown.name>
NeilBrown
>
> Best regards,
> Sergio Paracuellos
>
> Sergio Paracuellos (3):
> staging: mt7621-dts: simplify pcie phy bindings
> staging: mt7621-pci-phy: update bindings documentation
> staging: mt7621-pci-phy: change driver to don't use child nodes
>
> drivers/staging/mt7621-dts/mt7621.dtsi | 23 ++--------
> .../mediatek,mt7621-pci-phy.txt | 44 ++++---------------
> .../staging/mt7621-pci-phy/pci-mt7621-phy.c | 26 ++++++++---
> 3 files changed, 32 insertions(+), 61 deletions(-)
>
> --
> 2.19.1
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]
[-- Attachment #2: Type: text/plain, Size: 169 bytes --]
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
prev parent reply other threads:[~2019-03-29 6:09 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-29 5:52 [PATCH v2 0/3] staging: mt7621-pci-phy: simplify driver to don't use child nodes in DT Sergio Paracuellos
2019-03-29 5:52 ` [PATCH v2 1/3] staging: mt7621-dts: simplify pcie phy bindings Sergio Paracuellos
2019-03-29 5:52 ` [PATCH v2 2/3] staging: mt7621-pci-phy: update bindings documentation Sergio Paracuellos
2019-03-29 5:52 ` [PATCH v2 3/3] staging: mt7621-pci-phy: change driver to don't use child nodes Sergio Paracuellos
2019-03-29 6:09 ` NeilBrown [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=87ef6q2ovh.fsf@notabene.neil.brown.name \
--to=neil@brown.name \
--cc=driverdev-devel@linuxdriverproject.org \
--cc=gregkh@linuxfoundation.org \
--cc=sergio.paracuellos@gmail.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.