Hi Jose Miguel Abreu, Thanks for the review... > >> If so then there is no race condition, but the HW image that I have > >> does not have this register enabled so I was getting this result > >> (memory corruption because not all framebuffers had addresses set). > > Thanks for the explanation. > > Agree the issue that you mentioned won't come when > > XILINX_DMA_REG_FRMSTORE > > (C_ENABLE_DEBUG_INFO_5 and C_ENABLE_DEBUG_INFO_13) Register is > enabled in the IP. > > But this register won't get enabled with the default IP configuration > (C_ENABLE_DEBUG_INFO_5 and C_ENABLE_DEBUG_INFO_13). > > > > When user is not enabled XILINX_DMA_REG_FRMSTORE in the h/w and > submits frames less than h/w capable. > > The solution that I am thinking is to throw an error in the driver > > saying that either enable the num frame store feature in the IP or submit the > frames up to h/w capable what do you think??? > > Sounds fine by me. Thanks posted the v3 series please review when you have some time... Regards, Kedar. > > Best regards, > Jose Miguel Abreu > > > > > Regards, > > Kedar. > > > >> Best regards, > >> Jose Miguel Abreu > >> > >>> Regards, > >>> Kedar. > >>> > >>>> Best regards, > >>>> Jose Miguel Abreu > >>>> -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
