| Message ID | 20140117114319.2784.3961.stgit@gklab-154-244.igk.intel.com |
|---|---|
| State | Not Applicable |
| Delegated to: | David Miller |
| Headers | show |
On Fri, Jan 17, 2014 at 12:43:20PM +0100, Lukasz Dorau wrote: > There is an error in a comment introduced by: > commit fa070ee6dc70bcc19737a2406d741b089b3149d5 > libahci: fix turning on LEDs in ahci_start_port() > > Correct it. > > Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com> > --- > drivers/ata/libahci.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c > index c482f8c..1f526e6 100644 > --- a/drivers/ata/libahci.c > +++ b/drivers/ata/libahci.c > @@ -780,8 +780,8 @@ static void ahci_start_port(struct ata_port *ap) > 4); > /* > * If busy, give a breather but do not > - * release EH ownership by using msleep() > - * instead of ata_msleep(). EM Transmit > + * release EH ownership by using ata_msleep() > + * instead of msleep(). EM Transmit Can't the sentence be parsed to mean the same thing either way? "(do not release EH ownership) by using msleep..." or "do not (release EH ownership by using ata_msleep)..." With the context, I don't think which way to group things is ambiguous. In fact, saying "instead of msleep()" is kinda confusing because using ata_msleep() should be the default. Thanks.
On Friday, January 17, 2014 1:00 PM Tejun Heo <tj@kernel.org> wrote: > On Fri, Jan 17, 2014 at 12:43:20PM +0100, Lukasz Dorau wrote: > > There is an error in a comment introduced by: > > commit fa070ee6dc70bcc19737a2406d741b089b3149d5 > > libahci: fix turning on LEDs in ahci_start_port() > > > > Correct it. > > > > Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com> > > --- > > drivers/ata/libahci.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c > > index c482f8c..1f526e6 100644 > > --- a/drivers/ata/libahci.c > > +++ b/drivers/ata/libahci.c > > @@ -780,8 +780,8 @@ static void ahci_start_port(struct ata_port *ap) > > 4); > > /* > > * If busy, give a breather but do not > > - * release EH ownership by using msleep() > > - * instead of ata_msleep(). EM Transmit > > + * release EH ownership by using ata_msleep() > > + * instead of msleep(). EM Transmit > > Can't the sentence be parsed to mean the same thing either way? "(do > not release EH ownership) by using msleep..." or "do not (release EH > ownership by using ata_msleep)..." > > With the context, I don't think which way to group things is > ambiguous. In fact, saying "instead of msleep()" is kinda confusing > because using ata_msleep() should be the default. > You are right. I give up ;-) Thanks, Lukasz -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c index c482f8c..1f526e6 100644 --- a/drivers/ata/libahci.c +++ b/drivers/ata/libahci.c @@ -780,8 +780,8 @@ static void ahci_start_port(struct ata_port *ap) 4); /* * If busy, give a breather but do not - * release EH ownership by using msleep() - * instead of ata_msleep(). EM Transmit + * release EH ownership by using ata_msleep() + * instead of msleep(). EM Transmit * bit is busy for the whole host and * releasing ownership will cause other * ports to fail the same way.
There is an error in a comment introduced by: commit fa070ee6dc70bcc19737a2406d741b089b3149d5 libahci: fix turning on LEDs in ahci_start_port() Correct it. Signed-off-by: Lukasz Dorau <lukasz.dorau@intel.com> --- drivers/ata/libahci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html