Messages in this thread | | | From | Prathamesh Shete <> | Subject | RE: [PATCH v7 2/4] mmc: sdhci-tegra: Add support to program MC stream ID | Date | Thu, 13 Oct 2022 06:33:27 +0000 |
| |
Hi Ulf,
>> In that case, perhaps we can add a "depends on IOMMU_API" in the Kconfig >> instead? Or is the tegra driver used on platforms where IOMMU_API could be >> unset? Yes it can/will work with IOMMU disabled so its not recommended to add a "depends on" condition in Kconfig.
Thanks Prathamesh
> -----Original Message----- > From: Ulf Hansson <ulf.hansson@linaro.org> > Sent: Tuesday, October 11, 2022 6:04 PM > To: Prathamesh Shete <pshete@nvidia.com> > Cc: adrian.hunter@intel.com; thierry.reding@gmail.com; Jonathan Hunter > <jonathanh@nvidia.com>; p.zabel@pengutronix.de; linux- > mmc@vger.kernel.org; linux-tegra@vger.kernel.org; linux- > kernel@vger.kernel.org; Aniruddha Tvs Rao <anrao@nvidia.com>; Suresh > Mangipudi <smangipudi@nvidia.com>; Krishna Yarlagadda > <kyarlagadda@nvidia.com> > Subject: Re: [PATCH v7 2/4] mmc: sdhci-tegra: Add support to program MC > stream ID > > External email: Use caution opening links or attachments > > > On Tue, 11 Oct 2022 at 13:44, Prathamesh Shete <pshete@nvidia.com> wrote: > > > > Hi Ulf > > > > The initial patches were without the #ifdef. #ifdef is being added as per review > comments and kernel robot errors. > > Following error was detected by kernel robot > > >> > > All errors (new ones prefixed by >>): > > > > drivers/mmc/host/sdhci-tegra.c: In function 'sdhci_tegra_probe': > > >> drivers/mmc/host/sdhci-tegra.c:1794:54: error: 'struct iommu_fwspec' has > no member named 'ids' > > 1794 | tegra_host->streamid = fwspec->ids[0] & 0xffff; > > | ^~ > > > > > > vim +1794 drivers/mmc/host/sdhci-tegra.c > > >> > > Adrian also pointed out this issue so to address these issues #ifdef > > was added > > I see! > > In that case, perhaps we can add a "depends on IOMMU_API" in the Kconfig > instead? Or is the tegra driver used on platforms where IOMMU_API could be > unset? > > [...] > > Kind regards > Uffe
| |