Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: qvMUR81qz/blXHjC9i7F5U2cUTC/fbGgbRwyyh94wxUF6lQAm4kVLfzOhThVX6YDvfZyipZ4fY
 Nw4iQcIA1p1A==
IronPort-SDR: 30XZLlu8DuKecQ47qi0RyY8zYmdio8OvPzzcn4EJR3BlFbcGSiQV800aSXcet5FoD3VmHAQo8X
 IVd0UDCGuwmg==
From:   "Shaikh, Azhar" <azhar.shaikh@intel.com>
To:     Heikki Krogerus <heikki.krogerus@linux.intel.com>
CC:     "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH] usb: typec: intel_pmc_mux: Update IOM port status offset
 for AlderLake
Thread-Topic: [PATCH] usb: typec: intel_pmc_mux: Update IOM port status offset
 for AlderLake
Thread-Index: AQHXUQn5EYYa9JR3CkK0cmvUSE1py6r1TUyAgAEIaeA=
Date:   Wed, 26 May 2021 22:11:08 +0000
Message-ID: <MWHPR11MB1518ADD57FD98D687EB4FE8591249@MWHPR11MB1518.namprd11.prod.outlook.com>
References: <20210525020401.6738-1-azhar.shaikh@intel.com>
 <YK3pdXzYKdSy8d31@kuha.fi.intel.com>
In-Reply-To: <YK3pdXzYKdSy8d31@kuha.fi.intel.com>
Accept-Language: en-US
Content-Language: en-US
dlp-reaction: no-action
dlp-version: 11.5.1.3
dlp-product: dlpe-windows
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 3920c1f2-122e-42e4-ca73-08d920932a19
X-MS-Exchange-CrossTenant-originalarrivaltime: 26 May 2021 22:11:08.6961
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Tn0SrHiXPNVRTtLv7XDJu8E4r4UE6xePA74jtRhyiDMqDFGXEcmyJ6QD2v+BkEKDro49m1r9g3b/CMix8Eb8BA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1279
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Heikki Krogerus <heikki.krogerus@linux.intel.com>
> Sent: Tuesday, May 25, 2021 11:24 PM
> To: Shaikh, Azhar <azhar.shaikh@intel.com>
> Cc: gregkh@linuxfoundation.org; linux-usb@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Subject: Re: [PATCH] usb: typec: intel_pmc_mux: Update IOM port status
> offset for AlderLake
>=20
> On Mon, May 24, 2021 at 07:04:01PM -0700, Azhar Shaikh wrote:
> > Intel AlderLake(ADL) IOM has a different IOM port status offset than
> > Intel TigerLake.
> > Add a new ACPI ID for ADL and use the IOM port status offset as per
> > the platform.
> >
> > Signed-off-by: Azhar Shaikh <azhar.shaikh@intel.com>
> > ---
> >  drivers/usb/typec/mux/intel_pmc_mux.c | 28
> > ++++++++++++++++++++++-----
> >  1 file changed, 23 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/usb/typec/mux/intel_pmc_mux.c
> > b/drivers/usb/typec/mux/intel_pmc_mux.c
> > index 46a25b8db72e..5ef41cfa8213 100644
> > --- a/drivers/usb/typec/mux/intel_pmc_mux.c
> > +++ b/drivers/usb/typec/mux/intel_pmc_mux.c
> > @@ -83,8 +83,6 @@ enum {
> >  /*
> >   * Input Output Manager (IOM) PORT STATUS
> >   */
> > -#define IOM_PORT_STATUS_OFFSET				0x560
> > -
> >  #define IOM_PORT_STATUS_ACTIVITY_TYPE_MASK		GENMASK(9,
> 6)
> >  #define IOM_PORT_STATUS_ACTIVITY_TYPE_SHIFT		6
> >  #define IOM_PORT_STATUS_ACTIVITY_TYPE_USB		0x03
> > @@ -144,6 +142,7 @@ struct pmc_usb {
> >  	struct pmc_usb_port *port;
> >  	struct acpi_device *iom_adev;
> >  	void __iomem *iom_base;
> > +	u32 iom_port_status_offset;
> >  };
> >
> >  static void update_port_status(struct pmc_usb_port *port) @@ -153,7
> > +152,8 @@ static void update_port_status(struct pmc_usb_port *port)
> >  	/* SoC expects the USB Type-C port numbers to start with 0 */
> >  	port_num =3D port->usb3_port - 1;
> >
> > -	port->iom_status =3D readl(port->pmc->iom_base +
> IOM_PORT_STATUS_OFFSET +
> > +	port->iom_status =3D readl(port->pmc->iom_base +
> > +				 port->pmc->iom_port_status_offset +
> >  				 port_num * sizeof(u32));
> >  }
> >
> > @@ -559,14 +559,32 @@ static int is_memory(struct acpi_resource *res,
> void *data)
> >  	return !acpi_dev_resource_memory(res, &r);  }
> >
> > +/* IOM ACPI IDs and IOM_PORT_STATUS_OFFSET */ static const struct
> > +acpi_device_id iom_acpi_ids[] =3D {
> > +	/* TigerLake */
> > +	{ "INTC1072", 0x560, },
> > +
> > +	/* AlderLake */
> > +	{ "INTC1079", 0x160, },
> > +	{}
> > +};
> > +
> >  static int pmc_usb_probe_iom(struct pmc_usb *pmc)  {
> >  	struct list_head resource_list;
> >  	struct resource_entry *rentry;
> > -	struct acpi_device *adev;
> > +	static const struct acpi_device_id *id;
> > +	struct acpi_device *adev =3D NULL;
> >  	int ret;
> >
> > -	adev =3D acpi_dev_get_first_match_dev("INTC1072", NULL, -1);
> > +	for (id =3D &iom_acpi_ids[0]; id->id[0]; id++) {
> > +		if (acpi_dev_present((const char *)id, NULL, -1)) {

Also here I think it should be=20
if (acpi_dev_present((id->id, NULL, -1)) {

> > +			pmc->iom_port_status_offset =3D (u32)id-
> >driver_data;
> > +			adev =3D acpi_dev_get_first_match_dev((const char
> *)id, NULL, -1);
>=20
> This one I think you still need to change:
>=20
>                         adev =3D acpi_dev_get_first_match_dev(id->id, NUL=
L, -1)
>=20

Yup. Will fix this.

> > +			break;
> > +		}
> > +	}
> > +
> >  	if (!adev)
> >  		return -ENODEV;
> >
>=20
> thanks,
>=20
> --
> heikki