Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Subject: Re: [PATCH 0/4] usb: phy: generic: Support enabling/disabling VBUS
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        linux-usb@vger.kernel.org
Cc:     Robert Jarzmik <robert.jarzmik@free.fr>,
        Felipe Balbi <balbi@kernel.org>, linux-kernel@vger.kernel.org,
        Rob Herring <robh+dt@kernel.org>, devicetree@vger.kernel.org
References: <20220114170941.800068-1-sean.anderson@seco.com>
From:   Sean Anderson <sean.anderson@seco.com>
Message-ID: <ffee044a-d34f-6be5-1fa9-c14cf3bb30de@seco.com>
Date:   Tue, 5 Apr 2022 10:51:34 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
In-Reply-To: <20220114170941.800068-1-sean.anderson@seco.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RzMvbEZJTHE5RlplUnYvRjhHL1YreGdnNHREOTQwYzZpVzdySGl6VEhSZUVa?=
 =?utf-8?B?Z2EvbEYvSmlxMzUxVU1QdVFySWR0QXdWNVc2bGZSczlKekw2TXZNaW5FY3FB?=
 =?utf-8?B?b2Fub1huRkZzamJHYTI2bW9LbncvVzJEc1BSbHZtRVJWMWhVNjQxQzNGMzc2?=
 =?utf-8?B?YXRxM1M2eTVqSmJOSnhEQmRtSmRaTnZ3UFhtN01meGFVcjI1ZWc0SGlCYkIx?=
 =?utf-8?B?elJMdlY5Zy9tSzkyaTlqTldmSStTeVpzQlBXMUppNmcxUDdUZkgxWkJxR2JY?=
 =?utf-8?B?V0hCaWl3bm1Gd2dBTUNLcEk1dDZoU2Z1WTBDdkJXL3lGMUxlWDZib0laLzUz?=
 =?utf-8?B?enQ2cXQ0NEZOMVNtYTZ5cm1TVmlDekpCMFlsY2lHemE2dXRrRnFaMGJFU0cx?=
 =?utf-8?B?WjNIbGlMbjNoOENZWWNqd3cvampsamFsamh5RTBIQ3FxOEIrVGx3Z3FGcjdx?=
 =?utf-8?B?cDV0SXpJaUFoS09KQTJ4Q1hPSmJqalcxc0JFR1lUSzVIVUxKMHBydVBLaVZG?=
 =?utf-8?B?OFJwNkJOVm5UUjM4NDhSRGFPelVmaWVXSmgwOFF3OTlQbmRQaEFaazdieEQr?=
 =?utf-8?B?KzJzamZUdktpMSs0Mk9GRHRXVTlqSWVyVm9Nb0pqQXhpcm5VNzBlRXExOEh6?=
 =?utf-8?B?TUdYMWtOSFNhOWptS21zWGNQRllNV3BRc1NYOUZ5aE5vekU2NjVCUGZxV2kr?=
 =?utf-8?B?SmhoeXArRHpDSUExTnM3ak4wck1sWTdjbWxQS0grRXZyZUlJcHRtUnhHcWFI?=
 =?utf-8?B?QXNlcktrc0lOckNpQTlZV2g5a2hJbFRMbHc2TkVzd0kvMm0yNHp5dTl5M2F4?=
 =?utf-8?B?UXd6enIvQ09qdFY1TDRRUHEwdXdkVUxDdThvQlZXMWVBUjJzTWpBQ1E0Tkd6?=
 =?utf-8?B?M1MvdXkxalVxNHNMR2VzNzZibmpzemRleEVjRE4yd0VjaHBhKzhVWHFaakVE?=
 =?utf-8?B?a09UOFdsUE8xd0JreVVObkVXWVNCQkp1WUVqSmdYWDNJWFFMZ2tyME9WUk5O?=
 =?utf-8?B?YWFRQ091aVEvWFUzVzMyUFRtMURIT0FNYmlBMlZWTlZNSVk0c3NNZDQyUzJI?=
 =?utf-8?B?VGdRRjdQK01pL2Q3OTNEcXRXZDlXTGJLd2pTcTBGQURmUjNPR3lMNzE1UVBN?=
 =?utf-8?B?WnBiV0ZUa0RLM3gvRGFQTlpDbnJjWTNLaHB0VUFyMGkrMzJFOHhMWVhvTVBY?=
 =?utf-8?B?Mkg1Z2N0WWp1L0h2enYycWJ2UFY3SXh6NGVxaXhvNThiQVdTemtJcExOQVM5?=
 =?utf-8?B?Q0JKbFJLOFNuam1qelUvMmplemoxaEFnY2d2RUNRei9Db0xOMXRzVzFZTHJk?=
 =?utf-8?B?ZnBxWnl0TXZzZGRhSVpBaDVIdXQzVldJS2xqSTRmcGMvUlkvL1dOdDhMZno2?=
 =?utf-8?B?WDBpcnFDVnczTEY5NlVwNkZ0Zkdmd3pEQ0FHVmZnVXlEenVhcUVBbGpFVU9G?=
 =?utf-8?B?RjVJeHRQU1dWVllhQkdPT1NBOElmUnJNWWxyZUJQS2pWSll0TVVDTkgvSVdF?=
 =?utf-8?B?ZlhJK1J1Y3A5SUlIMEErNWJqYUFxa1FnNTdkWElLSkdpOXdXV2VGUk5oaHE4?=
 =?utf-8?B?RmQ4WVRlZVpFdnJWbXNRZXV4N3hDTThheTNvdG1Tb1U3THlPd1MwY2JhclRQ?=
 =?utf-8?B?ZERrS0lIeXU4YlJNN0ZPNVFIcEw3d1VJTElXYjhCazlhU3VIWVJWQWEzOW5G?=
 =?utf-8?B?My9zRjkxbFZRRXhmNnRuWlJnWHpXRGhGajBCaGRUNTBiS3ZubnJqR3BMZXow?=
 =?utf-8?B?NytkUi9vTmNUekZzS0hHZFVWMkt5dFNERzBkUlRDalBRYk5TZFZsS2xkbHhk?=
 =?utf-8?B?MmN1amdFYzhvUlNMT25idVVvN0N3VUhSZzJIUUhlbC9CSXJSWWQ5ejFpdXhQ?=
 =?utf-8?B?T2d2blRDcWJsc2hLblF2anZ6azRiL1JxbTg1dGh0d3lJcVo0dmx4K0VyYTFI?=
 =?utf-8?B?bUpCTFJabXNmOFVrazM0TUg4NWM0a043UG9HQXhiSWFrNEVyb29pQlBFRUdS?=
 =?utf-8?B?aXQ5T2hkcnZUcU5OVGJUc0piZnk2U05DUHVabTJjc1VhcFA4ZE1EUk54WllS?=
 =?utf-8?B?UzRYTnBhZC9LdFNtUE1UQVZDdGFwblNHM1JwcERaLzZTclVUTXVsNW9QUkww?=
 =?utf-8?B?UFByQ0N6akJSSStLcFZZc0dqeTBlOTRPcGVtS2FDNXVrZEk3TXN4V3NrZGJi?=
 =?utf-8?B?TEErQTRjcmlPNm5wT0RMcjJvL0FaUE1raFNFQ3ppc1BRWnJKZTNkOW40K3Zm?=
 =?utf-8?B?bEJibCtCcHhlNVFhTkFJZEtMUXYybEEwZ2pSd0Eva2pab24rZ0ZpWCtZTDdv?=
 =?utf-8?B?cjFmNWhENWF2SWhoZFZVdHhjeEFNOXE5b2ljQmt3ZFIvYWQ2YnhONkx2Q1hr?=
 =?utf-8?Q?y8Zcq47KsouQnhDM=3D?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 968ecc32-c7e1-4e0d-7d30-08da1713ca94
