Hi Peter,
> -----Original Message-----
> From: Peter Crosthwaite [mailto:crosthwaitepe...@gmail.com]
> Sent: Thursday, October 08, 2015 9:22 PM
> To: Alistair Francis
> Cc: Sai Pavan Boddu; Peter Maydell; Stefan Hajnoczi; qemu-
> de...@nongnu.org Developers; Edgar Iglesias; Alistair Francis
> Subject: Re: [Qemu-devel] [PATCH V1] sdhci: Fix hostctl2 write logic.
>
> On Tue, Sep 15, 2015 at 3:59 PM, Alistair Francis <alistai...@gmail.com>
> wrote:
> > On Sun, Sep 13, 2015 at 1:36 PM, Peter Crosthwaite
> > <crosthwaitepe...@gmail.com> wrote:
> >> On Fri, Sep 11, 2015 at 3:30 AM, Sai Pavan Boddu
> >> <sai.pavan.bo...@xilinx.com> wrote:
> >>> From: Peter Crosthwaite <peter.crosthwa...@xilinx.com>
> >>>
> >>> This should be a shifted MASKED_WRITE like all other instances of
> >>> non-word aligned registers.
> >>>
> >>> Signed-off-by: Peter Crosthwaite <peter.crosthwa...@xilinx.com>
> >
> > Looks good to me
> >
> > Reviewed-by: Alistair Francis <alistair.fran...@xilinx.com>
> >
> > Thanks,
> >
> > Alistair
> >
> >>
> >>
> >> As the sender, this requires your signed-off-by line (in addition to
> >> any originals). git commit --amend -s should do it.
> >>
>
> Ping!
>
> I think this is blocked by Pavan's missing SoB. Otherwise it should be ok.
I will resend the again, Present im seeing some merge conflicts. Let me resolve
those and send them soon.
Thanks,
Sai Pavan
>
> Regards,
> Peter
>
> >> Your own RB might help as well (I can't do review as author).
> >>
> >> Regards,
> >> Peter
> >>
> >>> ---
> >>> hw/sd/sdhci.c | 2 +-
> >>> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c
> >>> index 8fd75f7..fd354e3 100644
> >>> --- a/hw/sd/sdhci.c
> >>> +++ b/hw/sd/sdhci.c
> >>> @@ -1059,7 +1059,7 @@ sdhci_write(void *opaque, hwaddr offset,
> uint64_t val, unsigned size)
> >>> value |= SDHC_CTRL2_SAMPLING_CLKSEL;
> >>> }
> >>> s->acmd12errsts = value;
> >>> - s->hostctl2 = value >> 16;
> >>> + MASKED_WRITE(s->hostctl2, mask >> 16, value >> 16);
> >>> break;
> >>> case SDHC_CLKCON:
> >>> if (!(mask & 0xFF000000)) {
> >>> --
> >>> 2.1.1
> >>>
> >>
