On Thu, 14 Jul 2022 17:03:04 -0700 Dan Williams <dan.j.williams@xxxxxxxxx> wrote: > After all the soft validation of the region has completed, convey the > region configuration to hardware while being careful to commit decoders > in specification mandated order. In addition to programming the endpoint > decoder base-address, interleave ways and granularity, the switch > decoder target lists are also established. > > While the kernel can enforce spec-mandated commit order, it can not > enforce spec-mandated reset order. For example, the kernel can't stop > someone from removing an endpoint device that is occupying decoderN in a > switch decoder where decoderN+1 is also committed. To reset decoderN, > decoderN+1 must be torn down first. That "tear down the world" > implementation is saved for a follow-on patch. > > Callback operations are provided for the 'commit' and 'reset' > operations. While those callbacks may prove useful for CXL accelerators > (Type-2 devices with memory) the primary motivation is to enable a > simple way for cxl_test to intercept those operations. > > Signed-off-by: Dan Williams <dan.j.williams@xxxxxxxxx> LGTM Reviewed-by: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>