X-MS-Exchange-CrossTenant-AuthSource: VI1PR03MB4973.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Apr 2022 14:51:40.0955
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: URDJ5FPbBxKRz1XI6hpqb+mReeppzQtfJNGdEr7CxKdS9yVtxL3NMIlN071dqV9JC8iJ5z6eI/Ze5T9xKRwtIg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR03MB6207
X-Spam-Status: No, score=-4.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE,
	T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 1/14/22 12:09 PM, Sean Anderson wrote:
> The generic USB phy has had VBUS-related code for a long time, but it
> has always been broken, since the regulator was never gotten from the
> device tree. However, the support itself seems not very useful, since
> e.g.  usb_phy_vbus_on/off has no users and usb_phy_set_power is only
> used by gadgets to make sure they don't draw too much current. Instead,
> use the VBUS regulator to implement otg_set_vbus, which is called from
> several drivers. This results in a change in semantics of VBUS, but
> since support was always broken I don't think this will have any affect.
> 
> 
> Sean Anderson (4):
>   dt-bindings: usb: usb-nop-xceiv: Repurpose vbus-regulator
>   usb: phy: generic: Get the vbus supply
>   usb: phy: generic: Implement otg->set_vbus
>   usb: phy: generic: Disable vbus on removal
> 
>  .../bindings/usb/usb-nop-xceiv.yaml           |  8 +--
>  drivers/usb/phy/phy-generic.c                 | 55 +++++++++----------
>  2 files changed, 31 insertions(+), 32 deletions(-)
> 

ping?

When this was submitted I got an email saying that the merge window was
closed... but I think it has opened and closed again during the
intervening time.

--Sean