All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
To: Kishon Vijay Abraham I <kishon-l0cyMroinI0@public.gmane.org>
Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	mark.rutland-5wv7dgnIgG8@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org,
	khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org,
	johnyoun-HKixBCOQz3hWk0Htik3J/w@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH] phy: meson8b-usb2: add support for the USB PHY on Meson8 SoCs
Date: Mon, 8 May 2017 22:02:11 +0200	[thread overview]
Message-ID: <CAFBinCDed096pnD-mm+Y1iRXfZgBkZPgVjH537cUsZ+qd6g-2g@mail.gmail.com> (raw)
In-Reply-To: <950abc53-df2d-93dd-68d1-e9868445bb93-l0cyMroinI0@public.gmane.org>

On Mon, May 8, 2017 at 2:10 PM, Kishon Vijay Abraham I <kishon-l0cyMroinI0@public.gmane.org> wrote:
> Hi,
>
> On Saturday 06 May 2017 11:06 PM, Martin Blumenstingl wrote:
>> Meson8 uses the same USB PHY as found on the Meson8b and GXBB SoCs. Add
>> a new of_device_id to indicate this. Also update the documentation
>> (devicetree binding, Kconfig and MODULE_DESCRIPTION) accordingly.
>>
>> Signed-off-by: Martin Blumenstingl <martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
>> ---
>>  Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt | 3 ++-
>>  drivers/phy/Kconfig                                        | 4 ++--
>>  drivers/phy/phy-meson8b-usb2.c                             | 5 +++--
>>  3 files changed, 7 insertions(+), 5 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt b/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> index 5fa73b9d20f5..d81d73aea608 100644
>> --- a/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> +++ b/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> @@ -1,7 +1,8 @@
>> -* Amlogic Meson8b and GXBB USB2 PHY
>> +* Amlogic Meson8, Meson8b and GXBB USB2 PHY
>>
>>  Required properties:
>>  - compatible:        Depending on the platform this should be one of:
>> +     "amlogic,meson8-usb2-phy"
>>       "amlogic,meson8b-usb2-phy"
>>       "amlogic,meson-gxbb-usb2-phy"
>
> Please split the dt binding documentation into a separate patch.
OK, however I guess I'll wait for a few days until that restructuring
patch is merged

thanks for reviewing this so quick!


Regards,
Martin
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: martin.blumenstingl@googlemail.com (Martin Blumenstingl)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] phy: meson8b-usb2: add support for the USB PHY on Meson8 SoCs
Date: Mon, 8 May 2017 22:02:11 +0200	[thread overview]
Message-ID: <CAFBinCDed096pnD-mm+Y1iRXfZgBkZPgVjH537cUsZ+qd6g-2g@mail.gmail.com> (raw)
In-Reply-To: <950abc53-df2d-93dd-68d1-e9868445bb93@ti.com>

On Mon, May 8, 2017 at 2:10 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote:
> Hi,
>
> On Saturday 06 May 2017 11:06 PM, Martin Blumenstingl wrote:
>> Meson8 uses the same USB PHY as found on the Meson8b and GXBB SoCs. Add
>> a new of_device_id to indicate this. Also update the documentation
>> (devicetree binding, Kconfig and MODULE_DESCRIPTION) accordingly.
>>
>> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
>> ---
>>  Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt | 3 ++-
>>  drivers/phy/Kconfig                                        | 4 ++--
>>  drivers/phy/phy-meson8b-usb2.c                             | 5 +++--
>>  3 files changed, 7 insertions(+), 5 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt b/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> index 5fa73b9d20f5..d81d73aea608 100644
>> --- a/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> +++ b/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> @@ -1,7 +1,8 @@
>> -* Amlogic Meson8b and GXBB USB2 PHY
>> +* Amlogic Meson8, Meson8b and GXBB USB2 PHY
>>
>>  Required properties:
>>  - compatible:        Depending on the platform this should be one of:
>> +     "amlogic,meson8-usb2-phy"
>>       "amlogic,meson8b-usb2-phy"
>>       "amlogic,meson-gxbb-usb2-phy"
>
> Please split the dt binding documentation into a separate patch.
OK, however I guess I'll wait for a few days until that restructuring
patch is merged

thanks for reviewing this so quick!


Regards,
Martin

WARNING: multiple messages have this Message-ID (diff)
From: martin.blumenstingl@googlemail.com (Martin Blumenstingl)
To: linus-amlogic@lists.infradead.org
Subject: [PATCH] phy: meson8b-usb2: add support for the USB PHY on Meson8 SoCs
Date: Mon, 8 May 2017 22:02:11 +0200	[thread overview]
Message-ID: <CAFBinCDed096pnD-mm+Y1iRXfZgBkZPgVjH537cUsZ+qd6g-2g@mail.gmail.com> (raw)
In-Reply-To: <950abc53-df2d-93dd-68d1-e9868445bb93@ti.com>

