On Mon 11 Jan 11:43 CST 2021, Vinod Koul wrote:

On 11-01-21, 09:34, Bjorn Andersson wrote:

quoted

On Mon 11 Jan 09:16 CST 2021, Vinod Koul wrote:

quoted

I2C geni driver needs to access struct geni_wrapper, so move it to
header.

Please tell me more!

Glanced through the other patches and the only user I can find it in
patch 5 where you use this to get the struct device * of the wrapper.

That is correct. The dma mapping needs to be done with SE device.

quoted

At least in the DT case this would be [SE]->dev->parent, perhaps we
can't rely on this due to ACPI?

I would have missed that then, but I somehow recall trying that.. Though
I have not looked into ACPI..

Given that we would need to worry about ACPI, do you recommend using
parent or keeping this

We get the wrapper by the means of dev_drv_getdata(dev->parent),
so afaict we need to figure out how to get hold of the wrapper for ACPI
to work either way.

We then need to lug around the wrapper's device for your uses and
exposing the wrapper struct solves this for us. So I'm okay with your
proposal.

Regards,
Bjorn