Received-SPF: pass (google.com: domain of linux-kernel+bounces-43065-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Received-SPF: Pass (protection.outlook.com: domain of seco.com designates
 20.160.56.85 as permitted sender) receiver=protection.outlook.com;
 client-ip=20.160.56.85; helo=repost-eu.tmcas.trendmicro.com; pr=C
Authentication-Results-Original: dkim=none (message not signed)
 header.d=none;dmarc=none action=none header.from=seco.com;
Message-ID: <39a17b10-c66a-432e-b4ed-012374dcbe20@seco.com>
Date: Mon, 29 Jan 2024 11:18:19 -0500
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v3 2/4] usb: phy: generic: Get the vbus supply
Content-Language: en-US
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>, linux-usb@vger.kernel.org
Cc: Robert Jarzmik <robert.jarzmik@free.fr>, Felipe Balbi <balbi@kernel.org>,
 Linus Walleij <linus.walleij@linaro.org>,
 Jonathan Hunter <jonathanh@nvidia.com>,
 Mike Looijmans <mike.looijmans@topic.nl>,
 Jules Maselbas <jmaselbas@kalray.eu>, linux-kernel@vger.kernel.org
References: <20240123225111.1629405-1-sean.anderson@seco.com>
 <20240123225111.1629405-3-sean.anderson@seco.com>
From: Sean Anderson <sean.anderson@seco.com>
In-Reply-To: <20240123225111.1629405-3-sean.anderson@seco.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-Microsoft-Antispam-Message-Info-Original:
 rIYmaihUVp/1WNXyq7iJuDY81jBDyD8s+qB/pBnyc1UnqwsETHCD7zrjKeh2IZUb5wQ4i92uIph42uXvqJrXOqLjBcDO1gZxbXMHvI2+Wacex5jtLEL4f/t73uV3r3a4TXRa6DJX/Hgt91Fe/5tCWt2HeX93K0sptmweRRbtPoVy4r/zAahzM4m6GHsdrgnukhXBPVtFyNGPW0mpPCiCBM/DvLvs5BdojUz5qjJd1z4ER8BkX71VpYkc9VwdWkTJEnAgv7PpGXTwnx7JWrIlgE365NU12O42braq0sAE3kmKH4fguODXBM/pCAk2JI3QiJ4FOFiU/pP6yjGKUYtbVf+JT9I2bjCqgWTVfNlNzFs1ODgo0RKkVLiJUa767GbF+TlOkYuC5eYWg/FPZRNSJgC/FPU6UKPL7YqfUfn/4GSgE/AdxVN6ikBpuV1uQ74AJNCj/PZ7xZCW2ro/AN3keJvk9jSdjp013IwKZ7KABhLysRlElfLHItROiNIytEInJ1rtZi0e9C8a6PoJQy6PPda3XVkPNJvhCEEqSIrvabQYBTW3Y8zERj6Mh8NMpq86PCZTMR5eRpYYqO6igpj6BIPHNepDjj+XuaweUjxnBxG/HL/eB4fZ9rdjXdTQEZrSjH4+IRxyPADJ7JIskleNaDcCDClH8rudGWLVKlz7dAOE0GBsvK8U6U+VGYdji288hxfryAX9naZbjUVU+4JmGl5Wpte22lCJ+22anW34wRiWGOmv3bH/kv6rcll2612J
