linux-kernel - Re: [PATCH 08/10] usb: typec: Add support for usb role switch in rt1711h driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <d3268b28-db8c-fb1d-5cc4-5226c4181a2d@huawei.com>
Date:   Mon, 29 Oct 2018 20:22:27 +0800
From:   Chen Yu <chenyu56@...wei.com>
To:     Heikki Krogerus <heikki.krogerus@...ux.intel.com>
CC:     <wangbinghui@...ilicon.com>, <linux-usb@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <suzhuangluan@...ilicon.com>, <kongfei@...ilicon.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Lucas Tsai <lucas_tsai@...htek.com>,
        shufan_lee <shufan_lee@...htek.com>,
        John Stultz <john.stultz@...aro.org>
Subject: Re: [PATCH 08/10] usb: typec: Add support for usb role switch in
 rt1711h driver



On 2018/10/29 19:58, Heikki Krogerus wrote:
> On Sat, Oct 27, 2018 at 05:58:18PM +0800, Yu Chen wrote:
>> This patch add struct device_connection to rt1711h driver to support
>> find "usb-role-switch" in tcpm_register_port.
>>
>> Cc: Heikki Krogerus <heikki.krogerus@...ux.intel.com>
>> Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
>> Cc: Lucas Tsai <lucas_tsai@...htek.com>
>> Cc: shufan_lee <shufan_lee@...htek.com>
>> Cc: John Stultz <john.stultz@...aro.org>
>> Cc: Binghui Wang <wangbinghui@...ilicon.com>
>> Signed-off-by: Yu Chen <chenyu56@...wei.com>
>> ---
>>  drivers/usb/typec/tcpm/tcpci_rt1711h.c | 15 +++++++++++++++
>>  1 file changed, 15 insertions(+)
>>
>> diff --git a/drivers/usb/typec/tcpm/tcpci_rt1711h.c b/drivers/usb/typec/tcpm/tcpci_rt1711h.c
>> index 017389021b96..5f8b952612fe 100644
>> --- a/drivers/usb/typec/tcpm/tcpci_rt1711h.c
>> +++ b/drivers/usb/typec/tcpm/tcpci_rt1711h.c
>> @@ -39,6 +39,7 @@ struct rt1711h_chip {
>>  	struct tcpci_data data;
>>  	struct tcpci *tcpci;
>>  	struct device *dev;
>> +	struct device_connection dev_conn;
>>  };
>>  
>>  static int rt1711h_read16(struct rt1711h_chip *chip, unsigned int reg, u16 *val)
>> @@ -237,6 +238,7 @@ static int rt1711h_probe(struct i2c_client *client,
>>  {
>>  	int ret;
>>  	struct rt1711h_chip *chip;
>> +	const char *dev_conn_end;
>>  
>>  	ret = rt1711h_check_revision(client);
>>  	if (ret < 0) {
>> @@ -254,6 +256,7 @@ static int rt1711h_probe(struct i2c_client *client,
>>  		return PTR_ERR(chip->data.regmap);
>>  
>>  	chip->dev = &client->dev;
>> +	dev_set_name(chip->dev, "rt1711h");
>>  	i2c_set_clientdata(client, chip);
>>  
>>  	ret = rt1711h_sw_reset(chip);
>> @@ -264,6 +267,15 @@ static int rt1711h_probe(struct i2c_client *client,
>>  	if (ret < 0)
>>  		return ret;
>>  
>> +	ret = device_property_read_string(chip->dev, "dev-conn-end",
>> +			&dev_conn_end);
>> +	if (!ret) {
>> +		chip->dev_conn.endpoint[0] = "rt1711h";
>> +		chip->dev_conn.endpoint[1] = dev_conn_end;
>> +		chip->dev_conn.id = "usb-role-switch";
>> +		device_connection_add(&chip->dev_conn);
>> +	}
> That is wrong. You should not read name of a device from a property.
>
> Regardless of that, I don't think this patch is needed at all. You
> need to describe this kind connection with OF device graph
> (Documentation/devicetree/bindings/graph.txt).
>
> I've prepared a proposal how we should be able handle graphs in the
> device connection API:
> https://lkml.org/lkml/2018/10/24/613
I will fix this patch. Thanks!
>
> thanks,
>