Hi all,
Today's linux-next merge of the opp tree got a conflict in:
drivers/media/platform/qcom/venus/pm_helpers.c
between commit:
08b1cf474b7f ("media: venus: core, venc, vdec: Fix probe dependency error")
from the v4l-dvb tree and commit:
857219ae4043 ("media: venus: Convert to use resource-managed OPP API")
from the opp tree.
I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging. You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.
--
Cheers,
Stephen Rothwell
diff --cc drivers/media/platform/qcom/venus/pm_helpers.c
index e349d01422c5,e61317220b9a..000000000000
--- a/drivers/media/platform/qcom/venus/pm_helpers.c
+++ b/drivers/media/platform/qcom/venus/pm_helpers.c
@@@ -750,12 -753,11 +750,11 @@@ static int venc_power_v4(struct device
return ret;
}
-static int vcodec_domains_get(struct device *dev)
+static int vcodec_domains_get(struct venus_core *core)
{
int ret;
- struct opp_table *opp_table;
struct device **opp_virt_dev;
- struct venus_core *core = dev_get_drvdata(dev);
+ struct device *dev = core->dev;
const struct venus_resources *res = core->res;
struct device *pd;
unsigned int i;
@@@ -826,13 -825,11 +821,11 @@@ skip_pmdomains
if (core->opp_dl_venus)
device_link_del(core->opp_dl_venus);
-
- dev_pm_opp_detach_genpd(core->opp_table);
}
-static int core_get_v4(struct device *dev)
+static int core_get_v4(struct venus_core *core)
{
- struct venus_core *core = dev_get_drvdata(dev);
+ struct device *dev = core->dev;
const struct venus_resources *res = core->res;
int ret;
@@@ -871,35 -867,24 +863,24 @@@
}
}
- ret = vcodec_domains_get(dev);
+ ret = vcodec_domains_get(core);
- if (ret) {
- if (core->has_opp_table)
- dev_pm_opp_of_remove_table(dev);
- dev_pm_opp_put_clkname(core->opp_table);
+ if (ret)
return ret;
- }
return 0;
}
-static void core_put_v4(struct device *dev)
+static void core_put_v4(struct venus_core *core)
{
- struct device *dev = core->dev;
-
if (legacy_binding)
return;
- vcodec_domains_put(dev);
+ vcodec_domains_put(core);
-
- if (core->has_opp_table)
- dev_pm_opp_of_remove_table(dev);
- dev_pm_opp_put_clkname(core->opp_table);
-
}
-static int core_power_v4(struct device *dev, int on)
+static int core_power_v4(struct venus_core *core, int on)
{
- struct venus_core *core = dev_get_drvdata(dev);
+ struct device *dev = core->dev;
struct device *pmctrl = core->pmdomains[0];
int ret = 0;
Thanks Stephen!
On 3/23/21 2:27 AM, Stephen Rothwell wrote:
> Hi all,
>
> Today's linux-next merge of the opp tree got a conflict in:
>
> drivers/media/platform/qcom/venus/pm_helpers.c
>
> between commit:
>
> 08b1cf474b7f ("media: venus: core, venc, vdec: Fix probe dependency error")
>
> from the v4l-dvb tree and commit:
>
> 857219ae4043 ("media: venus: Convert to use resource-managed OPP API")
>
> from the opp tree.
>
> I fixed it up (see below) and can carry the fix as necessary. This
> is now fixed as far as linux-next is concerned, but any non trivial
> conflicts should be mentioned to your upstream maintainer when your tree
> is submitted for merging. You may also want to consider cooperating
> with the maintainer of the conflicting tree to minimise any particularly
> complex conflicts.
>
I don't know what is the best solution here.
Viresh, Can I take the OPP API changes through media-tree to avoid
conflicts?
--
regards,
Stan
On 24-03-21, 16:49, Stanimir Varbanov wrote:
> Thanks Stephen!
>
> On 3/23/21 2:27 AM, Stephen Rothwell wrote:
> > Hi all,
> >
> > Today's linux-next merge of the opp tree got a conflict in:
> >
> > drivers/media/platform/qcom/venus/pm_helpers.c
> >
> > between commit:
> >
> > 08b1cf474b7f ("media: venus: core, venc, vdec: Fix probe dependency error")
> >
> > from the v4l-dvb tree and commit:
> >
> > 857219ae4043 ("media: venus: Convert to use resource-managed OPP API")
> >
> > from the opp tree.
> >
> > I fixed it up (see below) and can carry the fix as necessary. This
> > is now fixed as far as linux-next is concerned, but any non trivial
> > conflicts should be mentioned to your upstream maintainer when your tree
> > is submitted for merging. You may also want to consider cooperating
> > with the maintainer of the conflicting tree to minimise any particularly
> > complex conflicts.
> >
>
> I don't know what is the best solution here.
>
> Viresh, Can I take the OPP API changes through media-tree to avoid
> conflicts?
I already suggested something similar earlier, and I was expecting
Thierry to respond to that.. Not sure who should pick those patches.
https://lore.kernel.org/lkml/20210318103250.shjyd66pxw2g2nsd@vireshk-i7/
Can you please respond to this series then ?
--
viresh
Hi Viresh,
On 3/25/21 6:25 AM, Viresh Kumar wrote:
> On 24-03-21, 16:49, Stanimir Varbanov wrote:
>> Thanks Stephen!
>>
>> On 3/23/21 2:27 AM, Stephen Rothwell wrote:
>>> Hi all,
>>>
>>> Today's linux-next merge of the opp tree got a conflict in:
>>>
>>> drivers/media/platform/qcom/venus/pm_helpers.c
>>>
>>> between commit:
>>>
>>> 08b1cf474b7f ("media: venus: core, venc, vdec: Fix probe dependency error")
>>>
>>> from the v4l-dvb tree and commit:
>>>
>>> 857219ae4043 ("media: venus: Convert to use resource-managed OPP API")
>>>
>>> from the opp tree.
>>>
>>> I fixed it up (see below) and can carry the fix as necessary. This
>>> is now fixed as far as linux-next is concerned, but any non trivial
>>> conflicts should be mentioned to your upstream maintainer when your tree
>>> is submitted for merging. You may also want to consider cooperating
>>> with the maintainer of the conflicting tree to minimise any particularly
>>> complex conflicts.
>>>
>>
>> I don't know what is the best solution here.
>>
>> Viresh, Can I take the OPP API changes through media-tree to avoid
>> conflicts?
>
> I already suggested something similar earlier, and I was expecting
> Thierry to respond to that.. Not sure who should pick those patches.
>
> https://lore.kernel.org/lkml/20210318103250.shjyd66pxw2g2nsd@vireshk-i7/
I guess you meant this thread.
https://lore.kernel.org/lkml/[email protected]/
>
> Can you please respond to this series then ?
>
Done.
--
regards,
Stan
On 25-03-21, 10:14, Stanimir Varbanov wrote:
> I guess you meant this thread.
>
> https://lore.kernel.org/lkml/[email protected]/
I actually thought some other stuff is having the merge issues (as I
was expecting something to happen there) :)
Anyway, you did the right thing. Thanks.
--
viresh