Messages in this thread | | | Date | Thu, 27 Feb 2014 13:02:59 -0600 | From | Felipe Balbi <> | Subject | Re: [PATCH v12 3/4] PHY: add APM X-Gene SoC 15Gbps Multi-purpose PHY driver |
| |
Hi,
On Thu, Feb 27, 2014 at 11:14:07AM -0700, Loc Ho wrote: > +/* > + * This function is used to configure the PHY to operation as either SATA Gen1 > + * or Gen2 speed. > + */ > +static void xgene_phy_sata_force_gen(struct xgene_phy_ctx *ctx, > + int lane, int gen)
why do you need to *force* the generation ? Is this because of some silicon errata ? It almost seems like this should be done through link negotiation between both link partners.
-- balbi [unhandled content-type:application/pgp-signature] | |