On Mon, May 8, 2017 at 2:10 PM, Kishon Vijay Abraham I <kishon@ti.com> wrote:
> Hi,
>
> On Saturday 06 May 2017 11:06 PM, Martin Blumenstingl wrote:
>> Meson8 uses the same USB PHY as found on the Meson8b and GXBB SoCs. Add
>> a new of_device_id to indicate this. Also update the documentation
>> (devicetree binding, Kconfig and MODULE_DESCRIPTION) accordingly.
>>
>> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
>> ---
>>  Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt | 3 ++-
>>  drivers/phy/Kconfig                                        | 4 ++--
>>  drivers/phy/phy-meson8b-usb2.c                             | 5 +++--
>>  3 files changed, 7 insertions(+), 5 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt b/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> index 5fa73b9d20f5..d81d73aea608 100644
>> --- a/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> +++ b/Documentation/devicetree/bindings/phy/meson8b-usb2-phy.txt
>> @@ -1,7 +1,8 @@
>> -* Amlogic Meson8b and GXBB USB2 PHY
>> +* Amlogic Meson8, Meson8b and GXBB USB2 PHY
>>
>>  Required properties:
>>  - compatible:        Depending on the platform this should be one of:
>> +     "amlogic,meson8-usb2-phy"
>>       "amlogic,meson8b-usb2-phy"
>>       "amlogic,meson-gxbb-usb2-phy"
>
> Please split the dt binding documentation into a separate patch.
OK, however I guess I'll wait for a few days until that restructuring
patch is merged

thanks for reviewing this so quick!


Regards,
Martin

  parent reply	other threads:[~2017-05-08 20:02 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-06 17:36 [PATCH] phy: meson8b-usb2: add support for the USB PHY on Meson8 SoCs Martin Blumenstingl
2017-05-06 17:36 ` Martin Blumenstingl
2017-05-06 17:36 ` Martin Blumenstingl
2017-05-08 12:10 ` Kishon Vijay Abraham I
2017-05-08 12:10   ` Kishon Vijay Abraham I
2017-05-08 12:10   ` Kishon Vijay Abraham I
     [not found]   ` <950abc53-df2d-93dd-68d1-e9868445bb93-l0cyMroinI0@public.gmane.org>
2017-05-08 20:02     ` Martin Blumenstingl [this message]
2017-05-08 20:02       ` Martin Blumenstingl
2017-05-08 20:02       ` Martin Blumenstingl
     [not found] ` <20170506173625.19501-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
2017-05-20 14:12   ` [PATCH v2 0/2] Meson8 USB PHY support Martin Blumenstingl
2017-05-20 14:12     ` Martin Blumenstingl
2017-05-20 14:12     ` Martin Blumenstingl
     [not found]     ` <20170520141250.27388-1-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
2017-05-20 14:12       ` [PATCH v2 1/2] phy: meson8b-usb2: add support for the USB PHY on Meson8 SoCs Martin Blumenstingl
2017-05-20 14:12         ` Martin Blumenstingl
2017-05-20 14:12         ` Martin Blumenstingl
2017-05-20 14:12       ` [PATCH v2 2/2] dt-bindings: phy: meson8b-usb2-phy: add support for the " Martin Blumenstingl
2017-05-20 14:12         ` Martin Blumenstingl
2017-05-20 14:12         ` Martin Blumenstingl
     [not found]         ` <20170520141250.27388-3-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org>
2017-05-30 22:51           ` Rob Herring
2017-05-30 22:51             ` Rob Herring
2017-05-30 22:51             ` Rob Herring
2017-06-01 13:12     ` [PATCH v2 0/2] Meson8 USB PHY support Kishon Vijay Abraham I
2017-06-01 13:12       ` Kishon Vijay Abraham I
2017-06-01 13:12       ` Kishon Vijay Abraham I
     [not found]       ` <02656f9a-33d8-c009-7785-a6bd7d4b2617-l0cyMroinI0@public.gmane.org>
2017-06-02 19:14         ` Martin Blumenstingl
2017-06-02 19:14           ` Martin Blumenstingl
2017-06-02 19:14           ` Martin Blumenstingl

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=CAFBinCDed096pnD-mm+Y1iRXfZgBkZPgVjH537cUsZ+qd6g-2g@mail.gmail.com \
    --to=martin.blumenstingl-gm/ye1e23mwn+bqq9rbeug@public.gmane.org \
    --cc=carlo-KA+7E9HrN00dnm+yROfE0A@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=johnyoun-HKixBCOQz3hWk0Htik3J/w@public.gmane.org \
    --cc=khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org \
    --cc=kishon-l0cyMroinI0@public.gmane.org \
    --cc=linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    /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.