X-Forefront-Antispam-Report-Untrusted:
 CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DB9PR03MB8847.eurprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(376002)(346002)(396003)(39850400004)(366004)(136003)(230922051799003)(451199024)(1800799012)(186009)(64100799003)(26005)(6512007)(38100700002)(2616005)(66556008)(8676002)(8936002)(4326008)(5660300002)(44832011)(6486002)(6506007)(53546011)(2906002)(54906003)(66476007)(478600001)(52116002)(6666004)(41300700001)(316002)(66946007)(38350700005)(31696002)(86362001)(36756003)(31686004)(41533002)(15583001)(43740500002)(41080700001)(45980500001);DIR:OUT;SFP:1101;
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR03MB8933
X-EOPAttributedMessage: 0
X-MS-Exchange-Transport-CrossTenantHeadersStripped:
 DB8EUR05FT024.eop-eur05.prod.protection.outlook.com
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id-Prvs:
	96f1017b-cc6b-4d3f-882b-08dc20e5eab4
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	yyjlJ+9bcDZyzh9gT6l4Zfn3HrHpdLWHEvrvo+FI2+P8xfo1oPj8OA9/iNA/G7xnGeSBYSYLEdGzqgCSHdnfKV37J0BP+Qk6gfOax+oKCJsZJmC20k96U6sKfek5N/ktj7PP/NrqKwrqRCRLvLfdQ0ilq/Vs187/CP1Ddp6kFGvG3HXfmyk/LL06UCgJ7z1LSs44u3A6T7BVXM3GvNGT4lnMBeIj8a2pDdBk8Dd5e2peSfUjSN0M83/5gjhPSXbM4l/gvs6jxQGGOcNQfgGoNyE+121N9rNW2SQM9sIEoyHcZ5vRFwp/A8ABiKHuezvRpizptvw+h81Pqd6RvlFW6Ve5HqMT0WROG8h0NDTpB4wMVWZS0ZfZmPi5nCE6kcvBK90cKZ1196oGpJbHzCLYwjT0WOsWFjgMCUlTmHCY/bLDjLuAPjHuGB+gc75A+/B6ijJd/gNsRutinTzEhMfQ6SYLzz96nnUO+VW5875B19ocRyqEQVB6HPLeRrM7H2li7BVfRPUMRTJZCHas93Y5VHbW1XWikoe/ZaFpc0xxJQavzsl598Giq9hrGOD7f3XfCxogjc+yt8OIkTtsrLP7sl4P2f5AQNzXGc0MsU1enQ0UeAYQ7oBFO3g7aLQICHl+3KXwpDH2F5sdR0yupx6V5P7iG/C3SrdfoVqXmHWV5JYS/Q0nUYH/jbuRsHIoHPB6QULwJNEuqhVqXXHeXB+Q8NBnX3p26YIRZsBJQ/Ld1Ca2g8kYU3/f8jDwJmn2Br8bQV12V1RT58Ezn3aXGgXVsO7EROa+BkcCawf2E/xyQsRzlQ/vjObuuwYJdJgaPWpERJW4L7JOpowWAx7MGaTLtxnmAg11DlbydW6sSzt7KmY=
X-Forefront-Antispam-Report:
	CIP:20.160.56.85;CTRY:NL;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:repost-eu.tmcas.trendmicro.com;PTR:repost-eu.tmcas.trendmicro.com;CAT:NONE;SFS:(13230031)(39850400004)(396003)(136003)(346002)(376002)(230922051799003)(82310400011)(186009)(1800799012)(451199024)(64100799003)(46966006)(36840700001)(36860700001)(31696002)(36756003)(47076005)(316002)(54906003)(70206006)(86362001)(70586007)(8676002)(6506007)(53546011)(8936002)(6486002)(478600001)(6666004)(4326008)(26005)(2906002)(336012)(44832011)(6512007)(5660300002)(2616005)(34020700004)(41300700001)(356005)(40480700001)(7596003)(7636003)(82740400003)(31686004)(41533002)(15583001)(43740500002)(41080700001);DIR:OUT;SFP:1101;
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jan 2024 16:18:27.1880
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 74407e92-e978-4c43-a72b-08dc20e5ecd3
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=bebe97c3-6438-442e-ade3-ff17aa50e733;Ip=[20.160.56.85];Helo=[repost-eu.tmcas.trendmicro.com]
X-MS-Exchange-CrossTenant-AuthSource:
	DB8EUR05FT024.eop-eur05.prod.protection.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR03MB6495

On 1/23/24 17:51, Sean Anderson wrote:
> While support for working with a vbus was added, the regulator was never
> actually gotten (despite what was documented). Fix this by actually
> getting the supply from the device tree.
> 
> Fixes: 7acc9973e3c4 ("usb: phy: generic: add vbus support")
> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
> ---
> 
> (no changes since v1)
> 
>  drivers/usb/phy/phy-generic.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/drivers/usb/phy/phy-generic.c b/drivers/usb/phy/phy-generic.c
> index 9ab50f26db60..16494030209e 100644
> --- a/drivers/usb/phy/phy-generic.c
> +++ b/drivers/usb/phy/phy-generic.c
> @@ -274,6 +274,13 @@ int usb_phy_gen_create_phy(struct device *dev, struct usb_phy_generic *nop)
>  		return dev_err_probe(dev, PTR_ERR(nop->vbus_draw),
>  				     "could not get vbus regulator\n");
>  
> +	nop->vbus_draw = devm_regulator_get_exclusive(dev, "vbus");
> +	if (PTR_ERR(nop->vbus_draw) == -ENODEV)
> +		nop->vbus_draw = NULL;
> +	if (IS_ERR(nop->vbus_draw))
> +		return dev_err_probe(dev, PTR_ERR(nop->vbus_draw),
> +				     "could not get vbus regulator\n");
> +
>  	nop->dev		= dev;
>  	nop->phy.dev		= nop->dev;
>  	nop->phy.label		= "nop-xceiv";

OK, so as it turns out, this patch (and only this one) got applied as
03e607cbb293 ("usb: phy: generic: Get the vbus supply"). Which of course
messes with the premise of this series that no one was using the vbus
supply (so we can repurpose it), by adding the possibility that someone
was using the vbus supply. However, the only in-tree user is in
arch/arm/boot/dts/armada-388-helios4.dts, which was added before this
patch (so presumably they don't care).

At the very least, this patch should get dropped (and hopefully no one
complains about the next patch).

--Sean