Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1531869 > unrolled thread

Re: [PATCH V12 1/1] usb:serial: Add Fintek F81532/534 driver

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH V12 1/1] usb:serial: Add Fintek F81532/534 driver "Ji-Ze Hong (Peter Hong)" <hpeter@gmail.com> - 2016-11-29 03:40 +0100
    Re: [PATCH V12 1/1] usb:serial: Add Fintek F81532/534 driver Johan Hovold <johan@kernel.org> - 2016-11-29 11:10 +0100

#1531869 — Re: [PATCH V12 1/1] usb:serial: Add Fintek F81532/534 driver

Hi Johan,

Johan Hovold 於 2016/11/24 下午 11:04 寫道:
> On Mon, Nov 14, 2016 at 01:37:59PM +0800, Ji-Ze Hong (Peter Hong) wrote:
>> This driver is for Fintek F81532/F81534 USB to Serial Ports IC.

>> +	}
>> +
>> +	if (size_bulk_out != F81534_WRITE_BUFFER_SIZE ||
>> +			size_bulk_in != F81534_MAX_RECEIVE_BLOCK_SIZE) {
>> +		dev_err(dev, "%s: endpoints packet size not matched\n",
>> +				__func__);
>
> Similarly: "unsupported endpoint max packet size\n".
>
> But just to be clear: You do want to bail out if connected at full
> speed? You could also ask usb-serial core to allocate large enough
> buffers (e.g. by setting the bulk_out_size driver field) and the host
> controller will handle partitioning.

Yes, this product cannot run on USB full-speed currently due to the
bulk-out endpoint design. It need use 512 bytes packet size to transmit
4 ports TX data, but USB Full-speed only provide 64 byte. The F81534
will treat the all bulk-out data as port 0 although the host will spilt
512byte to 8 x 64bytes packets.

-- 
With Best Regards,
Peter Hong

[toc] | [next] | [standalone]

#1532075

On Tue, Nov 29, 2016 at 10:33:16AM +0800, Ji-Ze Hong (Peter Hong) wrote:
> Hi Johan,
> 
> Johan Hovold 於 2016/11/24 下午 11:04 寫道:
> > On Mon, Nov 14, 2016 at 01:37:59PM +0800, Ji-Ze Hong (Peter Hong) wrote:
> >> This driver is for Fintek F81532/F81534 USB to Serial Ports IC.
> 
> >> +	}
> >> +
> >> +	if (size_bulk_out != F81534_WRITE_BUFFER_SIZE ||
> >> +			size_bulk_in != F81534_MAX_RECEIVE_BLOCK_SIZE) {
> >> +		dev_err(dev, "%s: endpoints packet size not matched\n",
> >> +				__func__);
> >
> > Similarly: "unsupported endpoint max packet size\n".
> >
> > But just to be clear: You do want to bail out if connected at full
> > speed? You could also ask usb-serial core to allocate large enough
> > buffers (e.g. by setting the bulk_out_size driver field) and the host
> > controller will handle partitioning.
> 
> Yes, this product cannot run on USB full-speed currently due to the
> bulk-out endpoint design. It need use 512 bytes packet size to transmit
> 4 ports TX data, but USB Full-speed only provide 64 byte. The F81534
> will treat the all bulk-out data as port 0 although the host will spilt
> 512byte to 8 x 64bytes packets.

Ok, thanks for confirming. I'll just fix up the minor comments I had on
the new probe function before applying for 4.10 (so no need to resend).

Thanks for sticking with it.

Johan

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web