> Subject: Re: [PATCH] extcon : register for cable interest by cable name
>
> > There are some scnearios where a driver/framework needs to register
> > interest for a particular cable without specifying the extcon device
> > name. One such scenario is charger notifications. The platform will
> > have charger cabel which will be bound to any extcon device. It's not
> > mandatory for the charger driver to know which extcon device it should
> > use. This patch enables the support for registering interest for a
> > cable just by cable name wihtout specifying the extcon device name
> >
> > Signed-off-by: Jenny TC <jenny...@intel.com>
> > ---
> []
> > +extern struct extcon_dev *register_interest_cable_byname
> > + (struct extcon_specific_cable_nb *extcon_dev,
> > + const char *cable_name, struct notifier_block *nb);
>
> What about allowing extcon_register_interest() to support the functionality?
>
register_interest_cable_byname internally uses extcon_register_interest
function.
If we want to use extcon_register_interest() in both scenarios, then we need
to define
two local functions
extcon_register_interest__ and register_interest_cable_byname.
Then the code would be
int extcon_register_interest(struct extcon_specific_cable_nb *obj,
const char *extcon_name, const char *cable_name,
struct notifier_block *nb) {
if (extcon_name == NULL)
return register_interest_cable_byname(obj,cable_name,nb);
else
return
extcon_register_interest__(obj,extcon_name,cable_name,nb);
}
Is that fine?
> I.e., allowing extcon_register_interest() to accept NULL extcon_name.
>
>
> ps. anyway, usually, the "platforms" have the list of node paths, don't they?
>
My intention is to make it generic and less dependent on platform data. This
way the same consumer
driver can work on any platform without looking for extcon dev name in the
platform data.
For example a charger driver can work on any platform without knowing the
extcon provider name for
The charger cable.
N�����r��y����b�X��ǧv�^�){.n�+����{����zX����ܨ}���Ơz�&j:+v�������zZ+��+zf���h���~����i���z� �w���?�����&�)ߢf��^jǫy�m��@A�a���
0��h���i
