On Thu, Jan 28, 2016 at 07:08:56PM +0100, Andrew Jones wrote:
> On Wed, Jan 27, 2016 at 11:51:38AM +0800, Shannon Zhao wrote:
> > @@ -804,10 +831,10 @@ static const struct sys_reg_desc sys_reg_descs[] = {
> > access_pmcr, reset_pmcr, },
> > /* PMCNTENSET_EL0 */
> > { Op0(0b11), Op1(0b011), CRn(0b1001), CRm(0b1100), Op2(0b001),
> > - trap_raz_wi },
> > + access_pmcnten, reset_unknown, PMCNTENSET_EL0 },
> > /* PMCNTENCLR_EL0 */
> > { Op0(0b11), Op1(0b011), CRn(0b1001), CRm(0b1100), Op2(0b010),
> > - trap_raz_wi },
> > + access_pmcnten, NULL, PMCNTENSET_EL0 },
>
> I don't think the reg field is needed, as the reset handler isn't
> defined and the access handler doesn't use it. Oh, and shouldn't it be
> PMCNTENCLR_EL0 anyway?
eh.. nevermind. Of course we just have the one sys_reg for both set/clr...
_______________________________________________
kvmarm mailing list
[email protected]
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
