Message ID | 20241113035826.923918-1-billy_tsai@aspeedtech.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | [v1] i3c: mipi-i3c-hci: Support SETDASA CCC | expand |
On 11/13/24 5:58 AM, Billy Tsai wrote: > When the I3C subsystem wants to assign a dynamic address using the SETDASA > CCC, it needs to attach the I3C device with device info that includes only > the static address. In the HCI, if the driver want to send this SETDASA > CCC, a DAT entry is required to temporarily fill the device's static > address into the dynamic address field. Afterward, the reattach API will > be executed to update the DAT with the correct dynamic addrees value. > > Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com> > --- > drivers/i3c/master/mipi-i3c-hci/core.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > (Minor comment to the subject. Is this rather fix than support SETDASA?) Reviewed-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
diff --git a/drivers/i3c/master/mipi-i3c-hci/core.c b/drivers/i3c/master/mipi-i3c-hci/core.c index a82c47c9986d..6655e8396a35 100644 --- a/drivers/i3c/master/mipi-i3c-hci/core.c +++ b/drivers/i3c/master/mipi-i3c-hci/core.c @@ -438,7 +438,8 @@ static int i3c_hci_attach_i3c_dev(struct i3c_dev_desc *dev) kfree(dev_data); return ret; } - mipi_i3c_hci_dat_v1.set_dynamic_addr(hci, ret, dev->info.dyn_addr); + mipi_i3c_hci_dat_v1.set_dynamic_addr(hci, ret, + dev->info.dyn_addr ?: dev->info.static_addr); dev_data->dat_idx = ret; } i3c_dev_set_master_data(dev, dev_data);
When the I3C subsystem wants to assign a dynamic address using the SETDASA CCC, it needs to attach the I3C device with device info that includes only the static address. In the HCI, if the driver want to send this SETDASA CCC, a DAT entry is required to temporarily fill the device's static address into the dynamic address field. Afterward, the reattach API will be executed to update the DAT with the correct dynamic addrees value. Signed-off-by: Billy Tsai <billy_tsai@aspeedtech.com> --- drivers/i3c/master/mipi-i3c-hci/core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)