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