Re: [PATCH 01/23] ARM: tegra: Create a DT header defining swgroups ID
On Wed, Jun 26, 2013 at 12:31:41PM +0200, Hiroshi Doyu wrote: > Hi Thierry, > > Thierry Reding wrote @ Wed, 26 Jun 2013 12:06:49 > +0200: > ... > > I'm not entirely sure where to find these mappings in the TRM. I see > > that there's a list of the groups in 15.10.11, but where do the numbers > > come from? > > To convert swgroup ID bit to register address, > > #define HWGRP_ASID_REG(x) ((x) * sizeof(u32) + SMMU_AFI_ASID) > > in "[PATCH 15/23] iommu/tegra: smmu: Calculate ASID register offset by ID" Oh, I see. Thanks for pointing that out. > > And why are some of the names aliased? If it's for > > readability only maybe we could add some more for SWGID_HC -> > > SWGID_HOST1X and perhaps SWGID_NV -> SWGID_GR3D. > > I used the exact same name from TRM register definition. I thought > that same naming in TRM may be better since this bit is converted to > register, but I'm quite open to rename if needed. I think it's fine to keep the names from the TRM. I was just wondering why some groups can be referenced using multiple names, like ISP2 -> ISP and MSENC -> MPE. Thierry pgpPvHywoewtF.pgp Description: PGP signature ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH 01/23] ARM: tegra: Create a DT header defining swgroups ID
On Wed, Jun 26, 2013 at 12:28:04PM +0300, Hiroshi Doyu wrote: [...] > +#define SWGID_AFI 0 > +#define SWGID_AVPC 1 > +#define SWGID_DC 2 > +#define SWGID_DCB 3 > +#define SWGID_EPP 4 > +#define SWGID_G2 5 > +#define SWGID_HC 6 > +#define SWGID_HDA 7 > +#define SWGID_ISP 8 > +#define SWGID_ISP2 SWGID_ISP > +/* UNUSED: 9 */ > +/* UNUSED: 10 */ > +#define SWGID_MPE 11 > +#define SWGID_MSENC SWGID_MPE > +#define SWGID_NV 12 > +#define SWGID_NV2 13 > +#define SWGID_PPCS 14 > +#define SWGID_SATA2 15 > +#define SWGID_SATA 16 > +#define SWGID_VDE 17 > +#define SWGID_VI 18 > +#define SWGID_VIC 19 > +#define SWGID_XUSB_HOST 20 > +#define SWGID_XUSB_DEV 21 > +#define SWGID_A9AVP 22 > +#define SWGID_TSEC 23 > +#define SWGID_PPCS1 24 > +/* UNUSED: 25 */ > +/* UNUSED: 26 */ > +/* UNUSED: 27 */ > +/* UNUSED: 28 */ > +/* UNUSED: 29 */ > +/* UNUSED: 30 */ > +/* UNUSED: 31 */ > + > +/* Reserved: 32-63 */ > + > +#define SWGID(x) (1ULL << SWGID_##x) I'm not entirely sure where to find these mappings in the TRM. I see that there's a list of the groups in 15.10.11, but where do the numbers come from? And why are some of the names aliased? If it's for readability only maybe we could add some more for SWGID_HC -> SWGID_HOST1X and perhaps SWGID_NV -> SWGID_GR3D. Thierry pgpA6aXEqUo3Z.pgp Description: PGP signature ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
