Re: [PATCH v2 0/5] Support for Marvell AHCI interface on Armada 38x

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 14, 2014 at 06:19:20PM +0200, Thomas Petazzoni wrote:
> Dear Bartlomiej Zolnierkiewicz,
> 
> On Mon, 14 Apr 2014 18:13:07 +0200, Bartlomiej Zolnierkiewicz wrote:
> > On Monday, April 14, 2014 06:01:42 PM Thomas Petazzoni wrote:
> > > Dear Bartlomiej Zolnierkiewicz,
> > > 
> > > On Mon, 14 Apr 2014 17:53:37 +0200, Bartlomiej Zolnierkiewicz wrote:
> > > 
> > > > >  * Fixed the module alias name. Suggested by Bartlomiej
> > > > >    Zolnierkiewicz.
> > > > 
> > > > Well, you could have also added a comment explaining the lack of
> > > > the Power Management support..
> > > 
> > > True, sorry for this. Should I resend a new version of the patch series
> > > with an indication about this in the commit log? Or is this OK as it is?
> > 
> > Please include the comment in the next revision if this is not too much
> > trouble (I assume that you would like to do another re-spin just to add
> > Acked-by/Reviewed-by tags anyway).
> 
> Well, I wasn't planning on re-spinning just to add Acked-by/Reviewed-by
> tags, as those are generally added by the subsystem maintainers when
> applying the patches. Though if Tejun says otherwise, I can definitely
> re-spin without any problem.
> 
> Tejun, just let me know what you prefer.

I can collect the -by's but better patch description is always nice.
Maybe just update the patch description and post that one as reply to
the patch posted in this thread?  Also, how do you want the patches to
be routed?  Can all go through libata?

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux