Skip to contentSkip to conversation

patch "phy: Add a driver for dm816x USB PHY" added to linux-phy tree

DISCUSSION Matthijs van Duin 3 participants 7 messages Mar 26, 2015 – Apr 10, 2015 1 message not in the archive

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): 20150325233606.GA7549@a0393678ub

#1 lore raw
Quoted 1 line · Kishon Vijay Abraham I

Let me know if you find any problems with this patch.

I spotted a minor issue in drivers/phy/Kconfig:

Quoted 1 line

+ Enable this for dm81xx USB to work.

This should say dm816x instead.

Matthijs

#2 lore raw
Quoted 8 lines · Matthijs van Duin
Quoted 1 line · Kishon Vijay Abraham I

Let me know if you find any problems with this patch.

I spotted a minor issue in drivers/phy/Kconfig:

Quoted 1 line

+ Enable this for dm81xx USB to work.

This should say dm816x instead.

Yes that's correct, the phy on dm814x seems to be like on am335x, not like on dm816x. Kishon, let me know if you prefer a follow-up patch to fix this.

I'm not seeing the original mail for this thread anywhere for some reason BTW. It does not seem to be in the mailing list archives either, well at least not yet.

Regards,

Tony

#3 lore raw

Hi Tony,

Quoted 6 lines · Tony Lindgren
Quoted 5 lines · Matthijs van Duin
Quoted 1 line · Kishon Vijay Abraham I

Let me know if you find any problems with this patch.

I spotted a minor issue in drivers/phy/Kconfig:

Quoted 7 lines
Quoted 3 lines
Quoted 1 line

+ Enable this for dm81xx USB to work.

This should say dm816x instead.

Yes that's correct, the phy on dm814x seems to be like on am335x, not like on dm816x. Kishon, let me know if you prefer a follow-up patch to fix this.

No. I've fixed this in my tree
https://git.kernel.org/cgit/linux/kernel/git/kishon/linux-phy.git/commit
/?h=next&id=9fa9fd4ffc90582091b8c6219f2054265f71e4ef

Quoted 4 lines

I'm not seeing the original mail for this thread anywhere for some reason BTW. It does not seem to be in the mailing list archives either, well at least not yet.

It's an automated mail that is sent to notify the authors and other who are explicitly added in the signed-off by section when a patch is being merged in the linux-phy tree. So it won't be in the mailing list archives.

But I'm not sure why you didn't receive it :-(

Thanks
Kishon

#4 lore raw
Quoted 13 lines · Kishon Vijay Abraham I

Hi Tony,

Quoted 10 lines · Tony Lindgren
Quoted 8 lines · Matthijs van Duin
Quoted 1 line · Kishon Vijay Abraham I

Let me know if you find any problems with this patch.

I spotted a minor issue in drivers/phy/Kconfig:

Quoted 1 line

+ Enable this for dm81xx USB to work.

This should say dm816x instead.

Quoted 7 lines
Quoted 3 lines

Yes that's correct, the phy on dm814x seems to be like on am335x, not like on dm816x. Kishon, let me know if you prefer a follow-up patch to fix this.

No. I've fixed this in my tree
https://git.kernel.org/cgit/linux/kernel/git/kishon/linux-phy.git/commit
/?h=next&id=9fa9fd4ffc90582091b8c6219f2054265f71e4ef

OK thanks!

Quoted 9 lines
Quoted 3 lines

I'm not seeing the original mail for this thread anywhere for some reason BTW. It does not seem to be in the mailing list archives either, well at least not yet.

It's an automated mail that is sent to notify the authors and other who are explicitly added in the signed-off by section when a patch is being merged in the linux-phy tree. So it won't be in the mailing list archives.

But I'm not sure why you didn't receive it :-(

No idea, maybe I accidentally deleted it or something.

Regards,

Tony

#5 lore raw
Quoted 18 lines · Tony Lindgren
Quoted 17 lines · Kishon Vijay Abraham I

Hi Tony,

Quoted 13 lines · Tony Lindgren
Quoted 8 lines · Matthijs van Duin
Quoted 1 line · Kishon Vijay Abraham I

Let me know if you find any problems with this patch.

I spotted a minor issue in drivers/phy/Kconfig:

Quoted 1 line

+ Enable this for dm81xx USB to work.

This should say dm816x instead.

Yes that's correct, the phy on dm814x seems to be like on am335x, not like on dm816x. Kishon, let me know if you prefer a follow-up patch to fix this.

Quoted 5 lines
Quoted 3 lines

No. I've fixed this in my tree
https://git.kernel.org/cgit/linux/kernel/git/kishon/linux-phy.git/commit
/?h=next&id=9fa9fd4ffc90582091b8c6219f2054265f71e4ef

OK thanks!

Kishon, looks like your next branch is not getting merged into Linux next for some reason?

Regards,

Tony

#6 lore raw

Hi Tony,

Quoted 19 lines · Tony Lindgren
Quoted 18 lines · Tony Lindgren
Quoted 17 lines · Kishon Vijay Abraham I

Hi Tony,

Quoted 13 lines · Tony Lindgren
Quoted 8 lines · Matthijs van Duin
Quoted 1 line · Kishon Vijay Abraham I

Let me know if you find any problems with this patch.

I spotted a minor issue in drivers/phy/Kconfig:

Quoted 1 line

+ Enable this for dm81xx USB to work.

This should say dm816x instead.

Yes that's correct, the phy on dm814x seems to be like on am335x, not like on dm816x. Kishon, let me know if you prefer a follow-up patch to fix this.

Quoted 8 lines
Quoted 5 lines
Quoted 3 lines

No. I've fixed this in my tree
https://git.kernel.org/cgit/linux/kernel/git/kishon/linux-phy.git/commit
/?h=next&id=9fa9fd4ffc90582091b8c6219f2054265f71e4ef

OK thanks!

Kishon, looks like your next branch is not getting merged into Linux next for some reason?

Greg merged it just now. So you should start seeing tomorrow.

Cheers
Kishon

#7 lore raw
Quoted 26 lines · Kishon Vijay Abraham I

Hi Tony,

Quoted 23 lines · Tony Lindgren
Quoted 22 lines · Tony Lindgren
Quoted 20 lines · Kishon Vijay Abraham I

Hi Tony,

Quoted 6 lines · Tony Lindgren
Quoted 5 lines · Matthijs van Duin
Quoted 1 line · Kishon Vijay Abraham I

Let me know if you find any problems with this patch.

I spotted a minor issue in drivers/phy/Kconfig:

Quoted 7 lines
Quoted 3 lines
Quoted 1 line

+ Enable this for dm81xx USB to work.

This should say dm816x instead.

Yes that's correct, the phy on dm814x seems to be like on am335x, not like on dm816x. Kishon, let me know if you prefer a follow-up patch to fix this.

No. I've fixed this in my tree
https://git.kernel.org/cgit/linux/kernel/git/kishon/linux-phy.git/commit
/?h=next&id=9fa9fd4ffc90582091b8c6219f2054265f71e4ef

Quoted 6 lines
Quoted 4 lines
Quoted 1 line

OK thanks!

Kishon, looks like your next branch is not getting merged into Linux next for some reason?

Greg merged it just now. So you should start seeing tomorrow.

OK thanks!

Tony