Re: [v6 6/7] phy: phy_brcmstb_sata: add data for phy version

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

 



On 15/12/15 03:23, Jaedon Shin wrote:
>>>   So an uninitialized .data field would mean the same?
>>
>> Are you planning to fix this?
>>
>> Thanks
>> Kishon
> 
> No, I do not have a plan in this version.
> 
> Of course, It is good that correspond to the case of NULL, but a .data field is not 
> empty at any time. I'm sure it must have BRCM_SATA_PHY_28NM or BRCM_SATA_PHY_40NM.
> 
> And Sergei, thanks to your good point.

I really do not this as a blocker to get these patches merged, and it
seems like we will miss another merge window on nitpicking if we
continue that route. Can we get these patches merged or are there more
blocking issues to be fixed?
-- 
Florian
--
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