> -----Original Message-----
> From: [email protected]
> [mailto:[email protected]] On Behalf Of Heikki Krogerus
> Sent: 2018年3月15日 19:21
> To: Jun Li <[email protected]>
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; dl-linux-imx <[email protected]>
> Subject: Re: [PATCH v3 05/12] usb: typec: add API to get sink and source
> config
>
> On Tue, Mar 13, 2018 at 05:34:31PM +0800, Li Jun wrote:
> > This patch add 2 APIs to get sink and source power config from
> > firmware description in case the port supports PD.
> >
> > Signed-off-by: Li Jun <[email protected]>
> > ---
> > drivers/usb/typec/tcpm.c | 47
> > +++++++++++++++++++++++++++++++++++++++++++++++
> > include/linux/usb/tcpm.h | 8 +++++---
> > 2 files changed, 52 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/usb/typec/tcpm.c b/drivers/usb/typec/tcpm.c index
> > 7500dc0..0bd34c9 100644
> > --- a/drivers/usb/typec/tcpm.c
> > +++ b/drivers/usb/typec/tcpm.c
> > @@ -13,6 +13,7 @@
> > #include <linux/module.h>
> > #include <linux/mutex.h>
> > #include <linux/proc_fs.h>
> > +#include <linux/property.h>
> > #include <linux/sched/clock.h>
> > #include <linux/seq_file.h>
> > #include <linux/slab.h>
> > @@ -3595,6 +3596,52 @@ static int tcpm_copy_vdos(u32 *dest_vdo, const
> u32 *src_vdo,
> > return nr_vdo;
> > }
> >
> > +int tcpm_get_src_config(struct fwnode_handle *fwnode, struct
> > +tcpc_config *tcfg) {
> > + int ret;
> > +
> > + if (!fwnode)
> > + return -EINVAL;
> > +
> > + ret = fwnode_property_read_u32_array(fwnode, "source-pdos",
> > + NULL, 0);
> > + if (ret <= 0)
> > + return -EINVAL;
> > +
> > + tcfg->nr_src_pdo = min(ret, PDO_MAX_OBJECTS);
> > + return fwnode_property_read_u32_array(fwnode, "source-pdos",
> > + tcfg->src_pdo, tcfg->nr_src_pdo);
> > }
> > +EXPORT_SYMBOL_GPL(tcpm_get_src_config);
> > +
> > +int tcpm_get_snk_config(struct fwnode_handle *fwnode, struct
> > +tcpc_config *tcfg) {
> > + int ret;
> > +
> > + if (!fwnode)
> > + return -EINVAL;
> > +
> > + if ((fwnode_property_read_u32(fwnode, "max-sink-microvolt",
> > + &tcfg->max_snk_mv) < 0) ||
> > + (fwnode_property_read_u32(fwnode, "max-sink-microamp",
> > + &tcfg->max_snk_ma) < 0) ||
> > + (fwnode_property_read_u32(fwnode, "max-sink-microwatt-hours",
> > + &tcfg->max_snk_mw) < 0) ||
> > + (fwnode_property_read_u32(fwnode, "op-sink-microwatt-hours",
> > + &tcfg->operating_snk_mw) < 0))
> > + return -EINVAL;
> > +
> > + ret = fwnode_property_read_u32_array(fwnode, "sink-pdos",
> > + NULL, 0);
> > + if (ret <= 0)
> > + return -EINVAL;
> > +
> > + tcfg->nr_snk_pdo = min(ret, PDO_MAX_OBJECTS);
> > + return fwnode_property_read_u32_array(fwnode, "sink-pdos",
> > + tcfg->snk_pdo, tcfg->nr_snk_pdo);
> > }
> > +EXPORT_SYMBOL_GPL(tcpm_get_snk_config);
>
> tcpm_register_port() can check these. No need to involve the tcpc drivers.
OK, I will read those properties directly to port->typec_caps without involve
tcpc->config.
>
> > int tcpm_update_source_capabilities(struct tcpm_port *port, const u32
> *pdo,
> > unsigned int nr_pdo)
> > {
> > diff --git a/include/linux/usb/tcpm.h b/include/linux/usb/tcpm.h index
> > e2e2db3..5d361f6 100644
> > --- a/include/linux/usb/tcpm.h
> > +++ b/include/linux/usb/tcpm.h
> > @@ -76,10 +76,10 @@ enum tcpm_transmit_type {
> > * @alt_modes: List of supported alternate modes
> > */
> > struct tcpc_config {
> > - const u32 *src_pdo;
> > + u32 *src_pdo;
> > unsigned int nr_src_pdo;
> >
> > - const u32 *snk_pdo;
> > + u32 *snk_pdo;
> > unsigned int nr_snk_pdo;
> >
> > const u32 *snk_vdo;
> > @@ -143,7 +143,7 @@ enum tcpc_mux_mode {
> > * @mux: Pointer to multiplexer data
> > */
> > struct tcpc_dev {
> > - const struct tcpc_config *config;
> > + struct tcpc_config *config;
>
> If you check the properties in tcpm, the above members can continue to be
> declared constant for now.
Agreed.
>
> > struct fwnode_handle *fwnode;
> >
> > int (*init)(struct tcpc_dev *dev);
> > @@ -189,5 +189,7 @@ void tcpm_pd_transmit_complete(struct tcpm_port
> *port,
> > enum tcpm_transmit_status status); void
> > tcpm_pd_hard_reset(struct tcpm_port *port); void
> > tcpm_tcpc_reset(struct tcpm_port *port);
> > +int tcpm_get_src_config(struct fwnode_handle *fwnode, struct
> > +tcpc_config *tcfg); int tcpm_get_snk_config(struct fwnode_handle
> > +*fwnode, struct tcpc_config *tcfg);
>
>
> Cheers,
>
> --
> heikki
> --
> To unsubscribe from this list: send the line "unsubscribe linux-usb" in the
> body
> of a message to [email protected] More majordomo info at
> https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fvger.k
> ernel.org%2Fmajordomo-info.html&data=02%7C01%7Cjun.li%40nxp.com%7C
> cb0c1b22c5a14db0ba3008d58a66de9d%7C686ea1d3bc2b4c6fa92cd99c5c30
> 1635%7C0%7C1%7C636567096807443014&sdata=LmlP8S7pXgzoveyxiAJHuR
> 6aeDiWhh1qj5Vh1KYWFZc%3D&reserved=0
N�Р骒r��y����b�X�肚�v�^�)藓{.n�+�伐�{焙柒��^n�r■�z���h�ㄨ��&Ⅷ �G���h�(�����茛j"���m����赇z罐��帼f＂�h���~�m�
