Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <ba4ea0d6-b134-c0ef-962c-0adf8b3c924b@nokia.com>
Date:   Wed, 29 Jun 2022 14:21:35 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Subject: Re: [PATCH 02/12] i2c: xiic: Enter standard mode only for > 255 byte
 read transfers
Content-Language: en-US
To:     Manikanta Guntupalli <manikanta.guntupalli@xilinx.com>,
        michal.simek@xilinx.com, michal.simek@amd.com,
        linux-arm-kernel@lists.infradead.org, linux-i2c@vger.kernel.org,
        linux-kernel@vger.kernel.org, git@amd.com
Cc:     Raviteja Narayanam <raviteja.narayanam@xilinx.com>
References: <1656072327-13628-1-git-send-email-manikanta.guntupalli@xilinx.com>
 <1656072327-13628-3-git-send-email-manikanta.guntupalli@xilinx.com>
From:   Krzysztof Adamski <krzysztof.adamski@nokia.com>
In-Reply-To: <1656072327-13628-3-git-send-email-manikanta.guntupalli@xilinx.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dHVqbktDdkZPVnVJSkpTaUU1WkN5a1dBSkdvdld1MURyQWh0Z3l3VmdndXdC?=
 =?utf-8?B?MEhGbzRFOUFvVDJpOUg0YWdqL21ra2tta0lOek1NZktKUFB0cnpHRWExNkla?=
 =?utf-8?B?bHFuTEJ2SDRMZjVzRm51VWhUaUVLWWxmZmpSWW5yUVVmanBxVE42WUNvTDlK?=
 =?utf-8?B?YlJCRS9Pa1BQZWk1QXJXT1ViMXVUQ1BYTWswSHBoazIwZlQrZHh3YXk4NEN3?=
 =?utf-8?B?NzRITStkbUoxVUtMallnVmk3cm1MQjB3aFR6c0VsaWNRM21pcHlMRDFMQ1RB?=
 =?utf-8?B?azluZm9GK2FrLzBYd3l1ZjBRWno3RnhJT2Nmd05QWUxFVVJZanI1VEYwNUJ2?=
 =?utf-8?B?R2dRdnl0bEV0a3hFWHJOWHB0MG9rT3F1RE5wMGg1NVhjYmU5a0NiaXFvV2Mv?=
 =?utf-8?B?aUp2M2tCd215bVZ1RmRNUno0MXpnRVdudklibHB2RXMvdU84RUtlaUc4Nk1B?=
 =?utf-8?B?LzRlMkNFemptS0wzT3BzYk42dEliUU1hdDJncTlId3h1WlUxblF1eTVQQ20v?=
 =?utf-8?B?NE9tbFlkOFBNb1o4MEtjdzA1Q2k5WHM1WXpySXdIcnAyd0JWUkVHcU5jTW9V?=
 =?utf-8?B?T0h0TUV3TjFUOGdVTHZhRWY4Z093Nk9QbU02OXR1c2dRUFcwaVNHUkVGVDkz?=
 =?utf-8?B?eEFQbkVCVTUxSkk4V25vcFJIZXY2TVNGTUxyK2d2WWdONUNiYjlldkk5T3VL?=
 =?utf-8?B?dEFpRU9lVStUTFhpOHdaN2RkeXN2YmUzTEQvWngvMTJjYkQzR1pqeGhLVVEz?=
 =?utf-8?B?K3d4RnU0bDhtZ09WVDdGR3VidHlkQjhXcEYwYzNJZWEydHoyT1Y5dzlhZ0Ra?=
 =?utf-8?B?Z3F0c3lLc2s0U1R4WGlqaFJXQXJULzlOUVZ6a2ZpdTRUL2V2NW9RcUNPT2d2?=
 =?utf-8?B?VXpFVjBXZ2kyekRCa0Vpd0ZjNldWL0pXbFk3YTNwVVRYWDVxcTlMUENncmdF?=
 =?utf-8?B?Rm5xWTdYM0QweWFhYy9oRWhabnRxSmpOMzUwbjd0NHdYQVJxT1JGNWs1Wm0v?=
 =?utf-8?B?M21qQ2h1V3d4b3phci9zV2FyM09DbysrM1Rhcm9OZXh4MHc0d0dBVTcyazM5?=
 =?utf-8?B?eWZ4alZ3d215RnFHcVE2Y0tnaXQrTURjTVVYNUJ5M2RQYXN6d3pOcGxoZEZx?=
 =?utf-8?B?Y09ZTFdjcVQwdVkwYWxmdlZLSzloeXpORnhuSkhrYXlLZnJoeTRHMHpFQUVk?=
 =?utf-8?B?d3ZYaEV6cm1jY2drcHFjOU10WjhCT0FDc0Z1WW13RWlGb2hKK1h1TmxuQlI0?=
 =?utf-8?B?czczTFhtTkNSUVZqMnU4ZS9Ncll2NTlSTkVXcVBZMFhBSHBmU1M2Qmo4T1FU?=
 =?utf-8?B?bzZHUEVJYzR0MVBzNTA3KzEyTmUzMHNBUXdoTzRlaEhTMCtuM0tzZnVqb1Jq?=
 =?utf-8?B?RzhqS05sZlBMb0RzZFVPcUFHb2ppRW95b0FsVEZlczdwSTBKZnhBVmljeEZR?=
 =?utf-8?B?NEkyRWo0RGM1L2ZPSmZYb1NZVU5ST2tETW1ZZWRhMktjMUtzTXBSVmZWbW9l?=
 =?utf-8?B?bGRIRDRmWFgzR2NyOWIxcGEyaGFlVldoSWVpeXN5UlNVcHVCQXZsTkEwbEJj?=
 =?utf-8?B?azkyZXFzeDZVcWRNUlNvNkQyK0s0MG0rQzkrVXlNTXg5S0dxa3hidkppbWFr?=
 =?utf-8?B?b294d3JOUEMxTDlSOFBYQmwyUnVkeHRkQnhrSnBTa085MXF2VXhDZmVpWVRH?=
 =?utf-8?B?MFFKeGlqNkFWa0l6VzZnb0o2LzNNM2xCUkV2NCsxaEJTVkM5WVd2eWRlTzBm?=
 =?utf-8?B?QkwvT1pwMVBTRGhCeVNiWW1wTUIra3NPSzE4ZEppdm1CUXRROVpWQXpnbDZZ?=
 =?utf-8?B?OEdWMlF2R1B1YlpxdC9hbEJXZHZsMGtyQlB2TUk5VENRMjRKYS91UE1BMVNz?=
 =?utf-8?B?dmk5ckFTZTRnYnpxNmY1KzFmaGM1U0dwTVZKL3IyZ3J3RGZUa3VCenpxMGtt?=
 =?utf-8?B?NnhhRkN4T3JjcUxaVERGRThoZmV0TDBpOHMybmZVYmZBZ3BsSlg2Z1FiclNX?=
 =?utf-8?B?dG8rKzc1bUs1UGV2MkxVbVptNlpVUWlDZXNQYkd6VjIrZCs4djdGVjJwOTRN?=
 =?utf-8?B?RnduYUplejB2L0JkZ2t3b29SRWJBeVdpM2M4aldMbVNvV3Y5Ui9iRVp3U0FY?=
 =?utf-8?B?WGdnbG5BMmlPL0xpc1lXU3M4RGJHd0E0NVZ5dVc1WkZNVmZwVzVlNks1dkx1?=
 =?utf-8?B?a2c9PQ==?=
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 20eb677c-0306-4fc4-18a7-08da59c9eab6
X-MS-Exchange-CrossTenant-AuthSource: DU2PR07MB8110.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Jun 2022 12:21:38.9165
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: e73MtzM7FWKKqq3Itf5gbkE9aNdeZ0N+a3WNYouspnYXlcpimSn8pro3o8cEVVMr2N08bRFojqAM4cv5u/7gJox1agBUZQjlFLr0GcRdAIM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR07MB8177
X-Spam-Status: No, score=-1.3 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,FORGED_SPF_HELO,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi,

W dniu 24.06.2022 o 14:05, Manikanta Guntupalli pisze:
> From: Raviteja Narayanam <raviteja.narayanam@xilinx.com>
>
> To maintain backward compatibility the default transfer mode is dynamic
> mode. Enter standard mode only when the size of read transfer is > 255
> bytes.
>
> Signed-off-by: Raviteja Narayanam <raviteja.narayanam@xilinx.com>
> Signed-off-by: Manikanta Guntupalli <manikanta.guntupalli@xilinx.com>

[...]

I don't really understand why this patch isn't squashed into previous 
one. The previous patch was only checking the first message which was 
wrong, this one fixes that.

Krzytof