Message ID | 20210115153049.3353008-1-f4bug@amsat.org (mailing list archive) |
---|---|
Headers | show |
Series | hw/ssi: imx_spi: Fix various bugs in the imx_spi model | expand |
Hi Philippe, On Fri, Jan 15, 2021 at 11:31 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: > > Hi, > > This is how I understand the ecSPI reset works, after > looking at the IMX6DQRM.pdf datasheet. > > This is a respin of Ben's v5 series [*]. > > Since v6: > - Dropped "Reduce 'change_mask' variable scope" patch > - Fixed inverted reset logic > - Added Juan R-b tags > - Removed 'RFC' tag as tests pass > > Based-on: <1608688825-81519-1-git-send-email-bmeng.cn@gmail.com> > (queued on riscv-next). > This series dropped my imx_spi_soft_reset() change that has the imx_spi_update_irq() moved from imx_spi_reset(). May I know why? Regards, Bin
On 1/16/21 3:03 PM, Bin Meng wrote: > Hi Philippe, > > On Fri, Jan 15, 2021 at 11:31 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: >> >> Hi, >> >> This is how I understand the ecSPI reset works, after >> looking at the IMX6DQRM.pdf datasheet. >> >> This is a respin of Ben's v5 series [*]. >> >> Since v6: >> - Dropped "Reduce 'change_mask' variable scope" patch >> - Fixed inverted reset logic >> - Added Juan R-b tags >> - Removed 'RFC' tag as tests pass >> >> Based-on: <1608688825-81519-1-git-send-email-bmeng.cn@gmail.com> >> (queued on riscv-next). >> > > This series dropped my imx_spi_soft_reset() change that has the > imx_spi_update_irq() moved from imx_spi_reset(). May I know why? Because we don't need it. My comment without looking at the datasheet was incorrect: there is only one single reset on the block. > > Regards, > Bin >
On Sat, Jan 16, 2021 at 11:07 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: > > On 1/16/21 3:03 PM, Bin Meng wrote: > > Hi Philippe, > > > > On Fri, Jan 15, 2021 at 11:31 PM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote: > >> > >> Hi, > >> > >> This is how I understand the ecSPI reset works, after > >> looking at the IMX6DQRM.pdf datasheet. > >> > >> This is a respin of Ben's v5 series [*]. > >> > >> Since v6: > >> - Dropped "Reduce 'change_mask' variable scope" patch > >> - Fixed inverted reset logic > >> - Added Juan R-b tags > >> - Removed 'RFC' tag as tests pass > >> > >> Based-on: <1608688825-81519-1-git-send-email-bmeng.cn@gmail.com> > >> (queued on riscv-next). > >> > > > > This series dropped my imx_spi_soft_reset() change that has the > > imx_spi_update_irq() moved from imx_spi_reset(). May I know why? > > Because we don't need it. My comment without looking at the datasheet > was incorrect: there is only one single reset on the block. Oh, you must have missed Peter's comments. See his comments here: http://patchwork.ozlabs.org/project/qemu-devel/patch/20201202144523.24526-2-bmeng.cn@gmail.com/ Regards, Bin